[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