<div dir="ltr">+1 to all of this. =] And to your thoughts about how to do the renaming Rafael. Generally, commit away, this looks like strict improvements.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 10, 2014 at 9:36 AM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mar 10, 2014, at 6:38 AM, Tobias Grosser <<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>> wrote:<br>

<br>
> On 03/10/2014 02:35 PM, Rafael Espíndola wrote:<br>
>>> OK. If this is not obviously beneficial, I don't have any strong opinion<br>
>>> here.<br>
>><br>
>> Same here. It is quiet a bit more complicated and requires patching<br>
>> llvm too, but it does avoid the double negation. Patches attached.<br>
><br>
> Wow. I am impressed. Thanks Rafael for going through the trouble of doing those patches. I think I prefer the new ones over what you proposed before.<br>
<br>
</div></div>I agree. This is much more clear than CLANG_IS_PRODUCTION. I recently came across that build setting and had to dig around to see if it was still being used for anything. I like this much better.</blockquote>
</div><br></div>