Bug 204701

Summary: Use default implementation for JSVALUE64 with GCC on unknown platform.
Product: WebKit Reporter: Eike Rathke <erack>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: berto, commit-queue, ews-watchlist, jbedard, jlewis3, keith_miller, mark.lam, msaboff, saam, simon.fraser, tsavell, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Eike Rathke
Reported 2019-11-29 06:54:00 PST
Use default implementation for JSVALUE64 with GCC on unknown platform.
Attachments
Patch (3.19 KB, patch)
2019-11-29 06:57 PST, Eike Rathke 		no flags
DetailsFormatted DiffDiff
Patch (2.54 KB, patch)
2019-11-29 07:02 PST, Eike Rathke 		no flags
DetailsFormatted DiffDiff
Patch (1.95 KB, patch)
2019-11-29 07:08 PST, Eike Rathke 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Eike Rathke
Comment 1 2019-11-29 06:57:01 PST
Created attachment 384504 [details] Patch
Eike Rathke
Comment 2 2019-11-29 07:02:43 PST
Created attachment 384505 [details] Patch
Eike Rathke
Comment 3 2019-11-29 07:06:19 PST
Grml.. webkit-patch insists on including it's on modification to force python2 even if already committed. I'll resubmit a manual patch.
Eike Rathke
Comment 4 2019-11-29 07:08:58 PST
Created attachment 384506 [details] Patch
Eike Rathke
Comment 5 2019-11-29 07:35:31 PST
Note: the style checker seems to be wrong in its analysis.
Saam Barati
Comment 6 2019-11-29 10:13:37 PST
Comment on attachment 384506 [details] Patch r=me
WebKit Commit Bot
Comment 7 2019-11-29 11:03:42 PST
Comment on attachment 384506 [details] Patch Clearing flags on attachment: 384506 Committed r252949: <https://trac.webkit.org/changeset/252949>
WebKit Commit Bot
Comment 8 2019-11-29 11:03:44 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9 2019-11-29 11:04:19 PST
<rdar://problem/57534280>
Alberto Garcia
Comment 10 2019-12-01 10:18:49 PST
*** Bug 204643 has been marked as a duplicate of this bug. ***
Jonathan Bedard
Comment 11 2019-12-17 08:23:29 PST
I think this broke Windows jsc tests: https://build.webkit.org/builders/Apple%20Win%2010%20Debug%20%28Tests%29?numbuilds=200 We don't have a super great range, unfortunately, since the build was broken, but this its the only commit which touched JSC stuff in https://results.webkit.org/commits?before_id=252951&after_id=252943
Eike Rathke
Comment 12 2019-12-18 05:24:00 PST
Only if the Windows build suddenly uses GCC on a non-x86-64 platform..
Jonathan Bedard
Comment 13 2019-12-18 07:52:08 PST
(In reply to Eike Rathke from comment #12) > Only if the Windows build suddenly uses GCC on a non-x86-64 platform.. That's what Saam said. Mark is looking into it (or at least was yesterday), even taking a deeper dive into the commits in our regression range, I don't see another commit that even touches JSC.
Note You need to log in before you can comment on or make changes to this bug.