[PATCH] D73058: [LoopRotate] add ability to repeat loop rotation until non-deoptimizing exit is found

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 21 09:38:48 PST 2020


fedor.sergeev marked an inline comment as done.
fedor.sergeev added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopRotationUtils.cpp:286
+    if (!OrigLatch)
       return false;
+
----------------
skatkov wrote:
> if it is a second iteration you should return true here and in other places.
thanks for the catch!
Perhaps, I should add a test verifying that...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73058





More information about the llvm-commits mailing list