[PATCH] D36040: Refine the PGOOpt and SamplePGOSupport handling.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 17:47:46 PDT 2017


danielcdh added a comment.

In https://reviews.llvm.org/D36040#825041, @chandlerc wrote:

> I think this adds way too much complexity to these predicates. I think we need a much better approach here... Have you looked at alternatives? Is the technique being used to add the descriminators pass just the wrong technique given the interaction with ThinLTO


This was how we handle AddDiscriminator pass in the legacy PM. I have not come up with a better alternative, could you help suggest? Thanks


https://reviews.llvm.org/D36040





More information about the llvm-commits mailing list