[llvm] [OpenMP] Fix -Wunused-variable (PR #195697)
via llvm-commits
llvm-commits at lists.llvm.org
Mon May 4 10:53:50 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-openmp
Author: Colten (ColtenOuO)
<details>
<summary>Changes</summary>
DidIt is only used inside an assert, so mark the structured binding `[[maybe_unused]]` to avoid a warning in non-asserts builds
---
Full diff: https://github.com/llvm/llvm-project/pull/195697.diff
1 Files Affected:
- (modified) llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp (+2-1)
``````````diff
diff --git a/llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp b/llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp
index 62e24825111f6..446a8609e36de 100644
--- a/llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp
+++ b/llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp
@@ -61,7 +61,8 @@ DirectiveNameParser::insertTransition(State *From, StringRef Tok) {
if (State *Next = From->next(Tok))
return Next;
- auto [Where, DidIt] = From->Transition->try_emplace(Tok, State());
+ [[maybe_unused]] auto [Where, DidIt] =
+ From->Transition->try_emplace(Tok, State());
assert(DidIt && "Map insertion failed");
return &Where->second;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/195697
More information about the llvm-commits
mailing list