[PATCH] D18614: Add TargetMachine::addEarlyAsPossiblePasses.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 09:02:48 PDT 2016


jlebar added a comment.

In http://reviews.llvm.org/D18614#394117, @chandlerc wrote:

> I feel like it would be useful to implement this at least for one target (I'm assuming NVPTX) to show the use case before landing it...


Yes, see http://reviews.llvm.org/D18616.

(The problem, it seems to me, is that we ask people to aggressively split up patches but then don't have a good way to give reviewers the full context.  I can manually go in to Phabricator and set the depends-on metadata, but when I'm posting a dozen small patches, the relative time cost of doing that is huge.  It needs to Happen Automatically.)


http://reviews.llvm.org/D18614





More information about the llvm-commits mailing list