tejohnson marked an inline comment as done.
tejohnson added inline comments.
================
Comment at: llvm/lib/LTO/LTOBackend.cpp:221
+static TargetLibraryInfoImpl *createTLII(Module &Mod, TargetMachine *TM) {
+ TargetLibraryInfoImpl *TLII =
----------------
tejohnson wrote:
> steven_wu wrote:
> > Should this be done not just for LTOBackend but for regular compilation as
> > well? LegacyCodegenerator and llc can all be benefit from a matching
> > TargetLibraryInfo?
> Yeah, probably. I think the best way to have this utilized everywhere is to
> move the below code into the TargetLibraryInfoImpl itself - by having it also
> take the Module as a parameter). Probably as a required parameter, to ensure
> it is used consistently. WDYT?
I meant, "into the TargetLibraryInfoImpl constructor"
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60162/new/
https://reviews.llvm.org/D60162
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
