[PATCH] D84656: [clang] Pass the NamedDecl* instead of the DeclarationName into many diagnostics.
Erich Keane via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 27 07:04:07 PDT 2020
erichkeane accepted this revision.
erichkeane added a comment.
This revision is now accepted and ready to land.
I looked through the code changes, and they all seem quite mechanical. I believe they are all correct.
After reading through the test changes, I believe that this change is strictly an improvement thanks to printing the template arguments. Therefore, I don't believe I know of any way that htis is controversial.
That said, because I'm approving so quickly after submission, please give at least a few hours before committing to give the other reviewers a chance to take a look.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84656/new/
https://reviews.llvm.org/D84656
More information about the cfe-commits
mailing list