In fillSpeculateCell and OSR exit, some unnecessary loads/stores can be eliminated.
Created attachment 112808 [details] the patch
Comment on attachment 112808 [details] the patch r=me
Comment on attachment 112808 [details] the patch Rejecting attachment 112808 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ed at 1 with fuzz 3. patching file Source/JavaScriptCore/dfg/DFGJITCompiler32_64.cpp Hunk #1 succeeded at 237 (offset 24 lines). Hunk #2 succeeded at 373 (offset 24 lines). patching file Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp Hunk #1 FAILED at 295. 1 out of 1 hunk FAILED -- saving rejects to file Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Filip Pizlo', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/10257430
Looks like this needs some merging, since I touched some of that same code. Yuqiang, can you merge this and land manually? R=me still applies.
Created attachment 113407 [details] patch patch merging the latest changes.
(In reply to comment #4) > Looks like this needs some merging, since I touched some of that same code. Yuqiang, can you merge this and land manually? R=me still applies. Thanks, the new patch has been uploaded. Can you please set cq+ if you're fine with it? I still don't have the commit access (though it's in process) by now.
Comment on attachment 113407 [details] patch Clearing flags on attachment: 113407 Committed r99129: <http://trac.webkit.org/changeset/99129>
All reviewed patches have been landed. Closing bug.