landed as r145530 (llvm) and r145531 (clang)<div>Thanks! <br><br><div class="gmail_quote">On Wed, Nov 30, 2011 at 12:13 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@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 class="gmail_quote"><div><div class="h5">On Wed, Nov 30, 2011 at 11:04 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">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>+cfe-commits (as the patch touches both llvm and clang)<br><br><div class="gmail_quote"><div><div>On Wed, Nov 30, 2011 at 9:33 AM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com" target="_blank">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></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>
</blockquote></div><br></div>