[PATCH] D61787: [GlobalISel Legalizer] Improve artifact combiner

[Amara Emerson via Phabricator via llvm-commits]

aemerson added a comment.

I'm trying to figure out whats' going on with this discussion. One thing I didn't quite see a response to (apologies if you answered it and I missed it) is that why is this the case:

> But in LegalizerInfo it says that G_MERGE is legal for {s128, s32} despite the fact that we cannot select it. Why? Because that is how current Artifact combiner works.

I understand that doing this may allow more artifacts to be combined away, but it shouldn't change the legality of anything coming out of the legalizer, if properly specified, right?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61787/new/

https://reviews.llvm.org/D61787