[PATCH] D125220: [InstCombine] (rot X, ?) == 0/-1 --> X == 0/-1
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 18 06:40:21 PDT 2022
spatel accepted this revision.
spatel added a comment.
LGTM - if you are planning to enhance more folds to allow undef in vector constants, please test correctness with Alive2. It is true that there are more folds like this, but it is not correct to propagate undef elements in general.
Please wait for @RKSimon for final approval.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125220/new/
https://reviews.llvm.org/D125220
More information about the llvm-commits
mailing list