[PATCH] D28627: [PM] Introduce an analysis set used to preserve all analyses over a function's CFG when that CFG is unchanged.

Sean Silva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 14 20:08:27 PST 2017


silvas added a comment.

It would be good to build up some verification machinery here at some point in the future.



================
Comment at: lib/Transforms/InstCombine/InstructionCombining.cpp:3180
   PreservedAnalyses PA;
+  PA.preserveSet<CFGAnalyses>();
   PA.preserve<AAManager>();
----------------
Is this actually true? I've seen InstCombine rewrite a switch. I know we *want* it to be true, but I'm not sure if it is right now.


https://reviews.llvm.org/D28627





More information about the llvm-commits mailing list