[clang] [clang][Sema] Combine fallout warnings to just one warning (PR #127546)

via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 18 12:44:10 PST 2025


================
@@ -696,47 +665,41 @@ static void CheckFallThroughForBody(Sema &S, const Decl *D, const Stmt *Body,
   if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
       return;
   SourceLocation LBrace = Body->getBeginLoc(), RBrace = Body->getEndLoc();
-  auto EmitDiag = [&](SourceLocation Loc, unsigned DiagID) {
-    if (IsCoroutine)
-      S.Diag(Loc, DiagID) << FSI->CoroutinePromise->getType();
-    else
-      S.Diag(Loc, DiagID);
-  };
----------------
Sirraide wrote:

Actually, we weren’t even doing anything with the promise type were we? Oh well, just get rid of it then (i.e. leave your changes here as-is); I don’t think we care too much about that.

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


More information about the cfe-commits mailing list