[PATCH] D99412: [AArch64][SVEIntrinsicOpts] Optimize tbl+dup into dup+extractelement

JunMa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 04:57:54 PDT 2021


junparser added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/sve-tbl-dupx.ll:12
+; CHECK-LABEL: @dup_ext_i8(
+; CHECK-NEXT:    [[TMP:%.*]] = call <vscale x 16 x i8> @llvm.aarch64.sve.dup.x.nxv16i8(i8 1)
+; CHECK-NEXT:    [[TMP1:%.*]] = extractelement <vscale x 16 x i8> [[DATA:%.*]], i8 1
----------------
paulwalker-arm wrote:
> This shows what I meant by my previous comment in that all the tests check for this stray instruction and so it seem preferable to me for SVEIntrinsicOpts to remove it when it's knowingly no longer used.
done, thanks for remind.


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

https://reviews.llvm.org/D99412



More information about the llvm-commits mailing list