[PATCH] D103077: [DAGCombine] Poison-prove scalarizeExtractedVectorLoad.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a subscriber: uweigand.
jonpa added a comment.

In D103077#2779949 <https://reviews.llvm.org/D103077#2779949>, @fhahn wrote:

> Adjust failing SystemZ test. It would be great if someone familiar with SystemZ could take a look to double-check this makes sense. Perhaps @jonpa?

Yes, this looks like a correct test update to me: the changed immediate operand reflects that now only two shifted bits are inserted into %r1 (instead of all 32).

I wonder what the rationale behind this patch is: if the instruction is poison to begin with, then the program is broken and I don't understand how changing the argument value can change that..?

Adding @uweigand as well just in case...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103077