rnk added a comment.

I was thinking of suggesting to put it under -Wextra, but -Wall is fine too.

I still don't think it should be on by default. I think there are a lot of 
Unix-only projects out there that aren't concerned with MSVC compatibility, and 
a warning about how things with in the Microsoft ABI would be surprising and 
noisy. Maybe we could tack on "... to make this code portable" or something to 
the diagnostic text to make it more clear why the user should be concerned 
about this?


https://reviews.llvm.org/D24289



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

Reply via email to