We should enable WREC on x86-64. This would be a >10% progression on Sunspider.
Created attachment 25595 [details] The patch, not marking for review yet. I'm not marking this for review, since I'm not sure whether we want to turn this on by default yet - currently I believe this will mark the whole heap RWX if WREC is enabled, which may be a bad thing.
Oh, that is, on PLATFORM(X86_64) the whole heap would be executable, if WREC is compiled in – it wouldn't affect other platforms. Still could be a bad idea.
I would be willing to turn it on only for 64-bit Mac OS. We really need to fix bug 21783, though.
Created attachment 25621 [details] New patch
Comment on attachment 25621 [details] New patch > #if ENABLE(ASSEMBLER) && (PLATFORM(GTK) || PLATFORM(X86_64)) This should only be enabled on 64-bit Mac OS, not all x86-64. Also, mention what you are doing with wx pages in the ChangeLog.
Sending JavaScriptCore/ChangeLog Sending JavaScriptCore/assembler/MacroAssembler.h Sending JavaScriptCore/assembler/X86Assembler.h Sending JavaScriptCore/wrec/WREC.h Sending JavaScriptCore/wrec/WRECGenerator.cpp Sending JavaScriptCore/wtf/Platform.h Sending JavaScriptCore/wtf/TCSystemAlloc.cpp Transmitting file data ....... Committed revision 38857.