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

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 04:04:09 PDT 2020


sdesmalen added a comment.

I don't know what route is normally taken in these situations; this change would indeed break code that relies on this 'feature' even though removing parsing support for w31/x31 is correct. An alternative approach would be to emit a warning diagnostic that says the use of `w31/x31` is deprecated and that support will be removed in a future release. Is that an approach we want to take, or is it fine to just remove support for it?


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