[PATCH] D36052: Update the new PM pipeline to make ICP aware if it is SamplePGO build.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 17:13:29 PDT 2017


danielcdh added a comment.

In https://reviews.llvm.org/D36052#825377, @chandlerc wrote:

> Looks awesome, LGTM.
>
> This does move ICP to pre-cleanup passes for non-ThinLTO builds with sample PGO. I assume you've tested it and this is fine. (It also moves the sample profile loading, but that one seems completely innocuous to me.)


Yes, verified that this does not affect non-ThinLTO performance. commit the patch now.


https://reviews.llvm.org/D36052





More information about the llvm-commits mailing list