[PATCH] D62793: [X86] X86DAGToDAGISel::matchBitExtract(): pattern b: truncation awareness

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:8623
+      .zextOrTrunc(VT.getSizeInBits())
+      .isAllOnesValue();
 }
----------------
RKSimon wrote:
> Could you not create an AllOnes equivalent MaskedValueIsZero instead? MaskedValueIsAllOnes?
Err, s/not//?
"Could you create an AllOnes equivalent of MaskedValueIsZero instead? MaskedValueIsAllOnes?"


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62793