[PATCH] D146041: Fix weirdly apologetic diagnostic messages
Aryan Godara via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 14 12:33:35 PDT 2023
AryanGodara added a comment.
In D146041#4193172 <https://reviews.llvm.org/D146041#4193172>, @junaire wrote:
> I'm not certain if it's bad to say 'sorry', IMHO it's fine?
>
> Anyway, you can't just simply delete those words (in diagnostic messages and regression tests), that doesn't work. To verify the patch is good, you can run `ninja check-all` to make sure the testsuite passes.
This was one of the issues for Outreachy '23, and I asked my outreachy mentor to work on this issue.
I'm looking for any help to make the patch more meaningful, and get guidance along the way.
I'll run `ninja check-all`, before pushing the next commit for revision.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146041/new/
https://reviews.llvm.org/D146041
More information about the cfe-commits
mailing list