[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