[PATCH] D154364: [LTO] Allow library calls to be internalized in freestanding mode
Joseph Huber via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 3 12:50:06 PDT 2023
jhuber6 added a comment.
In D154364#4469476 <https://reviews.llvm.org/D154364#4469476>, @efriedma wrote:
>> it sets TLII->disableAllFunctions();. I'm guessing that's not sufficient here?
>
> That just disables TargetLibraryInfo, i.e. recognition of non-intrinsic symbols as having some particular meaning. Backends generally still assume that compiler-rt.builtins or libgcc, plus memcpy/memmove/memset, are available.
The functions that I'm interested in internalizing are the math functions. Since the GPU resolves everything statically we should be able to internalize all of this and remove the overhead. I'm not sure if there's a good way to express that. Maybe a secondary list?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154364/new/
https://reviews.llvm.org/D154364
More information about the llvm-commits
mailing list