[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