[clang] [NFC] Fix warning: variable II set but not used (PR #133307)

via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 27 12:58:18 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: AdityaK (hiraditya)

<details>
<summary>Changes</summary>

```
llvm-project/clang/lib/Sema/SemaExprCXX.cpp:63:29: warning: variable 'II' set but not used [-Wunused-but-set-variable]
   63 |   if (const IdentifierInfo *II = NNS->getAsIdentifier())
      |                             ^
```

---
Full diff: https://github.com/llvm/llvm-project/pull/133307.diff


1 Files Affected:

- (modified) clang/lib/Sema/SemaExprCXX.cpp (+2-1) 


``````````diff
diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp
index b2310628adc64..a96c024694c59 100644
--- a/clang/lib/Sema/SemaExprCXX.cpp
+++ b/clang/lib/Sema/SemaExprCXX.cpp
@@ -60,9 +60,10 @@ ParsedType Sema::getInheritingConstructorName(CXXScopeSpec &SS,
                                               SourceLocation NameLoc,
                                               const IdentifierInfo &Name) {
   NestedNameSpecifier *NNS = SS.getScopeRep();
+#ifndef NDEBUG
   if (const IdentifierInfo *II = NNS->getAsIdentifier())
     assert(II == &Name && "not a constructor name");
-
+#endif
   QualType Type(NNS->translateToType(Context), 0);
   // This reference to the type is located entirely at the location of the
   // final identifier in the qualified-id.

``````````

</details>


https://github.com/llvm/llvm-project/pull/133307


More information about the cfe-commits mailing list