[PATCH] D118562: [AArch64] Allow copying of SVE registers in Streaming SVE

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 1 01:44:27 PST 2022


c-rhodes added a comment.

In D118562#3284538 <https://reviews.llvm.org/D118562#3284538>, @CarolineConcatto wrote:

> Just to check, this change depends on https://reviews.llvm.org/D118561. Correct?
> That is the reason your tests are failing for now. 
> LLVM ERROR: Don't know how to legalize this scalable vector type

Yeah that's right, I'll add it as a parent revision.

> I think you need to use git-clang-format.

I'll run it before committing if accepted.



================
Comment at: llvm/test/CodeGen/AArch64/sve-intrinsics-create-tuple.ll:7
 
-define <vscale x 16 x i8> @test_svcreate2_s8_vec0(i1 %p, <vscale x 16 x i8> %z0, <vscale x 16 x i8> %z1) local_unnamed_addr #0 {
 ; CHECK-LABEL: test_svcreate2_s8_vec0:
----------------
CarolineConcatto wrote:
> I guess this is just to leave a cleaner test. Correct?
> I guess this is just to leave a cleaner test. Correct?

Yeah I don't think `local_unnamed_addr` is needed for these tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118562



More information about the llvm-commits mailing list