[clang] [clang-offload-packager] Avoid repeated hash lookups (NFC) (PR #127302)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 14 21:36:29 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/127302.diff
1 Files Affected:
- (modified) clang/tools/clang-offload-packager/ClangOffloadPackager.cpp (+3-4)
``````````diff
diff --git a/clang/tools/clang-offload-packager/ClangOffloadPackager.cpp b/clang/tools/clang-offload-packager/ClangOffloadPackager.cpp
index c6d5b31ab512c..49cb0d70f492b 100644
--- a/clang/tools/clang-offload-packager/ClangOffloadPackager.cpp
+++ b/clang/tools/clang-offload-packager/ClangOffloadPackager.cpp
@@ -70,10 +70,9 @@ static DenseMap<StringRef, StringRef> getImageArguments(StringRef Image,
DenseMap<StringRef, StringRef> Args;
for (StringRef Arg : llvm::split(Image, ",")) {
auto [Key, Value] = Arg.split("=");
- if (Args.count(Key))
- Args[Key] = Saver.save(Args[Key] + "," + Value);
- else
- Args[Key] = Value;
+ auto [It, Inserted] = Args.try_emplace(Key, Value);
+ if (!Inserted)
+ It->second = Saver.save(It->second + "," + Value);
}
return Args;
``````````
</details>
https://github.com/llvm/llvm-project/pull/127302
More information about the cfe-commits
mailing list