[PATCH] D77484: [Vector] Pass VectLib to LTO backend so TLI build correct vector function list

Hongtao Yu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 5 10:40:26 PDT 2020


hoyFB added a comment.

In D77484#1962445 <https://reviews.llvm.org/D77484#1962445>, @tejohnson wrote:

> We're trying to move towards encoding all of this in the IR. And in fact, I recently implemented a series of patches to make the TLI to be built per-function, and along with some patches from @gchatelet to encode -fno-builtin* as function attributes, we now handle that part of the TLII with IR. See D67923 <https://reviews.llvm.org/D67923> which is the last patch in the series. We should encode the vectlib as function attributes similarly, and just thread that through to the TLI.


That's an interesting idea. How does the linkage work if two functions have different vectlib attributes? Linking against two vectlibs may cause name conflicts or other issues.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77484/new/

https://reviews.llvm.org/D77484





More information about the cfe-commits mailing list