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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 10:24:17 PST 2022


efriedma added a comment.

This combine seems a little fragile; I'd prefer a solution where generate the zip1 earlier.  I'm concerned some combine will hide the specific concat(zip1, zip2) pattern this patch checks for.

But not sure hard that would be; the current patch is clearly an improvement, in any case.


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

https://reviews.llvm.org/D121088



More information about the llvm-commits mailing list