[llvm] [MC][TableGen] Expand Opcode field of MCInstrDesc (PR #179652)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 5 08:59:24 PST 2026


================
@@ -474,7 +474,7 @@ void MapTableEmitter::emitTablesWithFunc(raw_ostream &OS) {
   const ListInit *ColFields = InstrMapDesc.getColFields();
   ArrayRef<const ListInit *> ValueCols = InstrMapDesc.getValueCols();
   OS << "// " << InstrMapDesc.getName() << "\nLLVM_READONLY\n";
-  OS << "int " << InstrMapDesc.getName() << "(uint16_t Opcode";
+  OS << "long " << InstrMapDesc.getName() << "(uint32_t Opcode";
----------------
topperc wrote:

We should not be using `long` for anything in llvm. If you need a 64-bit type you should use int64_t. The size of `long` is 32 bits on 32-bit targets and 64-bit Windows.

https://github.com/llvm/llvm-project/pull/179652


More information about the llvm-commits mailing list