================
@@ -1070,6 +1063,24 @@ void USRGenerator::VisitMSGuidDecl(const MSGuidDecl *D) {
   D->NamedDecl::printName(Out);
 }
 
+void USRGenerator::VisitBaseUsingDecl(const BaseUsingDecl *D) {
+  // Add the filename when needed to disambiguate using decls from different
+  // files.
+  if (ShouldGenerateLocation(D) && GenLoc(D, /*IncludeOffset=*/false))
----------------
sam-mccall wrote:

as discussed offline, ShouldGenerateLocation doesn't seem like the right check 
here:
 - we definitely want to include the filename for *.cpp
 - we may want to include the filename for *.h
 - we probably should exclude system headers
 - for using decls inside functions, we need to include the filename if the 
**function** is not externally visible (though it's probably fine to include it 
always)

https://github.com/llvm/llvm-project/pull/68329
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to