[PATCH] D49434: Put "built-in" function definitions in global Used list, for LTO. (fix bug 34169)

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 19 15:28:44 PDT 2018


tejohnson added a comment.

In https://reviews.llvm.org/D49434#1168858, @cmtice wrote:

> The old LTO implementation still internalizes the function definitions, but they do not get Dead Code Eliminated.


Quick clarification based on our discussion earlier - the old LTO implementation internalizes but does not eliminate with regular LTO. The old LTO implementation with ThinLTO is still broken (hence that part of the test is currently disabled).


https://reviews.llvm.org/D49434





More information about the llvm-commits mailing list