[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