[PATCH] D52304: [X86][BMI1]: X86DAGToDAGISel: select BEXTR from x & ~(-1 << nbits) pattern

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86ISelDAGToDAG.cpp:2697
+  // All the other bits are undefined, we do not care about them.
+  SDValue Undef = CurDAG->getUNDEF(NVT);
+  insertDAGNode(*CurDAG, NBits, Undef);
----------------
Shouldn't we be using TargetOpcode::IMPLICIT_DEF not Undef?


Repository:
  rL LLVM

https://reviews.llvm.org/D52304