Created attachment 229095 [details]
Comment on attachment 229095 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=229095&action=review
Could you remove the environment variable thingy?
Did you perf test this??
Extra whitespace here!
Transmitting file data .....
Committed revision 167102.
D'oh, forgot review comments. :-(
whitespace fixed in.
Transmitting file data ...
Committed revision 167103.
This just cost me a day of debugging.
If JSC decides to turn off the FTL JIT, then it shouldn't be possible for WK2 to turn it on.
I don't think that flipping JSC's Options directly is an appropriate way to do this. I will roll it out with MarkH's rubber stamp.
Rolled out in r168120.
We need to find a way of doing this that still allows JSC to disable the FTL JIT regardless of external settings.