Unique_Usman added a comment. In D148601#4301977 <https://reviews.llvm.org/D148601#4301977>, @aaron.ballman wrote:
> Thank you for working on this! These changes should come with test coverage > (we have a handful of verifier tests in `clang/test/Frontend` -- you can add > a new test file there), but I don't think a release note is required because > this is a fix for internal functionality. The test should cover the simple > case of `-verify=something` as well as a more complex test with > `-verify=something, something_else`. Thank you, will work on this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148601/new/ https://reviews.llvm.org/D148601 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits