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

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


jlebar added inline comments.

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

I can change it to [&] if you like that better.


http://reviews.llvm.org/D18617





More information about the cfe-commits mailing list