[PATCH] D33846: [IRCE] Canonicalize pre/post loops after the blocks are added into parent loop

[Anna Thomas via Phabricator via llvm-commits]

anna added inline comments.


================
Comment at: lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp:1380
   if (!PreLoop.Blocks.empty()) {
-    auto *L = createClonedLoopStructure(
+    PreL = createClonedLoopStructure(
         &OriginalLoop, OriginalLoop.getParentLoop(), PreLoop.Map);
----------------
As a later follow up change, I plan to:
1. rename `PreLoop`/ `PostLoop` and function `createClonedLoopStructure`.
2. move `DT.recalculate` just before calling `canonicalizeLoop`, so that all data structure updates are at one place.


https://reviews.llvm.org/D33846