Bug 171141

Summary: Remove LL/SC from Atomics
Product: WebKit Reporter: Keith Miller <keith_miller>
Component: New BugsAssignee: Keith Miller <keith_miller>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, buildbot, cdumez, cmarcelo, dbates, jfbastien, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch saam: review+

Keith Miller
Reported 2017-04-21 13:59:19 PDT
Remove LL/SC from Atomics
Attachments
Patch (14.16 KB, patch)
2017-04-21 14:00 PDT, Keith Miller 		no flags
DetailsFormatted DiffDiff
Patch (13.78 KB, patch)
2017-04-21 14:07 PDT, Keith Miller 		saam: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Keith Miller
Comment 1 2017-04-21 14:00:44 PDT
Created attachment 307795 [details] Patch
Keith Miller
Comment 2 2017-04-21 14:07:34 PDT
Created attachment 307797 [details] Patch
Saam Barati
Comment 3 2017-04-21 14:08:19 PDT
Comment on attachment 307797 [details] Patch r=me please also remove the code that defines HAVE_LL_SC unless it's used elsewhere.
JF Bastien
Comment 4 2017-04-21 14:10:42 PDT
r=metoo
Keith Miller
Comment 5 2017-04-21 14:13:26 PDT
(In reply to Saam Barati from comment #3) > Comment on attachment 307797 [details] > Patch > > r=me > please also remove the code that defines HAVE_LL_SC unless it's used > elsewhere. done.
Keith Miller
Comment 6 2017-04-25 10:21:39 PDT
Idk what happened to the changelog but this landed in: https://trac.webkit.org/changeset/215637/webkit
Note You need to log in before you can comment on or make changes to this bug.