[PATCH] D60162: [ThinLTO] Support TargetLibraryInfoImpl in the backend

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In D60162#1472147 <https://reviews.llvm.org/D60162#1472147>, @tejohnson wrote:

> In D60162#1472085 <https://reviews.llvm.org/D60162#1472085>, @dexonsmith wrote:
>
> > I wonder if we could add a module flag for the TLI, and then store that in the summary.  Would it work for both implementations in that case?
>
>
> A module flag summarizing the TLI should theoretically work (we wouldn't need to put anything in the ThinLTO summary, just needs to be in the IR for the backends). There is a lot of stuff in the TLI though, it would all need to be summarized.


Regarding a summary, I believe that TLI's only inputs (currently) are the target triple, the list of disabled functions (or the setting to disable all functions), and the kind of vector functions enabled.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60162