erichkeane added inline comments.
================ Comment at: clang/test/Sema/prototype-redecls.c:32 // clear that the previous declaration was a builtin. -float rintf() { // expected-error {{conflicting types for 'rintf'}} \ +float rintf() { // expected-warning {{incompatible redeclaration of library function 'rintf'}} \ expected-note {{'rintf' is a builtin with type 'float (float)'}} ---------------- Hmm... this is a definition of a builtin with a completely incompatible prototype. Do we REALLY want this to not be an error? I guess I could see it being OK with declarations, but it is odd with a definition that is incompatible. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131499/new/ https://reviews.llvm.org/D131499 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits