Bug 221710

Summary: [JSC] cloop broken on 32-bit big endian
Product: WebKit Reporter: Daniel Kolesa <dkolesa>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, saam, tzagallo, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Other   
OS: All   
Attachments:
Description Flags
patch
ysuzuki: review+, ews-feeder: commit-queue-
patch none

Daniel Kolesa
Reported 2021-02-10 12:18:16 PST
Right now, JSC CLoop is broken (segfaults) on 32-bit BE systems (LE is fine, as is 64-bit LE and BE).
Attachments
patch (2.48 KB, patch)
2021-02-10 12:19 PST, Daniel Kolesa 		ysuzuki: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
patch (2.46 KB, patch)
2021-02-18 02:41 PST, Daniel Kolesa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Daniel Kolesa
Comment 1 2021-02-10 12:19:21 PST
Created attachment 419887 [details] patch
Radar WebKit Bug Importer
Comment 2 2021-02-17 12:19:13 PST
<rdar://problem/74444921>
Yusuke Suzuki
Comment 3 2021-02-17 12:23:04 PST
Comment on attachment 419887 [details] patch r=me
EWS
Comment 4 2021-02-17 13:10:41 PST
commit-queue failed to commit attachment 419887 [details] to WebKit repository. To retry, please set cq+ flag again.
Daniel Kolesa
Comment 5 2021-02-18 02:41:49 PST
Created attachment 420817 [details] patch updated changelog
Daniel Kolesa
Comment 6 2021-02-18 02:49:02 PST
oh, actually looks like it was just a failed push? anything else needed from my side here?
Yusuke Suzuki
Comment 7 2021-02-18 12:38:08 PST
(In reply to Daniel Kolesa from comment #6) > oh, actually looks like it was just a failed push? > > anything else needed from my side here? Yeah, you need to put cq? and committer/reviewer can make it cq+ :)
EWS
Comment 8 2021-02-18 15:20:48 PST
Committed r273104: <https://commits.webkit.org/r273104> All reviewed patches have been landed. Closing bug and clearing flags on attachment 420817 [details].
Note You need to log in before you can comment on or make changes to this bug.