[clang] [Clang] add ext warning for missing return in 'main' for C89 mode (PR #134617)

[Aaron Ballman via cfe-commits]

================
@@ -16232,7 +16232,9 @@ Decl *Sema::ActOnFinishFunctionBody(Decl *dcl, Stmt *Body,
 
       // If the function implicitly returns zero (like 'main') or is naked,
       // don't complain about missing return statements.
-      if (FD->hasImplicitReturnZero() || FD->hasAttr<NakedAttr>())
+      if ((FD->hasImplicitReturnZero() &&
+           (getLangOpts().CPlusPlus || getLangOpts().C99 || !FD->isMain())) ||
----------------
AaronBallman wrote:

No, I think we should keep the behavior we have, it's that I think the analysis pass should be run and the diagnostic be suppressed from within there.

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