[PATCH] D52708: Add support for new pass manager

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 1 07:45:53 PDT 2018


tejohnson added a comment.

In https://reviews.llvm.org/D52708#1250954, @sebpop wrote:

> Please add a testcase that will exercise the new flag.


The new pass needs to be added to PassRegistry.def, then you can just test using the same test case added with your earlier patch and invoke opt a second time with "-passes=hotcoldsplit" (or whatever name you register it with) - the -passes= will use the new pass manager.



================
Comment at: lib/Passes/PassBuilder.cpp:195
 
+static cl::opt<bool> EnableHotColdSplit("npm-hot-cold-split", cl::init(false),
+    cl::Hidden, cl::desc("Enable hot-cold splitting pass"));
----------------
No need to make a new new pass manager specific option here. Just share the one used by the other pass manager. Which pass manager is used by the compile will determine where it gets enabled.


https://reviews.llvm.org/D52708





More information about the llvm-commits mailing list