[PATCH] D54564: [LTO] Load sample profile in LTO link step.

Xin Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 15 09:31:37 PST 2018


trentxintong added inline comments.


================
Comment at: lib/Passes/PassBuilder.cpp:1006
+    // Load sample profile before running the LTO optimization pipeline.
+    MPM.addPass(SampleProfileLoaderPass(PGOOpt->SampleProfileFile,
+                                        PGOOpt->ProfileRemappingFile,
----------------
davidxl wrote:
> I thought sample profile is already loaded twice -- one in FE and one in backend compilation. Is it not the case?
I see thats the case for ThinLTO. but not for LTO.


Repository:
  rL LLVM

https://reviews.llvm.org/D54564





More information about the llvm-commits mailing list