[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/

[Devang Patel]

On Sep 23, 2008, at 5:04 PM, Chris Lattner wrote:

>
> 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?

I like result attributes!
Are fnattrs allowed  anywhere before { ?
-
Devang