[PATCH] D121088: [AArch64] Concat zip1 and zip2 is a wider zip1

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

@efriedma I wrote this quickly and didn't have a lot of time to look into doing it differently. I got another report of the same thing. What do you think about getting this version in? It seems like a simple improvement, especially from intrinsic code which is unlikely to include undef. And for test like @combine2_v8i16 they already include the concat from the beginning in a separate shuffle. The patch still applies as-is (with updated tests), and I could add a new test for the extra case you mention with a FIXME comment?


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

https://reviews.llvm.org/D121088