[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