[PATCH] D94630: [LTO] Add support for existing Config::Freestanding option.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/LTO/LTOBackend.cpp:307
+  if (Conf.Freestanding)
+    TLII->disableAllFunctions();
+  FAM.registerPass([&] { return TargetLibraryAnalysis(*TLII); });
----------------
asbirlea wrote:
> 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.
Ah, there are 2 helpers for the `NewPM`, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94630