[llvm-commits] Patch for handling multiple values in ExpandIntegerOperand
Duncan Sands
baldrick at free.fr
Sat Oct 18 09:23:20 PDT 2008
Hi,
- assert(Res.getValueType() == N->getValueType(0) && N->getNumValues() == 1 &&
- "Invalid operand expansion");
+ // The result node should return same or more values as the original node.
+ assert (Res.getNode()->getNumValues() >= N->getNumValues());
I'd rather use ReplaceAllUsesWith and require the number
of values to be the same. This is more work of course,
since it means fixing up several places.
Ciao,
Duncan.
More information about the llvm-commits
mailing list