RESOLVED FIXED 22554
Enable WREC on x86-64 
https://bugs.webkit.org/show_bug.cgi?id=22554
Summary Enable WREC on x86-64
Gavin Barraclough
Reported 2008-11-29 04:45:52 PST
We should enable WREC on x86-64. This would be a >10% progression on Sunspider.
Attachments
The patch, not marking for review yet. (13.88 KB, patch)
2008-11-29 04:52 PST, Gavin Barraclough 		no flags
DetailsFormatted DiffDiff
New patch (14.97 KB, patch)
2008-11-30 23:24 PST, Gavin Barraclough 		zwarich: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2008-11-29 04:52:38 PST
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.
Gavin Barraclough
Comment 2 2008-11-29 04:54:01 PST
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.
Cameron Zwarich (cpst)
Comment 3 2008-11-29 05:42:02 PST
I would be willing to turn it on only for 64-bit Mac OS. We really need to fix bug 21783, though.
Gavin Barraclough
Comment 4 2008-11-30 23:24:54 PST
Created attachment 25621 [details] New patch
Cameron Zwarich (cpst)
Comment 5 2008-11-30 23:33:38 PST
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.
Gavin Barraclough
Comment 6 2008-12-01 01:18:43 PST
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.
Note You need to log in before you can comment on or make changes to this bug.