[PATCH] D147659: [CodeGen] Add Pre-commit tests for D146988

Thu Apr 6 13:32:56 PDT 2023

igor.kirillov marked an inline comment as done.
igor.kirillov added inline comments.

================
Comment at: llvm/test/CodeGen/AArch64/complex-deinterleaving-multiuses.ll:115
+; CHECK-NEXT:    ext v6.16b, v2.16b, v2.16b, #8
+; CHECK-NEXT:    fcmla v4.4s, v3.4s, v2.4s, #0
+; CHECK-NEXT:    fmul v5.2s, v16.2s, v7.2s
----------------
dmgreen wrote:
> igor.kirillov wrote:
> > Am I correct that we still want to deinterleave in case shufflevectors have external uses?
> I think we want the fastest code :) Which is usually the smallest but can depend on the exact instructions. I worry that it might depend on whether the shuffle will be folded into a ld2 or needs to be emitted anyway. Which may mean it needs to look at the shuffles operands to check whether it looks like they can be ignored or not.
Done - see `multiple_muls_shuffle_external_with_loads`

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147659