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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 16:34:51 PST 2017


davide added inline comments.


================
Comment at: lib/Transforms/Scalar/GuardWidening.cpp:662-663
   bool Changed = GuardWideningImpl(DT, PDT, LI).run();
-  return Changed ? PreservedAnalyses::none() : PreservedAnalyses::all();
+  return Changed ? PreservedAnalyses::allOnFunctionCFG()
+                 : PreservedAnalyses::all();
 }
----------------
hmm, this doesn't seem to have a FIXME but it's supposed to preserve the CFG analyses, so good that you caught it (here and elsewhere).


https://reviews.llvm.org/D28627





More information about the llvm-commits mailing list