[cfe-commits] r161703 - in /cfe/trunk: include/clang/AST/Stmt.h lib/AST/Stmt.cpp

[Chad Rosier]

On Aug 15, 2012, at 9:27 AM, Chad Rosier wrote:

> 
> On Aug 15, 2012, at 1:45 AM, Enea Zaffanella wrote:
> 
>> On 08/14/2012 08:56 PM, Abramo Bagnara wrote:
>>> Il 14/08/2012 20:19, Richard Smith ha scritto:
>> 
>> [...]
>> 
>>>> What about these:
>>>> 
>>>>    if (x)
>>>>       __asm out dx, ax  __asm out dx, ax
>>>>    if (x)
>>>>       __asm out dx, ax ; __asm out dx, ax
>>>>    if (x)
>>>>        __asm out dx, ax ; out dx, ax
>>>> 
>>>> To my reading, the first two are two statements and the third is one
>>>> statement.
>>> 
>>> Unfortunately not: in the first there is one statement formed by two
>>> "out dx, ax", in the second and the third the text after semicolon is
>>> ignored.
>> 
>> 
>> It seems that the first case above is still mis-parsed in r161931, producing two MSAsmStmt nodes instead of a single one.
> 
> I'll take care of this shortly..

This should be fixed as of r161986.

> Chad
> 
>> 
>> Enea.
>> 
>