[llvm] [OpenMP] Fix -Wunused-variable (PR #195697)

via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 10:52:47 PDT 2026


https://github.com/ColtenOuO created https://github.com/llvm/llvm-project/pull/195697

DidIt is only used inside an assert, so mark the structured binding `[[maybe_unused]]` to avoid a warning in non-asserts builds

>From a4d3a1b6a8b8f0d3696be2cb97e0dfa3dad2718d Mon Sep 17 00:00:00 2001
From: ColtenOuO <jun930436 at gmail.com>
Date: Mon, 4 May 2026 17:50:22 +0000
Subject: [PATCH] [OpenMP] Fix -Wunused-variable

---
 llvm/lib/Frontend/OpenMP/DirectiveNameParser.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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;
 }



More information about the llvm-commits mailing list