[cfe-commits] [PING^2] Handling of target attributes
clattner at apple.com
Sat Jan 9 17:07:06 PST 2010
On Jan 9, 2010, at 5:00 PM, Anton Korobeynikov wrote:
> Hi, Chris
>> + d->addAttr(::new (S.Context) MSP430InterruptAttr(Num));
>> + d->addAttr(::new (S.Context) NoInlineAttr());
>> + d->addAttr(::new (S.Context) UsedAttr());
>> It seems that you should only add the MSP430InterruptAttr attribute to the decl,
>> but that codegen should add the other attributes to the LLVM IR function.
>> Is there a reason to add noinline and used to the AST?
> There is no reason for noinline, I agree, since it's just a function
> attribute. Used is quite different, since it requires special codegen
> support and I don't want to reinvent the wheel.
> So, basically, I will move noinline to codegen part, but keep used here.
More information about the cfe-commits