[PATCH] Set "thumbv6k" CPU subtype to ARM_V6

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 10:22:39 PDT 2015


Nice. I’m not familiar with this code so I’ve got a few more questions: Why is armv6 the only case where we need to specify this explicitly? Do the other cases all default to something sane?

— adrian


> On Aug 11, 2015, at 10:07 AM, Vedant Kumar <vsk at apple.com> wrote:
> 
> Hi Adrian,
> 
> Yes, the updated patch has a test case.
> 
> thanks
> vedant
> 
> <armv6-thumbv6k.patch>
> 
> 
>> On Aug 10, 2015, at 1:58 PM, Adrian Prantl <aprantl at apple.com> wrote:
>> 
>> Would it be possible to write a test case for this?
>> 
>> -- adrian
>>> On Aug 10, 2015, at 1:37 PM, Vedant Kumar via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>>> 
>>> We currently produce ARMv7 machine code when the architecture is "thumbv6k".
>>> 
>>> This patch makes llvm emit ARMv6 code in this situation.
>>> 
>>> I don't have commit rights. Could someone take a look at this and commit it for me?
>>> 
>>> thanks
>>> vedant
>>> 
>>> <armv6-thumbv6k.patch>_______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>> 
> 



More information about the llvm-commits mailing list