[llvm] [TableGen] Avoid repeated hash lookups (NFC) (PR #123562)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 19 23:57:42 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-tablegen
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/123562.diff
1 Files Affected:
- (modified) llvm/utils/TableGen/Common/CodeGenSchedule.cpp (+5-9)
``````````diff
diff --git a/llvm/utils/TableGen/Common/CodeGenSchedule.cpp b/llvm/utils/TableGen/Common/CodeGenSchedule.cpp
index 7ae18d3ccc7bc5..8eaba05e65ce92 100644
--- a/llvm/utils/TableGen/Common/CodeGenSchedule.cpp
+++ b/llvm/utils/TableGen/Common/CodeGenSchedule.cpp
@@ -311,15 +311,13 @@ static void processSTIPredicate(STIPredicateFunction &Fn,
ConstRecVec Classes = Def->getValueAsListOfDefs("Classes");
for (const Record *EC : Classes) {
const Record *Pred = EC->getValueAsDef("Predicate");
- if (!Predicate2Index.contains(Pred))
- Predicate2Index[Pred] = NumUniquePredicates++;
+ if (Predicate2Index.try_emplace(Pred, NumUniquePredicates).second)
+ ++NumUniquePredicates;
ConstRecVec Opcodes = EC->getValueAsListOfDefs("Opcodes");
for (const Record *Opcode : Opcodes) {
- if (!Opcode2Index.contains(Opcode)) {
- Opcode2Index[Opcode] = OpcodeMappings.size();
+ if (Opcode2Index.try_emplace(Opcode, OpcodeMappings.size()).second)
OpcodeMappings.emplace_back(Opcode, OpcodeInfo());
- }
}
}
}
@@ -452,11 +450,9 @@ void CodeGenSchedModels::checkMCInstPredicates() const {
for (const Record *TIIPred :
Records.getAllDerivedDefinitions("TIIPredicate")) {
StringRef Name = TIIPred->getValueAsString("FunctionName");
- StringMap<const Record *>::const_iterator It = TIIPredicates.find(Name);
- if (It == TIIPredicates.end()) {
- TIIPredicates[Name] = TIIPred;
+ auto [It, Inserted] = TIIPredicates.try_emplace(Name, TIIPred);
+ if (Inserted)
continue;
- }
PrintError(TIIPred->getLoc(),
"TIIPredicate " + Name + " is multiply defined.");
``````````
</details>
https://github.com/llvm/llvm-project/pull/123562
More information about the llvm-commits
mailing list