[PATCH] D77925: Revert "[TLI] Per-function fveclib for math library used for vectorization"
Mehdi AMINI via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 20 16:17:22 PDT 2020
mehdi_amini added a comment.
Overall that would likely work for XLA. Something I'd like to mention though in response to:
> The veclib type is also tied to the accepted values for -fveclib, which is a list of supported lib,
`-fveclib` is a Clang thing, it shouldn't limit what LLVM does. Of course LLVM needs to support Clang, but does not have to inherit the limitation of map 1:1 to Clang UI.
In particular as a library, it isn't clear why we would make the choice to write LLVM VecLib support this way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77925/new/
https://reviews.llvm.org/D77925
More information about the cfe-commits
mailing list