[PATCH] Make v7(e)m default to thumb

James Molloy james at jamesmolloy.co.uk
Tue Oct 15 08:45:05 PDT 2013


Hi Bernie,

Generally LGTM - I'd prefer a testcase testing v6m too though (Cortex-M0 or
M0+).

Cheers,

James


On 15 October 2013 15:53, Bernie Ogden <bogden at arm.com> wrote:

> Attached simple patch makes thumb the default for armv7m and armv7em. Given
> that these targets don't support arm, this has to be a Good Thing.
>
> Fixes http://llvm.org/bugs/show_bug.cgi?id=17559.
>
> Note that this only fixes the clang side. M'colleague pointed out that
> other
> LLVM tools (e.g. llc) are broken in this respect. I took a look at what
> would be required to fix this and decided not to go there.
>
> Please review.
>
> Regards,
>
> Bernie
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131015/34ba1855/attachment.html>


More information about the llvm-commits mailing list