This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] 16/n: trans-mem: compiler parser/front-end

From: Richard Henderson <rth at redhat dot com>
To: "Joseph S. Myers" <joseph at codesourcery dot com>
Cc: Aldy Hernandez <aldyh at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 04 Nov 2011 11:54:31 -0700
Subject: Re: [patch] 16/n: trans-mem: compiler parser/front-end
References: <4EB2E588.1040806@redhat.com> <Pine.LNX.4.64.1111032030120.31547@digraph.polyomino.org.uk> <4EB2FD4B.4000306@redhat.com>

On 11/03/2011 01:44 PM, Richard Henderson wrote:
> On 11/03/2011 01:34 PM, Joseph S. Myers wrote:
>> On Thu, 3 Nov 2011, Aldy Hernandez wrote:
>>
>>> +/* Parse a transaction attribute (GCC Extension).
>>> +
>>> +   transaction-attribute:
>>> +     attributes
>>> +     [ [ any-word ] ]
>>
>> I don't see any syntax production comments including 
>> transaction-attribute.  Please update the comments to include this in 
>> every case where it is permitted.
> 
> Ah, the comment in question is currently
> 
>   __transaction_atomic attributes[opt] compound-statement
> 
> where that should be transaction-attributes[opt] instead.
> 
> Will fix.

Like so.


r~

Attachment: z
Description: Text document

References:
- [patch] 16/n: trans-mem: compiler parser/front-end
  - From: Aldy Hernandez
- Re: [patch] 16/n: trans-mem: compiler parser/front-end
  - From: Joseph S. Myers
- Re: [patch] 16/n: trans-mem: compiler parser/front-end
  - From: Richard Henderson

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]