[PATCH] D41340: [LTO] Remove temporary files corresponding to stream which LTO skips

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 22 14:32:21 PST 2017


vitalybuka added inline comments.


================
Comment at: llvm/test/ThinLTO/X86/delete-unused-temps.ll:9
+; RUN: llvm-lto2 run  %t.bc -o %t2 -thinlto-distributed-indexes
+; RUN: ls %t2* | count 1
+
----------------
tejohnson wrote:
> I'm not sure I understand what is going on - why are there fewer files after the -thinlto-distributed-indexes invocation than without?
not 100% sure without debugging, but I think it's because LTO::getMaxTasks() in both cases returns 2, so LTO client expects 2 streams. But with -thinlto-distributed-indexes nothing is written into the last stream as module is empty, and nothing deletes corresponding file from a previous runs. 


https://reviews.llvm.org/D41340





More information about the llvm-commits mailing list