[PATCH] D90153: [AArch64][AsmParser] Remove 'x31' alias for 'sp/xzr' register.

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 02:14:59 PDT 2020


SjoerdMeijer added a comment.

If this was allowed:

  ldrb	w0, [sp, x31]

but disallowed with this change, then this is a breaking change that doesn't seem reasonable.

> The reason for wanting to remove the alias 'x31' is because it allows users to write invalid asm that is not accepted by the GNU assembler.

So, why is this not allowed by the gnu assembler?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90153



More information about the llvm-commits mailing list