[PATCH] D31281: [LoopUnroll] Remap references in peeled iteration

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 23 12:51:58 PDT 2017


mkuper requested changes to this revision.
mkuper added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Transforms/Utils/LoopUnrollPeel.cpp:419
+
+    // Rewrite the cloned instruction operands to use the values created when
+    // the clone is created.
----------------
Something here is wrong. There's already a call to remapInstructionsInBlocks in line 443 (original) that does exactly this, so everything ought to be updated correctly. If it isn't, then something's going wrong between here and there, and I'm not sure what it is.


https://reviews.llvm.org/D31281





More information about the llvm-commits mailing list