https://github.com/MacDue commented:
Nit on the title. Could the title be something like: `[Arch64][SVE] Lower
svrev_* to llvm.vector.reverse`? Since nothing in this PR is doing the folding
(I think the description saying this enables the fold is good).
https://github.com/llvm/llvm-project/pul
@@ -1060,7 +1060,7 @@ let SVETargetGuard = "sve,bf16", SMETargetGuard =
"sme,bf16" in {
def SVEXT: SInst<"svext[_{d}]", "dddi", "csilUcUsUiUlhfd",
MergeNone, "aarch64_sve_ext", [VerifyRuntimeMode], [ImmCheck<2,
ImmCheckExtract, 1>]>;
defm SVLASTA : SVEPerm<"
jf-botto wrote:
@sdesmalen-arm fyi
https://github.com/llvm/llvm-project/pull/116422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jorge Botto (jf-botto)
Changes
This PR follows the approach specified in
https://github.com/llvm/llvm-project/issues/110444#issuecomment-2391540986 by
making clang emit `llvm.vector.reverse` instead of `llvm.aarch64.sve.rev`,
meaning ins
https://github.com/jf-botto ready_for_review
https://github.com/llvm/llvm-project/pull/116422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jf-botto wrote:
Sorry @MacDue, just wanted to check a few things before removing draft status.
https://github.com/llvm/llvm-project/pull/116422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
https://github.com/jf-botto updated
https://github.com/llvm/llvm-project/pull/116422
>From fdd0495d57ea7f59ba201b2b570d05d1ed144c99 Mon Sep 17 00:00:00 2001
From: Jorge Botto
Date: Fri, 15 Nov 2024 18:56:54 +
Subject: [PATCH] Making Clang emit llvm.vector.reverse instead of
llvm.aarch64.sv
MacDue wrote:
Is this intentionally left as a draft?
https://github.com/llvm/llvm-project/pull/116422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jf-botto created
https://github.com/llvm/llvm-project/pull/116422
This PR follows the approach specified in
https://github.com/llvm/llvm-project/issues/110444#issuecomment-2391540986 by
making clang emit `llvm.vector.reverse` instead of `llvm.aarch64.sve.rev`,
meaning instc
