[Webkit-unassigned] [Bug 19775] Alignment problems on Sparc

[bugzilla-daemon at webkit.org]

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





------- Comment #2 from mh+webkit at glandium.org  2008-06-26 10:50 PDT -------
(In reply to comment #1)
> This problem may have to do with this FIXME: in Vector.h: "Nothing guarantees
> this buffer is appropriately aligned to hold objects of type T". You could use
> non-standard gcc extension __align__ to make vector inline buffer aligned.
> Alternatively, you could make the buffer an array of elements that have
> suitable alignment requirements for a cross-compiler solution (patches are
> welcome!)

I'll try to try these, but will have to do some research first... Or maybe do
you have good pointers already?

> What are the cases where Sparc requires 64-bit alignment? Would an array of
> size32_t elements without padding be problematic, for example?

Basically, any 32-bit access must be 32-bit aligned, as with ARM, and any
64-bit access must be 64-bit aligned. Some assignments end up doing 64-bit
accesses.

> Apple's sources
> for iPhone have some alignment fixes for ARM, which you can look up at Apple's
> open source site, if you are OK with agreeing to the Apple Public Source
> License, but I don't know if Sparc requirements are the same.

AFAIK, ARM only requires 32-bit alignment, not 64-bit alignment. ARM alignment
requirements are already fulfilled by webkit code, apparently.


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