Bug 234555

Summary: [JSC] Remove m_calleeSaveRegisters from CodeBlock::JITData and rename it to BaselineJITData
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: New BugsAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, saam, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
ews-feeder: commit-queue-
Patch
none
Patch none

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
Patch (52.40 KB, patch)
2021-12-21 04:13 PST, Yusuke Suzuki
ews-feeder: commit-queue-
Patch (52.46 KB, patch)
2021-12-21 04:25 PST, Yusuke Suzuki
no flags
Patch (53.18 KB, patch)
2021-12-25 03:43 PST, Yusuke Suzuki
no flags
Yusuke Suzuki
Comment 1 2021-12-21 03:06:07 PST
Yusuke Suzuki
Comment 2 2021-12-21 04:13:46 PST
Yusuke Suzuki
Comment 3 2021-12-21 04:25:32 PST
Yusuke Suzuki
Comment 4 2021-12-25 03:43:31 PST
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
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.