[PATCH] D94630: [LTO] Add support for existing Config::Freestanding option.
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 21 13:24:47 PST 2021
asbirlea added inline comments.
================
Comment at: llvm/lib/LTO/LTOBackend.cpp:307
+ if (Conf.Freestanding)
+ TLII->disableAllFunctions();
+ FAM.registerPass([&] { return TargetLibraryAnalysis(*TLII); });
----------------
aeubanks wrote:
> fhahn wrote:
> > @asbirlea I am trying to make the new PM use a TargetLibraryInfo with all functions disabled here. But for some reason, it appears as if the custom TLI is not used. Do you have any idea what the right way to do that would be?
> I think you want `runNewPMPasses` on top of `runNewPMCustomPasses` (idk how much `runNewPMCustomPasses` is used).
^ add these changes to `runNewPMPasses` as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94630/new/
https://reviews.llvm.org/D94630
More information about the llvm-commits
mailing list