[llvm] [ThinLTO] Populate declaration import status except for distributed ThinLTO under a default-off new option (PR #88024)
Mingming Liu via llvm-commits
llvm-commits at lists.llvm.org
Thu May 16 15:43:34 PDT 2024
================
@@ -166,7 +166,11 @@ declare void @variadic_va_start(...)
; GUID-DAG: GUID {{.*}} is linkoncefunc
; DUMP: Module [[M1:.*]] imports from 1 module
-; DUMP-NEXT: 15 functions imported from [[M2:.*]]
-; DUMP-NEXT: 4 vars imported from [[M2]]
+; DUMP-NEXT: 15 function definitions and 0 function declarations imported from [[M2:.*]]
+; DUMP-NEXT: 4 var definitions and 0 var declarations imported from [[M2]]
+
+; The following information are printed from `FunctionImporter::importFunctions`
+; in the postlink pipeline.
+; TODO: Update debug information for postlink pipeline.
----------------
minglotus-6 wrote:
I'm thinking about printing counters for both definitions and declarations around these lines https://github.com/minglotus-6/llvm-project/blob/f72611b40de8c060c3901dab0ee9f8424dd3012b/llvm/lib/Transforms/IPO/FunctionImport.cpp#L1757-L1762
I removed the comments in this test case and added TODO in the source code to make it clearer.
https://github.com/llvm/llvm-project/pull/88024
More information about the llvm-commits
mailing list