[clang] [PATCH 1/6] [clang] Improve nested name specifier AST representation (PR #147835)
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 21 06:55:18 PDT 2025
================
@@ -307,7 +307,7 @@ class DynTypedNode {
/// For nodes which represent textual entities in the source code,
/// return their SourceRange. For all other nodes, return SourceRange().
- SourceRange getSourceRange() const;
+ SourceRange getSourceRange(bool IncludeQualifier = false) const;
----------------
erichkeane wrote:
I have a mild preference for this being `getSourceRangeIncludingQualifier` (that is, an overload instead of a default arg)? Unless it results in an `if/else` tree elsewhere.
https://github.com/llvm/llvm-project/pull/147835
More information about the cfe-commits
mailing list