[PATCH] D98534: [M68k] Introduce DReg bead

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


ricky26 added a comment.

It acts like a default when decoding (in one of the later patches). The DA bead is still needed since it's used for instructions where it can be either an address or data register.

As it stands, I could rename it but it might be a bit misleading. (Existing cases which don't need a specific register kind would all use `AReg`.)

Alternatively, perhaps I could add `AReg` and enforce that `DReg`/`AReg` cannot be combined with `DA` and keep `Reg` separate, solely for use with `DA`. I didn't think of that at the time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98534



More information about the llvm-commits mailing list