[PATCH] D152017: [DebugInfo] Add flag to only emit referenced member functions

Tue Jun 6 16:36:49 PDT 2023

rnk added a comment.

In D152017#4401075 <https://reviews.llvm.org/D152017#4401075>, @dblaikie wrote:

> Yeah, mixed feelings about a default-on flag, so you have to use `-gno-*`. Though one name in that form, might be `-gno-canonical-types`.
> (pedantically C++ calls these things "member functions" (& it seems nice to use consistent naming) rather than "methods", but `-g[no-]undefined-member-functions` would be a bit of a mouthful... )

We don't have great project-wide policy in Clang about whether to use colloquial terminology or standardese in user visible text. Some diagnostics go to great lengths to use the WG21 terminology, and others less so. Our AST node is a `CXXMethodDecl`, so I'd vote for using "methods" over "member functions".

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152017