[llvm] [ThinLTO]Record import type (declaration or definition) in GlobalValueSummary::GVFlags (PR #87597)

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 13:35:02 PDT 2024


================
@@ -472,14 +472,20 @@ class GlobalValueSummary {
     /// means the symbol was externally visible.
     unsigned CanAutoHide : 1;
 
+    /// This field is written by the ThinLTO indexing step to postlink
+    /// per-module summary. It indicates whether a global value (function or
----------------
teresajohnson wrote:

"postlink per-module" doesn't make sense - per-module is always pre-thinlink. Just say in the post link combined summary (which we do shard out for distributed ThinLTO, but it is still the "combined index" format).

https://github.com/llvm/llvm-project/pull/87597


More information about the llvm-commits mailing list