[PATCH] D52304: [X86][BMI1]: X86DAGToDAGISel: select BEXTR from x & ~(-1 << nbits) pattern
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 1 10:39:56 PDT 2018
lebedev.ri 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);
----------------
craig.topper wrote:
> Shouldn't we be using TargetOpcode::IMPLICIT_DEF not Undef?
Hm, i thought i tried that initially.
But this current version is the only one that did not contain unneeded extra moves.
I will double-check.
Repository:
rL LLVM
https://reviews.llvm.org/D52304
More information about the llvm-commits
mailing list