[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 4:53 PM, Devang Patel wrote:

>
> On Sep 23, 2008, at 4:51 PM, Chris Lattner wrote:
>
>> On Sep 23, 2008, at 4:27 PM, Devang Patel wrote:
>>
>>>> Is the syntax of notes in .ll files going to change to follow
>>>> attributes?
>>>
>>> No. We'll continue to use notes() syntax.
>>
>> Why?
>
> I am confused. What are you proposing?

Currently we have:

declare i32 @foo( ... param attrs ...) nounwind ...fnattrs...  
notes(...notes...) {


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?

-Chris