[clang] [clang][Sema] Combine fallout warnings to just one warning (PR #127546)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 18 13:09:50 PST 2025
Sirraide wrote:
Also, just a heads-up, #127623 is likely going to be merged soon and introduce some merge conflicts for you in `CheckFallThroughForBody()`. The code that that pr touches is just deleted here, so that’s fine I think, but maybe make sure that none of the diag ids are `0` if the function being analysed is a coroutine.
(We generate an implicit `return` statement at the end of every coroutine, and if the coroutine is `noreturn`, then we try to emit a ‘noreturn function should not return’ warning. The correct behaviour is to just not emit a warning here at all in that case)
https://github.com/llvm/llvm-project/pull/127546
More information about the cfe-commits
mailing list