[PATCH] D55722: [DAGCombiner] scalarize binop followed by extractelement

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

Ah, right, I missed that.

Now I'm wondering if we maybe introduce the BYTE_MASK too early.  It looks like X86 (and other targets) leave an all-ones constant vector as a BUILD_VECTOR until the very end of instruction selection; maybe we should do the same on SystemZ to allow common optimizations to still recognize the value ...


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

https://reviews.llvm.org/D55722