tstellar added a comment. The reason I used a minimal check string is because if the warning text changes at all, then the test becomes useful (this is the downside of using -NOT). I wonder if there is another way to test this that might be robust? If not, then I think this change is fine.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122487/new/ https://reviews.llvm.org/D122487 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits