[PATCH] D66919: Warn about zero-parameter K&R definitions in -Wstrict-prototypes
Aaron Puchert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 30 14:36:58 PDT 2019
aaronpuchert added a comment.
> IIRC, when we rolled out `-Wstrict-prototypes` we explicitly excluded this case since it hit a lot of code without finding bugs.
I'm not a historian, but I believe this was suggested by @rsmith in https://bugs.llvm.org/show_bug.cgi?id=20796#c8, and I think we agreed in IRC yesterday that this should in fact warn. As I wrote in the commit message, the standard isn't terribly clear on what a prototype is, but we already treat it as non-prototype and that's very likely correct.
There are no mentions of too many false positives in the bug report.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66919/new/
https://reviews.llvm.org/D66919
More information about the cfe-commits
mailing list