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: > > > 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 ? 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
