[llvm] [FileCheck] Avoid repeated hash lookups (NFC) (PR #131553)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 16 20:22:34 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-testing-tools
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/131553.diff
1 Files Affected:
- (modified) llvm/lib/FileCheck/FileCheck.cpp (+4-2)
``````````diff
diff --git a/llvm/lib/FileCheck/FileCheck.cpp b/llvm/lib/FileCheck/FileCheck.cpp
index 931a4d3c36f16..072dbefba1f1f 100644
--- a/llvm/lib/FileCheck/FileCheck.cpp
+++ b/llvm/lib/FileCheck/FileCheck.cpp
@@ -1010,8 +1010,10 @@ bool Pattern::parsePattern(StringRef PatternStr, StringRef Prefix,
// Handle substitution of string variables that were defined earlier on
// the same line by emitting a backreference. Expressions do not
// support substituting a numeric variable defined on the same line.
- if (!IsNumBlock && VariableDefs.find(SubstStr) != VariableDefs.end()) {
- unsigned CaptureParenGroup = VariableDefs[SubstStr];
+ decltype(VariableDefs)::iterator It;
+ if (!IsNumBlock &&
+ (It = VariableDefs.find(SubstStr)) != VariableDefs.end()) {
+ unsigned CaptureParenGroup = It->second;
if (CaptureParenGroup < 1 || CaptureParenGroup > 9) {
SM.PrintMessage(SMLoc::getFromPointer(SubstStr.data()),
SourceMgr::DK_Error,
``````````
</details>
https://github.com/llvm/llvm-project/pull/131553
More information about the llvm-commits
mailing list