[PATCH] D75674: [AArch64][SVE] Implement vector tuple intrinsics

Francesco Petrogalli via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 05:21:55 PDT 2020


fpetrogalli added a comment.

I think that in this patch we are missing tests that check the output code in situations in which the `tuple.get` operates on tuple that are passed to the definition of the function, as in the following example:

  define <vscale x 16 x i8> @foo(<vscale x 32 x i8> %res) {
  ; CHECK-LABEL: ld2b_i8_1:
  ; CHECK: mov z0.d, z1.d
  ; CHECK-NEXT: ret
    %v2 = call <vscale x 16 x i8> @llvm.aarch64.sve.tuple.get.nxv32i8(<vscale x 32 x i8> %res, i32 1)
    ret <vscale x 16 x i8> %v2
  }

@sdesmalen - should we add them? Am I missing something?

Reference comment: https://reviews.llvm.org/D75751#inline-697705

Francesco


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

https://reviews.llvm.org/D75674





More information about the llvm-commits mailing list