[llvm-commits] [llvm] r56511 - in /llvm/trunk: include/llvm/ lib/AsmParser/ lib/Bitcode/Reader/ lib/Bitcode/Writer/ lib/Target/X86/AsmPrinter/ lib/Transforms/IPO/ lib/Transforms/Scalar/ lib/Transforms/Utils/ lib/VMCore/

[Chris Lattner]

On Sep 23, 2008, at 5:14 PM, Devang Patel wrote:

>> I think we really want:
>>
>> declare ...result attrs... i32 @foo( ... param attrs ...)
>> nounwind ...fnattrs... {
>>
>> I don't think we need the notes syntax at all, do you agree?
>
> I like result attributes!
> Are fnattrs allowed  anywhere before { ?

Yep, I think we should migrate things like 'nounwind' to be function  
attrs (index ~0U) instead of result attrs (index 0).  We need to stay  
backwards compatible though.

-Chris