[clang] [TableGen] Avoid repeated map lookups (NFC) (PR #126381)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 8 09:21:26 PST 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/126381
None
>From 20fe55df1856cb8405af76671452229e01a048b5 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 8 Feb 2025 00:55:57 -0800
Subject: [PATCH] [TableGen] Avoid repeated map lookups (NFC)
---
clang/utils/TableGen/ClangOptionDocEmitter.cpp | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/clang/utils/TableGen/ClangOptionDocEmitter.cpp b/clang/utils/TableGen/ClangOptionDocEmitter.cpp
index e08fb11df310023..b6c1aad90b5cb91 100644
--- a/clang/utils/TableGen/ClangOptionDocEmitter.cpp
+++ b/clang/utils/TableGen/ClangOptionDocEmitter.cpp
@@ -109,13 +109,17 @@ Documentation extractDocumentation(const RecordKeeper &Records,
// Pretend no-X and Xno-Y options are aliases of X and XY.
std::string Name = std::string(R->getValueAsString("Name"));
if (Name.size() >= 4) {
- if (Name.substr(0, 3) == "no-" && OptionsByName[Name.substr(3)]) {
- Aliases[OptionsByName[Name.substr(3)]].push_back(R);
- continue;
+ if (Name.substr(0, 3) == "no-") {
+ if (const Record *Opt = OptionsByName[Name.substr(3)]) {
+ Aliases[Opt].push_back(R);
+ continue;
+ }
}
- if (Name.substr(1, 3) == "no-" && OptionsByName[Name[0] + Name.substr(4)]) {
- Aliases[OptionsByName[Name[0] + Name.substr(4)]].push_back(R);
- continue;
+ if (Name.substr(1, 3) == "no-") {
+ if (const Record *Opt = OptionsByName[Name[0] + Name.substr(4)]) {
+ Aliases[Opt].push_back(R);
+ continue;
+ }
}
}
More information about the cfe-commits
mailing list