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

[Jay Foad via Phabricator via llvm-commits]

foad 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};
----------------
barannikov88 wrote:
> 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.
> 
Yes, you are correct, it points to explicit ops. I just meant that that does not necessarily mean the variable has to be called "OpInfo".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142219