[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