[PATCH] D84658: [clang] Overload NamedDecl::printName to provide a user-friendly name for unnamed entities

Mon Jul 27 07:46:24 PDT 2020

erichkeane added inline comments.

================
Comment at: clang/lib/AST/Decl.cpp:2032
+  // additional complexity needed to prevent this is not worthwhile.
+  OS << (Policy.MSVCFormatting ? '`' : '(')
+     << (IsAnonymousStructOrUnion ? "anonymous" : "unnamed") << ' '
----------------
riccibruno wrote:
> erichkeane wrote:
> > This pattern is happening a bunch too....  Does it deserve some function?  Perhaps a function that either takes Policy, or a member of Policy to get the formatting open-tick or close-tick?
> Do you mean a member function of `PrintingPolicy`, something like `getOpenTick` and `getCloseTick`?
Yeah, exactly, something like that. "Delimiter" is perhaps better than 'tick', but I'm not attached to either.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84658/new/

https://reviews.llvm.org/D84658