r315614 - [Sema] Diagnose tautological comparison with type's min/max values
Roman Lebedev via cfe-commits
cfe-commits at lists.llvm.org
Sat Nov 11 04:18:59 PST 2017
On Fri, Nov 10, 2017 at 9:14 PM, Hans Wennborg <hans at chromium.org> wrote:
> On Thu, Oct 12, 2017 at 2:33 PM, Roman Lebedev via cfe-commits
> <cfe-commits at lists.llvm.org> wrote:
>> On Fri, Oct 13, 2017 at 12:12 AM, Nico Weber <thakis at chromium.org> wrote:
>>> Huh, I consider clang not warning on this a feature, not a bug. Why are we
>>> trying to match what gcc does here?
>> Because i have been bitten by this more than once.
>> It did result in big amount of wasted time, and this "thread":
>> As with all the clang diagnostics, you can disable them
>> I personally don't have any opinion on comparisons with
> To disable the new warning
Will the https://reviews.llvm.org/D39462 address the reasons for disabling it?
> but keep the ones we used to have enabled,
> we now need to pass -Wno-tautological-constant-compare
> -Wtautological-constant-out-of-range-compare, which is quite a
> mouthful. It would perhaps be better if the new warning could have
> been behind a flag that could be disabled separately.
More information about the cfe-commits