[cfe-commits] [LLVMdev] Instrumentation passes and -O0 optimization level

Kostya Serebryany kcc at google.com
Wed Nov 30 14:23:52 PST 2011


landed as r145530 (llvm) and r145531 (clang)
Thanks!

On Wed, Nov 30, 2011 at 12:13 PM, Chandler Carruth <chandlerc at google.com>wrote:

> On Wed, Nov 30, 2011 at 11:04 AM, Kostya Serebryany <kcc at google.com>wrote:
>
>> +cfe-commits (as the patch touches both llvm and clang)
>>
>> On Wed, Nov 30, 2011 at 9:33 AM, Devang Patel <dpatel at apple.com> wrote:
>>
>>>
>>> On Nov 29, 2011, at 11:26 PM, Alexander Potapenko wrote:
>>>
>>> >>
>>> >> Alex,
>>> >> Now, the patch is actually a bit confusing to me.
>>> >> EP_AlwaysEnabled should mean "works with O0 after inliner and with >=
>>> O1
>>> >> somewhere late", but it doesn't look like it works this way
>>> (otherwise, you
>>> >> wouldn't need to call PMBuilder.addExtension twice).
>>> >> ?
>>> > This was actually my question to Devang.
>>> > Any other suggestions for the EP name?
>>>
>>> OK, I withdraw my suggestion :)
>>>
>>
>> Does anyone else have comments to the original patch (attached)?
>> Thanks,
>>
>
> I already commented (and gave a reminder to send to cfe-commits as well
> for patches to both).
>
> it LGTM. =]
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111130/fc6baf27/attachment.html>


More information about the cfe-commits mailing list