[PATCH] D20888: [LTO] Add --lto-aa-pipeline

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 15:30:42 PDT 2016


LGTM from the lld side.

Cheers,
Rafael


On 1 June 2016 at 16:51, Davide Italiano <dccitaliano at gmail.com> wrote:
> davide added a comment.
>
> In http://reviews.llvm.org/D20888#446361, @chandlerc wrote:
>
>> Yes, this is roughly how I expected the API to be used. Are you running into problems with it?
>
>
> No, not at all.
>
>> You might want to name it with a newpm prefix much like the other flag, but I'll leave that mostly up to the LLD folks -- I suspect the local conventions are the most important there.
>
>
> Sure, thanks for your review.
>
>
> http://reviews.llvm.org/D20888
>
>
>


More information about the llvm-commits mailing list