[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