<div class="gmail_quote">On Wed, Nov 30, 2011 at 11:04 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":7gv">+cfe-commits (as the patch touches both llvm and clang)<br><br><div class="gmail_quote"><div><div class="h5">On Wed, Nov 30, 2011 at 9:33 AM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com" target="_blank" class="cremed">dpatel@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><div><br>
On Nov 29, 2011, at 11:26 PM, Alexander Potapenko wrote:<br>
<br>
>><br>
>> Alex,<br>
>> Now, the patch is actually a bit confusing to me.<br>
>> EP_AlwaysEnabled should mean "works with O0 after inliner and with >= O1<br>
>> somewhere late", but it doesn't look like it works this way (otherwise, you<br>
>> wouldn't need to call PMBuilder.addExtension twice).<br>
>> ?<br>
> This was actually my question to Devang.<br>
> Any other suggestions for the EP name?<br>
<br>
</div></div>OK, I withdraw my suggestion :)<br></blockquote><div><br></div></div></div><div>Does anyone else have comments to the original patch (attached)?</div><div>Thanks, </div></div></div></blockquote><div><br></div>
<div>I already commented (and gave a reminder to send to cfe-commits as well for patches to both).</div><div><br></div><div>it LGTM. =] </div></div><br>