[PATCH] D116811: [SCEVExpander] Skip creating <u 0 check, which is always false.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 7 08:35:31 PST 2022


nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp:2653
+    return B;
+  return Builder.CreateOr(A, B);
+}
----------------
FWIW CreateOr already folds LHS | 0 -> LHS, so I think it would make sense to also fold 0 | RHS -> RHS there, for the sake of consistency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116811



More information about the llvm-commits mailing list