================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to