[PATCH] D93686: [LoopUnroll] Fix a crash

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 4 18:13:03 PST 2021


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

LGTM, thank you.



================
Comment at: llvm/lib/Transforms/Utils/LoopUnroll.cpp:365-366
 
+  // All these values should be taken after peeling did its job due to it
+  // can change them.
+  BasicBlock *Preheader = L->getLoopPreheader();
----------------
[grammar] All these values should be taken only after peeling because they might have changed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93686/new/

https://reviews.llvm.org/D93686



More information about the llvm-commits mailing list