[PATCH] D87596: [Coroutines] Reuse storage for local variables with non-overlapping lifetimes

Xun Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 11:57:07 PDT 2020


lxfind added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:881
   if (PTO.Coroutines)
-    MainCGPipeline.addPass(CoroSplitPass());
+    MainCGPipeline.addPass(CoroSplitPass(Level));
 
----------------
I don't quite understand how the pass registration works... Seems that here we are creating the CoroSplitPass with level passed in already. Why do we need to do it again in PassRegistry.def?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87596/new/

https://reviews.llvm.org/D87596



More information about the llvm-commits mailing list