[PATCH] D41267: [LTO] Write LTO output for empty RegularLTO.ModsWithSummaries

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 14 23:09:05 PST 2017


mehdi_amini added a comment.

The commit message does not make much sense to me in the context of this C++ API: there is a non-negligible layer between the build and the build system rules and the LTO implementation, i.e. the linker. 
Implementing a contract that makes it convenient for your build system in the implementation of the LTO "engine" behind this API looks like leaking abstractions to me.


https://reviews.llvm.org/D41267





More information about the llvm-commits mailing list