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-&gt;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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to