[PATCH] D128216: [X86] Add SimplifyMultipleUseDemandedBitsForTargetNode X86ISD::ANDNP handling

[Phoebe Wang via Phabricator via llvm-commits]

pengfei added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:41821
+    if (DemandedBits.isSubsetOf(LHSKnown.One | RHSKnown.One))
+      return LHS;
+    if (DemandedBits.isSubsetOf(RHSKnown.Zero | LHSKnown.Zero))
----------------
Why return `LHS` rather than `~LHS`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128216