[PATCH] D90304: [GlobalISel] Introduce optimal variant of regbankselect

[Gabriel Hjort Åkerlund via Phabricator via llvm-commits]

ehjogab marked an inline comment as not done.
ehjogab added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp:1051-1053
+bool isMachineOperandWithVirtualReg(MachineOperand MO) {
+  return MO.isReg() && Register::isVirtualRegister(MO.getReg());
+}
----------------
ehjogab wrote:
> ehjogab wrote:
> > ehjogab wrote:
> > > arsenm wrote:
> > > > arsenm wrote:
> > > > > Can do just .getReg().isVirtual()
> > > > This name is a bit obnoxious. If it's really necessary, I would rather just add this helper directly to MachineOperand
> > > I agree, it's not a good name. I'll see if I can either get rid of this function, or else put it directly in MachineOperand.
> > You mean this works even if MO is not actually a register? I just assumed that you're not allowed to invoke getReg() unless you are certain that MO is really a register.
> Just checked MachineOperand.h, and there's an assert that will fail if getReg() is invoked and isReg() does not hold.
Argh, my mistake; I saw what you meant now...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90304