aeubanks added a comment. The summary of this change should be "add warning..." not "fix warning..."
Could you add more detail into the commit description? About newly introduced diagnostic groups, etc. Also, I thought I saw tests in a previous revision but now they seem to be gone? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79895/new/ https://reviews.llvm.org/D79895 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits