[PATCH] D153791: [MachineInst] Bump NumOperands back up to 24bits

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 10:46:15 PDT 2023


xbolva00 added a comment.

Looks fine



================
Comment at: llvm/lib/CodeGen/MachineInstr.cpp:195
 void MachineInstr::addOperand(MachineFunction &MF, const MachineOperand &Op) {
-  assert(NumOperands < USHRT_MAX && "Cannot add more operands.");
+  assert(NumOperands < ((1u<<LLVM_MID_NUMOPERANDS_BITS)-1) && "Cannot add more operands.");
   assert(MCID && "Cannot add operands before providing an instr descriptor");
----------------
formatting


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153791



More information about the llvm-commits mailing list