================
@@ -64,7 +80,11 @@ class USRGenerator : public ConstDeclVisitor<USRGenerator> {
   ASTContext *Context;
   const LangOptions &LangOpts;
   bool IgnoreResults = false;
-  bool generatedLoc = false;
+  // The flag below ensures that, when a source location needs to be printed,
+  // the file entry is printed at most once during the recursive visit.  The
+  // offset part may be printed multiple times, since sub-decls along the
+  // path may each need a distinct location to disambiguate them.
+  bool GeneratedFileEntry = false;
----------------
steakhal wrote:

It was puzzling for me what `file entry` means in this context. How I 
understand it right now, it means `file path`. So maybe we should use `path` in 
the name?

https://github.com/llvm/llvm-project/pull/205430
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to