[PATCH] D148594: [X86] Transform `(cmp eq/ne trunc(X), C)` -> `(cmp eq/ne X, Zext(C))`

[Phoebe Wang via Phabricator via llvm-commits]

pengfei added a comment.

Should add non-zero tests?



================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:54454
         const TargetLowering &TLI = DAG.getTargetLoweringInfo();
+        ConstantSDNode *C = dyn_cast<ConstantSDNode>(RHS);
         if (DAG.MaskedValueIsZero(LHS.getOperand(0), UpperBits) &&
----------------
cast


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148594