[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