[llvm-commits] Patch for handling multiple values in ExpandIntegerOperand

[Duncan Sands]

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.