<div dir="ltr">> <span style="font-family:arial,sans-serif;font-size:13px">Yours and Alex's suggestion</span><div><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">("Yours" meaning </span><font face="arial, sans-serif">C Bergström's... sorry, the pronoun was a dangling pointer :-) )</font></div>
<div><font face="arial, sans-serif"><br></font></div><div><font face="arial, sans-serif"><br></font></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 1 May 2014 12:09, Dario Domizioli <span dir="ltr"><<a href="mailto:dario.domizioli@gmail.com" target="_blank">dario.domizioli@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div class="gmail_extra"><div class="gmail_quote"><div class="">On 1 May 2014 09:14, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On 1 May 2014 06:43, "C. Bergström" <<a href="mailto:cbergstrom@pathscale.com" target="_blank">cbergstrom@pathscale.com</a>> wrote:<br>


> /* ignorable rant<br>
> No - you just listened to a select group of (important) individuals and<br>
> pushed this through<br>
> */<br>
<br>
</div>Isn't it how *all* consensus are reached? :)<br>
<span><font color="#888888"><br></font></span></blockquote><div><br></div></div><div>:-)</div><div>Sorry, I didn't mean to exclude anybody. To be honest, what we care about the most is that there is consensus on the need for the feature, so the community will be OK with having it - and that seems to have been reached.</div>

<div><br></div><div>Yours and Alex's suggestion of "#pragma clang optimize on/off" (i.e. removing the double negative) has also the benefit of sounding similar to MSVC which is what our multi-platform users will see the most. Considering that we have determined that it'll be in the clang namespace we have less concerns over compatibility, so I suppose it would be OK for other people as well.<br>

</div><div>Is removing the double negative (but keeping the on/off based semantics) a good compromise for everybody?<br></div><div><br></div><div>I have now looked at the code for handling pragmas in clang and it is quite easy to extend (I'm working on a basic version of the feature and it's almost done); furthermore I have a little layer of API in Sema to handle the underlying implementation for a generic "range based optnone", so the syntax will only be important for the actual PragmaHandler. I can just change a string and flip the on/off sense and it'll be exactly the same.</div>
<div class="">
<div><br></div><div>Cheers,</div><div>    Dario Domizioli</div><div>    SN Systems - Sony Computer Entertainment Group</div><div><br></div><div><br></div><div><br></div></div></div></div></div>
</blockquote></div><br></div>