[PATCH] D157312: [InstCombine] Add transforms for `(or/and (icmp eq/ne X,0),(icmp eq/ne X,Pow2OrZero))`

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 27 11:43:36 PDT 2023


nikic added a comment.

Okay, let's give this a try.



================
Comment at: llvm/test/Transforms/InstCombine/and-or-icmps.ll:3135
   ret i1 %r
 }
----------------
Please also add tests with logical and/or that don't fold (with pow2 on the right).

I thin kwe'd also missing a test where the comparison is not against zero but another constant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157312



More information about the llvm-commits mailing list