[PATCH] D101190: [LoopIdiom] Ignore memory accesses to constant memory

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 23 11:40:19 PDT 2021


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:881
+        } else {
+          return true;
+        }
----------------
xbolva00 wrote:
> just return true; (no else)
then the newly introduced if condition is useless if we always return true


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101190



More information about the llvm-commits mailing list