[webkit-reviews] review requested: [Bug 186765] [Armv7] Linkbuffer: executableOffsetFor() fails for location 2 : [Attachment 342936] Patch

[bugzilla-daemon at webkit.org]

Guillaume Emont <guijemont at igalia.com> has asked  for review:
Bug 186765: [Armv7] Linkbuffer: executableOffsetFor() fails for location 2
https://bugs.webkit.org/show_bug.cgi?id=186765

Attachment 342936: Patch

https://bugs.webkit.org/attachment.cgi?id=342936&action=review




--- Comment #1 from Guillaume Emont <guijemont at igalia.com> ---
Created attachment 342936

  --> https://bugs.webkit.org/attachment.cgi?id=342936&action=review

Patch

This seems to fix it. I'm unsure of how
recordLinkOffsets()/executableOffsetFor() are supposed to work on thumb2, as
they seem to assume that jump origins and destinations are 4-byte aligned,
though I might be misunderstanding something, as it looks like this has been
working like that since the introduction of thumb2 support.