RESOLVED FIXED 234555
[JSC] Remove m_calleeSaveRegisters from CodeBlock::JITData and rename it to BaselineJITData 
https://bugs.webkit.org/show_bug.cgi?id=234555
Summary [JSC] Remove m_calleeSaveRegisters from CodeBlock::JITData and rename it to B...
Yusuke Suzuki
Reported 2021-12-21 03:05:41 PST
[JSC] Remove m_calleeSaveRegisters from CodeBlock::JITData and rename it to BaselineJITData
Attachments
Patch (34.62 KB, patch)
2021-12-21 03:06 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (52.40 KB, patch)
2021-12-21 04:13 PST, Yusuke Suzuki 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (52.46 KB, patch)
2021-12-21 04:25 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (53.18 KB, patch)
2021-12-25 03:43 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2021-12-21 03:06:07 PST
Created attachment 447707 [details] Patch
Yusuke Suzuki
Comment 2 2021-12-21 04:13:46 PST
Created attachment 447710 [details] Patch
Yusuke Suzuki
Comment 3 2021-12-21 04:25:32 PST
Created attachment 447711 [details] Patch
Yusuke Suzuki
Comment 4 2021-12-25 03:43:31 PST
Created attachment 447958 [details] Patch
Yusuke Suzuki
Comment 5 2021-12-25 03:46:54 PST
Comment on attachment 447958 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=447958&action=review > Source/JavaScriptCore/llint/LLIntSlowPaths.cpp:481 > + if (*ptr >= Options::earlyReturnFromInfiniteLoopsLimit()) > LLINT_RETURN_TWO(LLInt::fuzzerReturnEarlyFromLoopHintEntrypoint().code().executableAddress(), callFrame->topOfFrame()); We do not need ensureJITData since we no longer load a pointer from JITData (we removed that indirection).
Radar WebKit Bug Importer
Comment 6 2021-12-28 03:06:16 PST
<rdar://problem/86954269>
Saam Barati
Comment 7 2022-01-04 13:16:33 PST
Comment on attachment 447958 [details] Patch r=me
EWS
Comment 8 2022-01-04 16:21:52 PST
Committed r287596 (245724@main): <https://commits.webkit.org/245724@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 447958 [details].
Note You need to log in before you can comment on or make changes to this bug.