[PATCH] D65553: Move register namespacing definitions from TargetRegisterInfo to Register and a new MCRegister

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 13:11:08 PDT 2019


dsanders marked an inline comment as done.
dsanders added inline comments.


================
Comment at: llvm/include/llvm/MC/MCRegister.h:19-20
+/// implementation will follow.
+class MCRegister {
+public:
+  // Register numbers can represent physical registers, virtual registers, and
----------------
arsenm wrote:
> The actual MCPhysReg member is missing?
The note on line 17 is referring to this. It's added in D65599


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65553





More information about the llvm-commits mailing list