[PATCH] D71046: Support Intel "l" suffixes for x86_64 R8-R15 registers.

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 16 12:43:59 PST 2019


jyknight added a comment.

Yes, I had expressed a dislike to adding these alises, as there's no pressing need to do so.

X86_64 has been around for 20 years now -- and in all that time, none of the widely-used assemblers have supported these aliases. Adding new aliases now is just adds to confusion and non-portability, which doesn't really help anyone.

Given that the only thing actually using these register names appears to be documentation which is going to be adjusted, that's even more reason not to do it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71046





More information about the llvm-commits mailing list