bcain added a comment.

In https://reviews.llvm.org/D39462#959822, @phosek wrote:

> FWIW we've already rolled Clang that contains 
> `-Wtautological-constant-compare` to our codebase and we had to set 
> `-Wno-tautological-constant-compare` globally because we were getting bogus 
> warnings in many places, similarly to https://reviews.llvm.org/D41368. If 
> that warning ships as part of Clang 6, it's going to be be a major pain for 
> many users so we really need some solution.


Let's escalate this discussion to llvm-dev, then?  If we don't do anything 
soon, it will ship w/clang 6.


Repository:
  rL LLVM

https://reviews.llvm.org/D39462



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

Reply via email to