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