[PATCH] D18617: Call TargetMachine::addEarlyAsPossiblePasses from BackendUtil.

Chandler Carruth via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 7 00:26:04 PDT 2016


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

Minor tweak, but with that LGTM once the LLVM side is in place.


================
Comment at: lib/CodeGen/BackendUtil.cpp:347
@@ +346,3 @@
+        PassManagerBuilder::EP_EarlyAsPossible,
+        [this](const PassManagerBuilder &, legacy::PassManagerBase &PM) {
+          TM->addEarlyAsPossiblePasses(PM);
----------------
Why capture this? Capturing TM would seem narrower. If you don't want a narrow capture, I'd just use [&]...


http://reviews.llvm.org/D18617





More information about the cfe-commits mailing list