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

[Duncan Sands]

Hi,

> What's the best way to find out all places that need fixing up?

I think code inspection is the only way.

> Should I just look into the Expand* functions in all targets and find
> out what needs to use MergeValues?

Yes.  I can give you a hand with this, but not this week.

> Or should I just make this change in
> ExpandIntegerOperand() and let target owners take care of it. What all
> targets are currently using -enable-legalize-types? I guess only X86.

LegalizeTypes is not for any particular target.  Today it is not turned
on by default, but I hope to change that by the end of the week.

Ciao,

Duncan.