[PATCH] D42680: [ThinLTO] Ignore -fthinlto-index= for non-ThinLTO files

Peter Collingbourne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 30 09:16:35 PST 2018


pcc requested changes to this revision.
pcc added a comment.
This revision now requires changes to proceed.

This doesn't seem right to me. In a mixed full/thin LTO link the full LTO module would be compiled during the indexing phase. We don't want to compile it again in the backend as it could lead at best to duplicate symbol errors and at worst to miscompiles of `llvm.type.*` intrinsic calls.


https://reviews.llvm.org/D42680





More information about the cfe-commits mailing list