[PATCH] D159265: [AArch64] Remove copy instruction between uaddlv and urshr

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 6 11:57:23 PDT 2023


efriedma added a comment.

Do we need to be concerned at all about big-endian here?  (Actually, also for D159267 <https://reviews.llvm.org/D159267>.)  This is basically bitcasting from `<2 x i32>` to `<1 x i64>`.

We might want to consider teaching DAGCombine to optimize this sequence to an ISD::BITCAST instead of pattern-matching it late.  Might unblock other optimizations?  Maybe there aren't really any other optimizations we can do on a uaddlv, though.


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

https://reviews.llvm.org/D159265



More information about the llvm-commits mailing list