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

[Xinliang David Li via llvm-commits]

ok. thanks.

On Thu, Nov 15, 2018 at 9:31 AM Xin Tong via Phabricator via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> 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
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181115/9159f2b4/attachment.html>