================
@@ -971,6 +972,34 @@ StmtResult Sema::ActOnIfStmt(SourceLocation IfLoc,
if (!ConstevalOrNegatedConsteval && !elseStmt)
DiagnoseEmptyStmtBody(RParenLoc, thenStmt, diag::warn_empty_if_body);
+ // Checks for if/else-if condition variable usage in else-if/else scope
+ if (elseStmt) {
+ VarDecl *ConditionVar = nullptr;
+
+ if (auto *CondVar = Cond.get().first) {
+ ConditionVar = dyn_cast<VarDecl>(CondVar);
+ }
+
+ if (ConditionVar) {
+ struct ElseVariableUsageChecker
+ : public RecursiveASTVisitor<ElseVariableUsageChecker> {
----------------
Sirraide wrote:
The CRTP-based version has a horrible binary size overhead and we’ve actively
tried to stop using it as much as possible; see #110040 for more context.
https://github.com/llvm/llvm-project/pull/156436
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
