[PATCH] D148601: [Clang] Handle Error message to output proper Prefix
Timm Bäder via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 26 22:20:19 PDT 2023
tbaeder accepted this revision.
tbaeder added a comment.
This revision is now accepted and ready to land.
When I run with `-verify=foo,bar`I only get the output for `bar-no-diagnostics`. Adding `bar-no-diagnostics`, I get no output at all and the test succeeds.
I didn't know how passing multiple prefixes works in practice, seems like it matches any of the given ones. So the new output is still better than the old one.
LGTM but I'll wait a while for someone else to chime in.
In the meantime, please state a name and email address to use for the git commit, in case you don't have commit access.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148601/new/
https://reviews.llvm.org/D148601
More information about the cfe-commits
mailing list