And PARALLEL_GC should be enabled, as well.
Created attachment 118263 [details] the patch
Created attachment 118265 [details] the patch Removed accidental debug goop in build-jsc.
Comment on attachment 118265 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=118265&action=review > Source/JavaScriptCore/wtf/Atomics.h:140 > + "strex %1, %4, %0\n\t" I think you need to test and branch after strex to see if your strex succeeded, no? http://www.doulos.com/knowhow/arm/Hints_and_Tips/Implementing_Semaphores/
(In reply to comment #3) > (From update of attachment 118265 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=118265&action=review > > > Source/JavaScriptCore/wtf/Atomics.h:140 > > + "strex %1, %4, %0\n\t" > > I think you need to test and branch after strex to see if your strex succeeded, no? http://www.doulos.com/knowhow/arm/Hints_and_Tips/Implementing_Semaphores/ weakCompareAndSwap returns a boolean that tells you if the strex succeeded. So the inline asm doesn't need to branch.
Comment on attachment 118265 [details] the patch r=me
Landed in http://trac.webkit.org/changeset/102295
Comment on attachment 118265 [details] the patch Clearing flags.