[clang] Surface error for plain return statement in coroutine earlier (PR #100985)
Ilya Biryukov via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 1 02:34:50 PDT 2024
================
@@ -694,6 +706,10 @@ bool Sema::ActOnCoroutineBodyStart(Scope *SC, SourceLocation KWLoc,
auto *ScopeInfo = getCurFunction();
assert(ScopeInfo->CoroutinePromise);
+ if (ScopeInfo->FirstCoroutineStmtLoc == KWLoc) {
----------------
ilya-biryukov wrote:
NIT: remove braces from simple statements, see [LLVM Style Guide](https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements)
https://github.com/llvm/llvm-project/pull/100985
More information about the cfe-commits
mailing list