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

[Bernie Ogden]

Hi James,

 

There's already a test there, but it relies on the target being darwin. We
ought to be testing in general. I'll fix that.

 

Thanks,

 

Bernie

 

From: mankeyrabbit at gmail.com [mailto:mankeyrabbit at gmail.com] On Behalf Of
James Molloy
Sent: 15 October 2013 16:45
To: Bernard Ogden
Cc: LLVM Commits
Subject: Re: [PATCH] Make v7(e)m default to thumb

 

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/20131016/7570b7f8/attachment.html>