+cfe-commits (as the patch touches both llvm and clang)<br><br><div class="gmail_quote">On Wed, Nov 30, 2011 at 9:33 AM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com">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 class="HOEnZb"><div class="h5"><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>Does anyone else have comments to the original patch (attached)?</div><div>Thanks, </div><div><br></div><div>--kcc </div><div><br></div><div>
<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
-<br>
Devang<br>
<br>
</blockquote></div><br>