[PATCH] D52836: [LTO] Account for overriding lib calls via the alias attribute

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.
Herald added a subscriber: dang.

I'll respond more on the bug (especially since I can't reproduce the actual link time error), but I think the right way to fix these types of issues is to compile with one of -ffreestanding, -fno-builtin, or -fno-builtin-log (or whatever is being overridden), so that clang doesn't translate the call to an intrinsic. Clang translates to the intrinsic assuming it is calling the builtin, which means it can assume specific semantics for the call, and there is nothing stopping the compiler from translating that intrinsic call to e.g. an inline expansion of the function (that currently happens e.g. for memcpy and some other builtins).


https://reviews.llvm.org/D52836