[PATCH] D65009: [LTO] Don't mark regular LTO units with EnableSplitLTOUnit
Peter Collingbourne via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 19 13:26:48 PDT 2019
pcc added a comment.
Sorry, just realized this. If I do
clang++ -c -flto a.cpp # "split"
clang++ -c -flto=thin b.cpp -fwhole-program-vtables # non-split
clang++ a.o b.o
this should fail, right? If I'm not mistaken, this patch series will cause this to succeed. I think we need to change this patch so that it always sets `EnableSplitLTOUnit` to 1 for regular LTO, then you can drop the other patch.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65009/new/
https://reviews.llvm.org/D65009
More information about the cfe-commits
mailing list