Bug 80080 – WebKit compiled by gcc (Xcode 3.2.6) hangs while running DOM/Accessors.html

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Template Framework

[Ryosuke Niwa]

I attempted a fix in http://trac.webkit.org/changeset/109478 but it had to be reverted due to a build failure: http://trac.webkit.org/changeset/109481

Not sure what the right fix is here.

[Filip Pizlo]

(In reply to comment #0)
> I attempted a fix in http://trac.webkit.org/changeset/109478 but it had to be reverted due to a build failure: http://trac.webkit.org/changeset/109481
> 
> Not sure what the right fix is here.

The build failure is due to the fact that on 32-bit, uintptr_t and unsigned might be the same type.  Well, they're always compatible in type in the sense that unsigned and uintptr_t will both be 32-bit but there's an open question as to whether uintptr_t is "unsigned int" or "unsigned long".

The most robust solution is to rename the two weakCompareAndSwaps to:

weakCompareAndSwapUnsigned(...)

and:

weakCompareAndSwapPointer(...)

Then there won't be problems.

[Ryosuke Niwa]

Created attachment 131123 [details]
My desparate attempt to fix JSC

[Filip Pizlo]

Comment on attachment 131123 [details]
My desparate attempt to fix JSC

I like this approach - this side-steps all manner of C++ overload strangeness due to the plethora of ways that uintptr_t can be typedef'd.  R=me.

[Ryosuke Niwa]

Committed r110342: <http://trac.webkit.org/changeset/110342>