[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