[PATCH] D106932: [AArch64][SVE][InstCombine] Move last{a,b} before binop if one operand is a splat value

Usman Nadeem via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 5 18:39:02 PDT 2021


mnadeem marked an inline comment as done.
mnadeem added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/AArch64/sve-intrinsic-opts-lasta-lastb.ll:348
+
+define i8 @lastb_binop_LHS_RHS_splat_add(<vscale x 16 x i1> %pg, i8 %scalar1, i8 %scalar2) #0 {
+; OPT-LABEL: @lastb_binop_LHS_RHS_splat_add(
----------------
paulwalker-arm wrote:
> I was going to group this test with the above comment but looking at the output there is no transformation.  Is this intended? I would have expected a scalar `add`.
Removed that test as it was not relevant to this patch. The shuffle was pushed after the add by some other transformation.


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

https://reviews.llvm.org/D106932



More information about the llvm-commits mailing list