[PATCH] D20783: [PM] Port IndVarSimplify to the new pass manager

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 4 23:56:25 PDT 2016


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:2238
@@ +2237,3 @@
+    return PreservedAnalyses::all();
+
+  return getLoopPassPreservedAnalyses();
----------------
davide wrote:
> The old PM 'preserves the CFG'. We can't do that yet in the new PM. Can you please add a FIXME here so we won't forget?
Yea, good to add a FIXME.


http://reviews.llvm.org/D20783





More information about the llvm-commits mailing list