[PATCH] D14908: [IndVars] Preserve LCSSA on inner loops
hfinkel@anl.gov via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 26 11:32:24 PST 2015
hfinkel added a subscriber: hfinkel.
hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.
LGTM.
================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:2185
@@ +2184,3 @@
+ // When expanding add recurrences for loop L, SCEVExpander can break LCSSA for
+ // loops inside it (i.e. L's children).
+ if (Changed)
----------------
This does not seem to be documented explicitly anywhere, could you also add some comments in SCEVExpander's header file to this effect as well.
http://reviews.llvm.org/D14908
More information about the llvm-commits
mailing list