[PATCH] Enable ARM tail calls for ELF

Saleem Abdulrasool compnerd at compnerd.org
Mon Jul 1 20:21:12 PDT 2013


On Sun, Jun 30, 2013 at 3:12 AM, Tim Northover <t.p.northover at gmail.com>wrote:

> Hi,
>
> The attached patch enables tail calls for every target that hasn't
> explicitly been marked as bad (iOS < 5.0). This amounts to any ELF
> target and being more liberal on some ghost MachO targets (e.g.
> arm-apple-darwin9) that no-one probably cares about.
>
> I think the ELF side is justified because the required linker support
> is there in the ARM ELF specification so anywhere it doesn't work has
> a broken linker. I've also tested it on Linux and our MCJIT dynamic
> loader with no issues.
>
> Does anyone object to the change?
>

Minor nit-pick: I think that it would be "cleaner" to actually set the
default to true in the initializeEnvironment function, and allow the iOS
targets override it as per the iOS version.


> Cheers.
>
> Tim.
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130701/f62579c5/attachment.html>


More information about the llvm-commits mailing list