Bug 68794 – Refactor code for JSVALUE32_64 DFG JIT

• Component: JavaScriptCore
• Status: RESOLVED INVALID

[Yuqiang Xian]

JSVALUE32_64 DFG JIT code should share JSVALUE64 code as much as possible.

[Yuqiang Xian]

Created attachment 108649 [details]
WIP patch

This is the initial patch (WIP). Basically,

1) DFGJITCompiler.cpp and DFGJITCompiler32_64.cpp are merged, and some old unused routines for original spec->nonspec purpose have been removed.
2) Commonly used helper functions in DFGJITCodeGenerator have been extracted out to a newly created file "DFGJITCodeGeneratorHelpers.cpp". DFGJITCodeGenerator.cpp and DFGJITCodeGenerator32_64.cpp now only contain format specific nonSpeculative opcode implementations.
3) A DFG node is marked with more flags to indicate whether it should consume or produce a JS Value. For those nodes that don't involve JS values the code between JSVALUE64 and JSVALUE32_64 is shared in SpeculativeJIT. Currently we put all the stuffs into the single big DFGSpeculativeJIT.cpp file with "ifdef"s. Not sure whether we need to split them into different files. Also some problems still exist especially for ValueAdd and ArithAdd, where both nodes share some code but ValueAdd is format specific while ArithAdd is format neutral. So current WIP patch still has some unnecessary duplicates. Ideas on improving this are highly appreciated. Thanks a lot!

[Yuqiang Xian]

Created attachment 108818 [details]
updated patch

This patch mainly splits the stuffs in the big DFGSpeculativeJIT.cpp file of the original patch into multiple files (Common, 64 and 32_64). Also rename the DFGJITCodeGenerator*.cpp to match the DFGSpeculativeJIT*.cpp naming conventions.

[Early Warning System Bot]

Comment on attachment 108818 [details]
updated patch

Attachment 108818 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/9883202

[Gustavo Noronha (kov)]

Comment on attachment 108818 [details]
updated patch

Attachment 108818 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/9877301

[Yuqiang Xian]

Created attachment 108844 [details]
patch

This should fix the qt and gtk ports issues.

[Gavin Barraclough]

Comment on attachment 108844 [details]
patch

r- because this is way too much to be doing in one change.  Per my email, I'd suggest letting me make some of these changes since I can probably get a faster turnaround on getting the bulk of them in.

[Gavin Barraclough]

This patch won't still apply, much refactoring has taken place, there may still be more to do but I don't think this bug is still tracking active work.