[Webkit-unassigned] [Bug 74023] Compare and Swap should be enabled on ARMv7

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=74023





--- Comment #4 from Filip Pizlo <fpizlo at apple.com>  2011-12-07 14:30:59 PST ---
(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.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.