PiotrZSL accepted this revision. PiotrZSL added a comment. This revision is now accepted and ready to land.
LGTM - Missing release notes entry. - I'm not sure about change in StmtCXX.h, do we need it ? Coudn't we just use children() + getBody() - CHECK-MESSAGES-NOT get me a little bit confused. Maybe we dont need those entrys, just make sure that test fail when non listed warning show up. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147417/new/ https://reviews.llvm.org/D147417 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
