[PATCH] D105751: WIP/RFC: Introduce GenericMachineInstr classes and derivatives for idiomatic LLVM RTTI.

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/GenericMachineInstrs.h:24
+/// A base class for all GenericMachineInstrs.
+class GenericMachineInstr : public MachineInstr {
+public:
----------------
Does this need to be `public`?


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/GenericMachineInstrs.h:29
+  /// Access the Idx'th operand as a register and return it.
+  Register getReg(unsigned Idx) { return getOperand(Idx).getReg(); }
+
----------------
Can this be `protected`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105751