[clang] [Clang][Sema]: Diagnose lambda to bool implicit casts (PR #83152)
Vinayak Dev via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 28 09:10:06 PST 2024
================
@@ -16538,6 +16538,21 @@ void Sema::DiagnoseAlwaysNonNullPointer(Expr *E,
}
}
+ // Complain if we are converting a lambda expression to a boolean value
+ if (const auto *MCallExpr = dyn_cast<CXXMemberCallExpr>(E)) {
+ if (const auto *MRecordDecl = MCallExpr->getRecordDecl();
+ MRecordDecl && MRecordDecl->isLambda()) {
+ std::string Str;
+ llvm::raw_string_ostream S(Str);
+
+ E->printPretty(S, nullptr, getPrintingPolicy());
+ Diag(E->getExprLoc(), diag::warn_impcast_pointer_to_bool)
+ << /*LambdaPointerConversionOperatorType*/ 3 << S.str()
+ << MRecordDecl->getSourceRange() << Range << IsEqual;
----------------
vinayakdsci wrote:
Whoops, I accidentally missed that! Thanks for pointing out the mistakes. Since we don't use `S` anyway, can I omit the call to `printPretty()` altogether?
https://github.com/llvm/llvm-project/pull/83152
More information about the cfe-commits
mailing list