[llvm] [TableGen] Avoid repeated hash lookups (NFC) (PR #126405)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 8 21:29:26 PST 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/126405
None
>From 2fbbc8fbef20500d183b77e9d077fae352bccae1 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 8 Feb 2025 11:41:15 -0800
Subject: [PATCH] [TableGen] Avoid repeated hash lookups (NFC)
---
llvm/utils/TableGen/PseudoLoweringEmitter.cpp | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/llvm/utils/TableGen/PseudoLoweringEmitter.cpp b/llvm/utils/TableGen/PseudoLoweringEmitter.cpp
index bcbc6ea20751fe7..6d39d1496c09e82 100644
--- a/llvm/utils/TableGen/PseudoLoweringEmitter.cpp
+++ b/llvm/utils/TableGen/PseudoLoweringEmitter.cpp
@@ -79,8 +79,9 @@ unsigned PseudoLoweringEmitter::addDagOperandMapping(
// "zero_reg" definition.
if (DI->getDef()->isSubClassOf("Register") ||
DI->getDef()->getName() == "zero_reg") {
- OperandMap[BaseIdx + i].Kind = OpData::Reg;
- OperandMap[BaseIdx + i].Data.Reg = DI->getDef();
+ auto &Entry = OperandMap[BaseIdx + i];
+ Entry.Kind = OpData::Reg;
+ Entry.Data.Reg = DI->getDef();
++OpsAdded;
continue;
}
@@ -105,12 +106,14 @@ unsigned PseudoLoweringEmitter::addDagOperandMapping(
OperandMap[BaseIdx + i + I].Kind = OpData::Operand;
OpsAdded += Insn.Operands[i].MINumOperands;
} else if (const IntInit *II = dyn_cast<IntInit>(Dag->getArg(i))) {
- OperandMap[BaseIdx + i].Kind = OpData::Imm;
- OperandMap[BaseIdx + i].Data.Imm = II->getValue();
+ auto &Entry = OperandMap[BaseIdx + i];
+ Entry.Kind = OpData::Imm;
+ Entry.Data.Imm = II->getValue();
++OpsAdded;
} else if (const auto *BI = dyn_cast<BitsInit>(Dag->getArg(i))) {
- OperandMap[BaseIdx + i].Kind = OpData::Imm;
- OperandMap[BaseIdx + i].Data.Imm = *BI->convertInitializerToInt();
+ auto &Entry = OperandMap[BaseIdx + i];
+ Entry.Kind = OpData::Imm;
+ Entry.Data.Imm = *BI->convertInitializerToInt();
++OpsAdded;
} else if (const DagInit *SubDag = dyn_cast<DagInit>(Dag->getArg(i))) {
// Just add the operands recursively. This is almost certainly
More information about the llvm-commits
mailing list