Skip to content

Commit 47a7644

Browse files
jurahultmsri
authored andcommitted
[LLVM][TableGen] Change X86MnemonicTables to use const RecordKeeper (llvm#109053)
Change X86MnemonicTables to use const RecordKeeper. This is a part of effort to have better const correctness in TableGen backends: https://discourse.llvm.org/t/psa-planned-changes-to-tablegen-getallderiveddefinitions-api-potential-downstream-breakages/81089
1 parent b3103ae commit 47a7644

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

llvm/utils/TableGen/X86MnemonicTables.cpp

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ using namespace llvm;
2222
namespace {
2323

2424
class X86MnemonicTablesEmitter {
25-
CodeGenTarget Target;
25+
const CodeGenTarget Target;
2626

2727
public:
28-
X86MnemonicTablesEmitter(RecordKeeper &R) : Target(R) {}
28+
X86MnemonicTablesEmitter(const RecordKeeper &R) : Target(R) {}
2929

3030
// Output X86 mnemonic tables.
3131
void run(raw_ostream &OS);
@@ -34,15 +34,13 @@ class X86MnemonicTablesEmitter {
3434
void X86MnemonicTablesEmitter::run(raw_ostream &OS) {
3535
emitSourceFileHeader("X86 Mnemonic tables", OS);
3636
OS << "namespace llvm {\nnamespace X86 {\n\n";
37-
Record *AsmWriter = Target.getAsmWriter();
37+
const Record *AsmWriter = Target.getAsmWriter();
3838
unsigned Variant = AsmWriter->getValueAsInt("Variant");
3939

4040
// Hold all instructions grouped by mnemonic
4141
StringMap<SmallVector<const CodeGenInstruction *, 0>> MnemonicToCGInstrMap;
4242

43-
ArrayRef<const CodeGenInstruction *> NumberedInstructions =
44-
Target.getInstructionsByEnumValue();
45-
for (const CodeGenInstruction *I : NumberedInstructions) {
43+
for (const CodeGenInstruction *I : Target.getInstructionsByEnumValue()) {
4644
const Record *Def = I->TheDef;
4745
// Filter non-X86 instructions.
4846
if (!Def->isSubClassOf("X86Inst"))

0 commit comments

Comments
 (0)