Re: [PATCH] D11908: Clang support for -fthinlto.

This revision was automatically updated to reflect the committed changes. Closed by commit rL250398: Clang support for -flto=thin. (authored by tejohnson). Changed prior to commit: http://reviews.llvm.org/D11908?vs=36453=37478#toc Repository: rL LLVM http://reviews.llvm.org/D11908 Files:

Re: [PATCH] D11908: Clang support for -fthinlto.

Ping. Thanks, Teresa On Sat, Oct 3, 2015 at 6:51 PM, Teresa Johnson wrote: > tejohnson updated this revision to Diff 36453. > tejohnson added a comment. > > - Rename code gen option to EmitFunctionSummary as per review for > http://reviews.llvm.org/D13107. > - Clang format

Re: [PATCH] D11908: Clang support for -fthinlto.

joker.eph accepted this revision. joker.eph added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D11908 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D11908: Clang support for -fthinlto.

tejohnson updated this revision to Diff 36453. tejohnson added a comment. - Rename code gen option to EmitFunctionSummary as per review for http://reviews.llvm.org/D13107. - Clang format the latest changes. Let me know if it looks good now. http://reviews.llvm.org/D11908 Files:

Re: [PATCH] D11908: Clang support for -fthinlto.

tejohnson added a comment. Sorry for the duplicate - my previous response didn't go to Duncan or Mehdi for some reason. Trying again... In http://reviews.llvm.org/D11908#258540, @klimek wrote: > Perhaps "sharded" would fit what it is? You could have a sharded mode for full FDO (like gcc's

Re: [PATCH] D11908: Clang support for -fthinlto.

> On 2015-Oct-02, at 08:59, Teresa Johnson wrote: > > On Fri, Oct 2, 2015 at 8:53 AM, Manuel Klimek wrote: >> klimek added a comment. >> >> In http://reviews.llvm.org/D11908#258570, @tejohnson wrote: >> >>> Sorry for the duplicate - my previous

Re: [PATCH] D11908: Clang support for -fthinlto.

On Wed, Sep 30, 2015 at 2:33 PM, Duncan P. N. Exon Smith wrote: > +echristo, +espindola, +pcc > >> On 2015-Sep-30, at 12:39, Teresa Johnson wrote: >> >> On Wed, Sep 30, 2015 at 11:11 AM, Duncan P. N. Exon Smith >> wrote: >>>

Re: [PATCH] D11908: Clang support for -fthinlto.

On Wed, Sep 30, 2015 at 10:19 AM, Duncan P. N. Exon Smith wrote: > >> On 2015-Sep-23, at 10:28, Teresa Johnson wrote: >> >> tejohnson updated this revision to Diff 35527. >> tejohnson added a comment. >> >> Updated the patch for the new LLVM/gold patch

Re: [PATCH] D11908: Clang support for -fthinlto.

> On 2015-Sep-23, at 10:28, Teresa Johnson wrote: > > tejohnson updated this revision to Diff 35527. > tejohnson added a comment. > > Updated the patch for the new LLVM/gold patch > (http://reviews.llvm.org/D13107). > > Two changes: > > - I put back the original change

Re: [PATCH] D11908: Clang support for -fthinlto.

> On 2015-Sep-30, at 10:40, Teresa Johnson wrote: > > On Wed, Sep 30, 2015 at 10:19 AM, Duncan P. N. Exon Smith > wrote: >> >>> On 2015-Sep-23, at 10:28, Teresa Johnson wrote: >>> >>> tejohnson updated this revision to Diff

Re: [PATCH] D11908: Clang support for -fthinlto.

+echristo, +espindola, +pcc > On 2015-Sep-30, at 12:39, Teresa Johnson wrote: > > On Wed, Sep 30, 2015 at 11:11 AM, Duncan P. N. Exon Smith > wrote: >> >>> On 2015-Sep-30, at 10:40, Teresa Johnson wrote: >>> >>> On Wed, Sep

Re: [PATCH] D11908: Clang support for -fthinlto.

tejohnson updated this revision to Diff 35527. tejohnson added a comment. Updated the patch for the new LLVM/gold patch (http://reviews.llvm.org/D13107). Two changes: - I put back the original change to pass a new plugin option to gold. Since the function index design/name has been generalized