[PATCH] D94461: [llvm-ifs] Add option to use InterfaceStub library
Puyan Lotfi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 12 16:25:36 PST 2021
plotfi added a comment.
In D94461#2494710 <https://reviews.llvm.org/D94461#2494710>, @compnerd wrote:
> You mean use `armv7---elf` as a triple? While that is possible, keep in mind that it might change the ABI of the generated interfaces. While it is unlikely to matter as the ifso cannot be used at runtime, if the linker attempts to do something like symbolic resolution of ABI dependent specialization (e.g. `strcmp.eabi` vs `strcmp` where the latter is generic), that could matter. However, adding support to explicitly override the encoded triple based on the user's input is reasonable (though might deserve a warning that the triple is being overridden).
I think an override is reasonable.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94461/new/
https://reviews.llvm.org/D94461
More information about the llvm-commits
mailing list