xbolva00 marked an inline comment as done. xbolva00 added inline comments.
================ Comment at: include/clang/Basic/DiagnosticSemaKinds.td:5625 + "evaluate to 'true'">, + InGroup<IntInBoolContext>; + ---------------- aaron.ballman wrote: > xbolva00 wrote: > > aaron.ballman wrote: > > > xbolva00 wrote: > > > > aaron.ballman wrote: > > > > > This one seems like it should be in the `TautologicalConstantCompare` > > > > > group, no? > > > > Yes, we could move it there. > > > Don't *all* of these effectively boil down to a tautological comparison? > > So you propose put IntInBoolContext into TautologicalConstantCompare ? > That's what I am thinking, but I was curious if you agreed. But that is not enabled by -Wall :/ I would like to have this under -Wall. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63082/new/ https://reviews.llvm.org/D63082 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
