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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to