c-rhodes added inline comments.
================
Comment at: llvm/test/CodeGen/AArch64/sve-intrinsics-create-tuple.ll:100
+
+define <vscale x 8 x bfloat> @test_svcreate2_bf16_vec0(i1 %p, <vscale x 8 x
bfloat> %z0, <vscale x 8 x bfloat> %z1) local_unnamed_addr #0 {
+; CHECK-LABEL: test_svcreate2_bf16_vec0:
----------------
fpetrogalli wrote:
> nit: remove `local_unnamed_addr ` from all tests you have added.
I copied these from the existing tests, not sure why we used
`local_unnamed_addr` in the first place. I'll create a patch to remove it
================
Comment at: llvm/test/CodeGen/AArch64/sve-intrinsics-create-tuple.ll:109
+L2:
+ %extract = tail call <vscale x 8 x bfloat>
@llvm.aarch64.sve.tuple.get.nxv8bf16.nxv16bf16(<vscale x 16 x bfloat> %tuple,
i32 0)
+ ret <vscale x 8 x bfloat> %extract
----------------
fpetrogalli wrote:
> out of curiosity.... why not test the create intrinsic directly?
to test we can copy tuple types between basic blocks
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82665/new/
https://reviews.llvm.org/D82665
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
