[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