[PATCH] D154193: [X86]Remove TEST in AND32ri+TEST16rr in peephole-opt

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 4 01:40:28 PDT 2023


skan added inline comments.


================
Comment at: llvm/test/CodeGen/X86/peephole-test-after-add.mir:57
 
-  define i16 @erase_test16_bigimm(i16 %0, i32 %1, ptr nocapture %2) {
+  define i16 @erase_test16_bigimm(i16 %0, i32 %1, ptr nocapture %2, ptr %store) {
   entry:
----------------
XinWang10 wrote:
> skan wrote:
> > Why change this?
> The original intention is to test andri whose imm is not fit in 16bit, but the mir convert the 123456 to 57920, it offend the intention.
Then remove %2, or reuse %2


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154193



More information about the llvm-commits mailing list