[PATCH] D41297: [ThinLTO] Implement summary visualizer

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 added inline comments.


================
Comment at: lib/Transforms/IPO/LowerTypeTests.cpp:1532
 bool LowerTypeTestsModule::runForTesting(Module &M) {
-  ModuleSummaryIndex Summary;
+  ModuleSummaryIndex Summary(true);
 
----------------
tejohnson wrote:
> Why is this one true? It doesn't seem like we should be marking as InAnalysis. 
Yep, looks like a bug


https://reviews.llvm.org/D41297