efriedma added a comment.

If the declaration we're redeclaring is a builtin, should the diagnostic be in 
the "-Wincompatible-library-redeclaration" warning group?  With this patch, we 
treat redefinitions of builtins without a prototype differently from 
redefinitions with a prototype, for example:

  void acos() {} // error
  void acos(void) {} // warning

Just ran into some code in Android which is using the first form.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123627/new/

https://reviews.llvm.org/D123627

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

Reply via email to