[cfe-commits] [PATCH] Add the readonly attribute to inline asm

Eli Friedman eli.friedman at gmail.com
Wed May 2 17:21:14 PDT 2012


On Wed, May 2, 2012 at 3:38 PM, Chad Rosier <mcrosier at apple.com> wrote:
> And here's the patch in case anyone would like to take a look.. :D
>
>  Chad
>
>
>
>
> On May 2, 2012, at 3:34 PM, Chad Rosier wrote:
>
>> The attached patch adds the readonly attribute to inline assembly if the asm is not marked volatile and it does not clobber memory or condition codes.

You're missing a check here: you also need to check the operands for
the "m" constraint etc.

-Eli




More information about the cfe-commits mailing list