[PATCH] D43540: [WebAssembly] Enable -Werror=strict-prototypes by default

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 17:53:22 PST 2018


efriedma added a comment.

If someone is compiling C code that doesn't have undefined behavior, it should work; if it doesn't, that's a clear bug.  (As far as I know, there shouldn't be any issues here, but if there are, file a bug and CC me.)

WebAssembly is not the only platform where varargs and non-varargs calls are incompatible.  Some other popular platforms where this is a problem are 64-bit Windows and hard-float AArch32.  So I don't think it makes sense to enable this specifically for one target.


Repository:
  rL LLVM

https://reviews.llvm.org/D43540





More information about the llvm-commits mailing list