[PATCH] D33429: [NewPM] Initial pipeline for ThinLTO

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 23 11:46:48 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D33429#762340, @timshen wrote:

> Hi Davide,
>
> Did you ever use -Xclang=-fthin-link-bitcode? In our build we use this flag, and it turns out not to be handled with/without your patch. See https://github.com/llvm-mirror/clang/blob/0cb0ba28983d349395d2d38bdedcd9129ce556c4/lib/CodeGen/BackendUtil.cpp#L752 .
>
> I'm going to use your patch as a start point and add the flag support.


I think you just need to add the pass to `EmitAssemblyWithNewPassManager`.
I had that line, but it got lost in the rebase.

https://github.com/llvm-mirror/clang/blob/0cb0ba28983d349395d2d38bdedcd9129ce556c4/lib/CodeGen/BackendUtil.cpp#L1099


https://reviews.llvm.org/D33429





More information about the llvm-commits mailing list