Bug 149436

Summary: Get rid of ENABLE(PARALLEL_GC)
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, barraclough, benjamin, bfulgham, cmarcelo, commit-queue, ggaren, mark.lam, mhahnenb, msaboff, nrotem, oliver, saam, sam
Priority: P2    
Version: Other   
Hardware: All   
OS: All   
Bug Depends on: 149438    
Bug Blocks: 149435    
Attachments:
Description Flags
the patch
mark.lam: review+
patch for landing none

Filip Pizlo
Reported 2015-09-21 17:52:49 PDT
We no longer really use platforms where CAS is unavailable.
Attachments
the patch (15.54 KB, patch)
2015-09-22 12:00 PDT, Filip Pizlo 		mark.lam: review+
DetailsFormatted DiffDiff
patch for landing (15.58 KB, patch)
2015-09-22 12:27 PDT, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2015-09-22 12:00:59 PDT
Created attachment 261757 [details] the patch
Filip Pizlo
Comment 2 2015-09-22 12:03:21 PDT
The main side effect of this is that this enables parallel GC on Windows. I don't see anything POSIX-specific in our parallel GC implementation, so I think that this is OK.
Mark Lam
Comment 3 2015-09-22 12:04:11 PDT
Comment on attachment 261757 [details] the patch r=me
Filip Pizlo
Comment 4 2015-09-22 12:27:18 PDT
Created attachment 261760 [details] patch for landing Fixes Windows, hopefully.
Alex Christensen
Comment 5 2015-09-22 13:37:16 PDT
basic browsing on windows seems to work as well as it does without this patch.
Filip Pizlo
Comment 6 2015-09-22 13:43:51 PDT
Landed in http://trac.webkit.org/changeset/190131
Note You need to log in before you can comment on or make changes to this bug.