[PATCH] D130365: [AArch64][SVE] Allow XZR as offset reg in contiguous loads/stores
Cullen Rhodes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 25 08:58:28 PDT 2022
c-rhodes abandoned this revision.
c-rhodes added a comment.
In D130365#3676362 <https://reviews.llvm.org/D130365#3676362>, @paulwalker-arm wrote:
> Following the link in the summary and looking at LD1H I see `if Rm == '11111' then UNDEFINED;` which suggests whilst the encoding might be valid the result is undefined and so I doubt we want users to fall into such a trap.
ah, well spotted, I didn't see that. I was looking for `Rm != 11111` in the encoding as I've seen for other instructions in the past, not at the pseudo code. It seems XZR shouldn't be accepted by the assembler after all, abandoning this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130365/new/
https://reviews.llvm.org/D130365
More information about the llvm-commits
mailing list