[llvm] [ThinLTO] Add HasLocal flag to GlobalValueSummaryInfo (PR #164647)
    Teresa Johnson via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Oct 22 10:39:02 PDT 2025
    
    
  
================
@@ -183,6 +185,22 @@ struct alignas(8) GlobalValueSummaryInfo {
   /// compiling without sufficient distinguishing path, or (theoretically) hash
   /// collisions. Each summary is from a different module.
   GlobalValueSummaryList SummaryList;
+
+  /// True if the SummaryList contains at least one summary with local linkage.
+  /// In most cases there should be only one, unless translation units with
+  /// same-named locals were compiled without distinguishing path. And generally
+  /// there should not be a mix of local and non-local summaries, because the
+  /// GUID for a local is computed with the path prepended and a ':' delimiter.
----------------
teresajohnson wrote:
Thanks, forgot about that change. I'll update the comment and PR description accordingly
https://github.com/llvm/llvm-project/pull/164647
    
    
More information about the llvm-commits
mailing list