minglotus-6 wrote:

https://github.com/llvm/llvm-project/pull/87600 is a functional change and the 
diffbase of this patch, and 
`llvm/test/ThinLTO/X86/import_callee_declaration.ll` should be a test case for 
both patches. 

In the [diffbase](https://github.com/llvm/llvm-project/pull/87600), bitcode 
writer takes maps as additional parameters to populate import status, and it's 
not straightforward to construct regression tests there without this patch.  I 
wonder if I shall introduce `cl::list<std::string>` in llvm-lto/llvm-lto2 (as a 
repeated arg) to specify `filename:GUID` to test the diffbase alone. 

https://github.com/llvm/llvm-project/pull/88024
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to