[PATCH] D98536: [M68k] Convert register Aliases to AltNames

Ricky Taylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 16 13:25:40 PDT 2021


ricky26 added a comment.

In D98536#2627583 <https://reviews.llvm.org/D98536#2627583>, @myhsu wrote:

> Are you trying to distinguish the sub-register use cases from alternative naming (e.g. a5 & bp)?

Kind of. I was running into a number of cases where I had to handle A7/SP separately (for example). I was worried this was eventually going to lead to significant code duplication and/or errors. `AltNames` seemed to make the most sense. SP/etc are really only used to make assembly more readable. (For that reason, I actually made SP the default rendering of A7.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98536



More information about the llvm-commits mailing list