[PATCH] D142219: [MC] Store operand info immediately after the TargetInsts table. NFC.

[Sergei Barannikov via Phabricator via llvm-commits]

barannikov88 added inline comments.


================
Comment at: llvm/include/llvm/MC/MCInstrDesc.h:577
   ArrayRef<MCPhysReg> implicit_defs() const {
     auto ImplicitOps = reinterpret_cast<const MCPhysReg *>(this + Opcode + 1);
     return {ImplicitOps + ImplicitOffset + NumImplicitUses, NumImplicitDefs};
----------------
foad wrote:
> barannikov88 wrote:
> > ImplicitOps -> OpInfo (this is where it points now)
> > 
> > 
> You're right that it points to the start of the OpInfo, but I'm not sure that's important. It's also the "origin" from which implicit op list offsets are measured. So maybe `ImplicitOpOrigin`?
I mean that it now points to explicit ops. Implicit ops follow explicit ops (offseted by `ImplicitOffset`). 
Please correct me if I'm wrong.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142219