[llvm-commits] [llvm] r72593 - in /llvm/trunk/lib/Target/ARM: ARM.td ARMInstrInfo.td ARMInstrThumb2.td ARMSubtarget.cpp ARMSubtarget.h

[Evan Cheng]

Sent from my iPhone

On May 30, 2009, at 12:48 PM, Anton Korobeynikov <anton at korobeynikov.info 
 > wrote:

> Hello, Evan
>
>> I should clarify. I think triplet should be used to specify arch,  
>> i.e.
>> arm / thumb. -mcpu and -mattr should be used to control what
>> instructions are available to isel.
> Ok, this makes sense, I'll fix the code. I'd like to outline some
> inconsistency here: it seems that currently we're encoding ARM
> architecture number in the target triple, should this be fixed
> someday?

That's an unfortunate workaround for lto. When the linker invokes the  
code generator it doesn't know the arm variant to set the right  
attributes. Yes, it should be fixed one day by encoding them as  
function notes.

Evan

>
> -- 
> With best regards, Anton Korobeynikov
> Faculty of Mathematics and Mechanics, Saint Petersburg State  
> University
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits