RESOLVED FIXED 149436
Get rid of ENABLE(PARALLEL_GC) 
https://bugs.webkit.org/show_bug.cgi?id=149436
Summary Get rid of ENABLE(PARALLEL_GC)
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.