[patch] Enabling EH in MCJIT

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri May 3 08:23:59 PDT 2013


On 2 May 2013 17:31, Kaylor, Andrew <andrew.kaylor at intel.com> wrote:
> Hi Rafael,
>
> I've read through this quickly, but I haven't had time for a careful review yet.  On first pass, it looks pretty good.  Thanks for returning to this issue!
>
> Have you tested your patch with ARM?  As I recall the extra size thing was introduced for ELF on ARM, but the way you're handling it may be sufficient.

Thanks for the reminder. I have now checked it on chromebook and the
results were surprising: there is no __register_frame :-(

I checked the old JIT and it disables the EH bits on ARM, so I have
done the same in this new patch.

> -Andy
>

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: application/octet-stream
Size: 13268 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130503/0b25128a/attachment.obj>


More information about the llvm-commits mailing list