[Mlir-commits] [mlir] [TableGen] Avoid repeated hash lookups (NFC) (PR #107429)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Thu Sep 5 10:00:29 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/107429.diff
1 Files Affected:
- (modified) mlir/lib/TableGen/Pattern.cpp (+4-5)
``````````diff
diff --git a/mlir/lib/TableGen/Pattern.cpp b/mlir/lib/TableGen/Pattern.cpp
index afb69e7cc55866..1be0e744ffbc86 100644
--- a/mlir/lib/TableGen/Pattern.cpp
+++ b/mlir/lib/TableGen/Pattern.cpp
@@ -137,11 +137,10 @@ llvm::StringRef DagNode::getSymbol() const { return node->getNameStr(); }
Operator &DagNode::getDialectOp(RecordOperatorMap *mapper) const {
llvm::Record *opDef = cast<llvm::DefInit>(node->getOperator())->getDef();
- auto it = mapper->find(opDef);
- if (it != mapper->end())
- return *it->second;
- return *mapper->try_emplace(opDef, std::make_unique<Operator>(opDef))
- .first->second;
+ auto [it, inserted] = mapper->try_emplace(opDef);
+ if (inserted)
+ it->second = std::make_unique<Operator>(opDef);
+ return *it->second;
}
int DagNode::getNumOps() const {
``````````
</details>
https://github.com/llvm/llvm-project/pull/107429
More information about the Mlir-commits
mailing list