Author: dehao
Date: Thu Aug 24 14:37:33 2017
New Revision: 311707
URL: http://llvm.org/viewvc/llvm-project?rev=311707&view=rev
Log:
Expose -mllvm -accurate-sample-profile to clang.
Summary: With accurate sample profile, we can do more aggressive size
optimization. For some size-critical applicat
Wed, Jan 18, 2017 at 4:44 PM, Dehao Chen via cfe-commits
> wrote:
> > Author: dehao
> > Date: Wed Jan 18 18:44:21 2017
> > New Revision: 292458
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=292458&view=rev
> > Log:
> > Add -fdebug-info-for-
Author: dehao
Date: Thu Jul 27 08:29:53 2017
New Revision: 309282
URL: http://llvm.org/viewvc/llvm-project?rev=309282&view=rev
Log:
Make new PM honor -fdebug-info-for-profiling (clang side)
Summary: The new PM needs to invoke add-discriminator pass when building with
-fdebug-info-for-profiling.
This test was originally added in https://reviews.llvm.org/D34721 with
clang change. It's kind of dup of the previous test
(-check-prefix=SAMPLEPGO) in terms of testing the clang bits. But we
want to make sure the new PM has the expected behavior. I guess it
would be acceptable to remove one of the
Author: dehao
Date: Mon Jul 10 13:31:37 2017
New Revision: 307584
URL: http://llvm.org/viewvc/llvm-project?rev=307584&view=rev
Log:
Use DenseMap instead std::map for GVSummaryMapTy
Summary: Frontend change for https://reviews.llvm.org/D35148
Reviewers: tejohnson
Reviewed By: tejohnson
Subscrib
Author: dehao
Date: Fri Jul 7 15:40:37 2017
New Revision: 307449
URL: http://llvm.org/viewvc/llvm-project?rev=307449&view=rev
Log:
Revert r307445 as it breaks on certain platforms.
Modified:
cfe/trunk/test/CodeGen/Inputs/pgo-sample.prof
cfe/trunk/test/CodeGen/pgo-sample.c
Modified: cfe/
Author: dehao
Date: Fri Jul 7 15:01:47 2017
New Revision: 307445
URL: http://llvm.org/viewvc/llvm-project?rev=307445&view=rev
Log:
Add sample PGO integration test to cover profile annotation and inlining.
Summary: The patch makes the integration test cover major sample PGO components.
Reviewers
Author: dehao
Date: Fri Jul 7 13:53:17 2017
New Revision: 307438
URL: http://llvm.org/viewvc/llvm-project?rev=307438&view=rev
Log:
Enable the new PM + SamlePGO + ThinLTO testing.
Summary: This patch should be enabled after https://reviews.llvm.org/D34895
Reviewers: chandlerc, tejohnson, davidxl
Author: dehao
Date: Thu Jun 29 16:33:13 2017
New Revision: 306764
URL: http://llvm.org/viewvc/llvm-project?rev=306764&view=rev
Log:
[PM] Add support for sample PGO in the new pass manager (clang-side)
Summary: This implements the clang bits of https://reviews.llvm.org/D34720, and
add correspondi
Author: dehao
Date: Tue Jun 27 10:45:40 2017
New Revision: 306434
URL: http://llvm.org/viewvc/llvm-project?rev=306434&view=rev
Log:
Update the test comment to clarify the intention of the test.
Modified:
cfe/trunk/test/CodeGen/pgo-sample-thinlto-summary.c
Modified: cfe/trunk/test/CodeGen/pgo
Author: dehao
Date: Tue Jun 27 10:23:42 2017
New Revision: 306430
URL: http://llvm.org/viewvc/llvm-project?rev=306430&view=rev
Log:
Update test for enabling ICP for AutoFDO.
Summary: This is the test update patch for https://reviews.llvm.org/D34662
Reviewers: davidxl
Reviewed By: davidxl
Subsc
My understanding is that r302938 makes clang generate incorrect code (clang
itself), which lead to unexpected clang behavior. Is it correct? If yes,
how can I reproduce this issue so that I can try to triage/fix the problem?
Thanks,
Dehao
On Thu, May 18, 2017 at 1:22 PM, Richard Smith
wrote:
>
What's the issue? Build breaking? Performance regression? It's not clear
from the limit info in this thread...
Dehao
On Thu, May 18, 2017 at 1:02 PM, Vitaly Buka wrote:
> Local build: r302937 no issue, r302938 has issue.
>
> On Thu, May 18, 2017 at 7:23 AM Dehao Chen wrote:
>
>> Could you give
Could you give some context on how r302938 is related to this?
Thanks,
Dehao
On Wed, May 17, 2017 at 11:14 PM, Vitaly Buka wrote:
> +Dehao Chen
> it started from r302938
>
> On Wed, May 17, 2017 at 8:09 PM Jordan Rose via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Thanks, this is
Author: dehao
Date: Thu Mar 23 16:20:17 2017
New Revision: 298647
URL: http://llvm.org/viewvc/llvm-project?rev=298647&view=rev
Log:
Update the SamplePGO test to verify that unroll/icp is not invoked in thinlto
compile phase.
Summary: This is the test added for https://reviews.llvm.org/D31217
Re
Author: dehao
Date: Tue Mar 21 16:40:53 2017
New Revision: 298446
URL: http://llvm.org/viewvc/llvm-project?rev=298446&view=rev
Log:
Add support for -fno-auto-profile and -fno-profile-sample-use
Summary: We need to be able to disable samplepgo for specific files by
supporting -fno-auto-profile an
Author: dehao
Date: Tue Mar 21 14:55:46 2017
New Revision: 298429
URL: http://llvm.org/viewvc/llvm-project?rev=298429&view=rev
Log:
Clang change: Do not inline hot callsites for samplepgo in thinlto compile
phase.
Summary:
Because SamplePGO passes will be invoked twice in ThinLTO build: once at
Author: dehao
Date: Tue Feb 21 14:36:21 2017
New Revision: 295764
URL: http://llvm.org/viewvc/llvm-project?rev=295764&view=rev
Log:
Only enable AddDiscriminator pass when -fdebug-info-for-profiling is true
Summary: AddDiscriminator pass is only useful for sample pgo. This patch
restricts AddDisc
Author: dehao
Date: Wed Feb 1 16:45:21 2017
New Revision: 293834
URL: http://llvm.org/viewvc/llvm-project?rev=293834&view=rev
Log:
Change debug-info-for-profiling from a TargetOption to a function attribute.
Summary: cfe change for https://reviews.llvm.org/D29203
Reviewers: echristo, dblaikie
Author: dehao
Date: Wed Jan 18 18:44:21 2017
New Revision: 292458
URL: http://llvm.org/viewvc/llvm-project?rev=292458&view=rev
Log:
Add -fdebug-info-for-profiling to emit more debug info for sample pgo profile
collection
Summary:
SamplePGO uses profile with debug info to collect profile. Unlike
On Thu, Jan 12, 2017 at 3:27 PM, Teresa Johnson via Phabricator <
revi...@reviews.llvm.org> wrote:
> tejohnson added a comment.
>
> In https://reviews.llvm.org/D28588#644489, @danielcdh wrote:
>
> > Thanks for the prompt response.
> >
> > But looks like several other tests also has "-mllvm
> -debu
Author: dehao
Date: Thu Jan 12 19:09:43 2017
New Revision: 291874
URL: http://llvm.org/viewvc/llvm-project?rev=291874&view=rev
Log:
Temporarily revert the test change in 291870, which is broken in certain
buildbots.
Modified:
cfe/trunk/test/CodeGen/thinlto_backend.ll
Modified: cfe/trunk/tes
Author: dehao
Date: Thu Jan 12 18:51:55 2017
New Revision: 291870
URL: http://llvm.org/viewvc/llvm-project?rev=291870&view=rev
Log:
Pass -fprofile-sample-use to lto backends.
Summary: LTO backend will not invoke SampleProfileLoader pass even if
-fprofile-sample-use is specified. This patch passe
Author: dehao
Date: Thu Jan 12 10:56:18 2017
New Revision: 291775
URL: http://llvm.org/viewvc/llvm-project?rev=291775&view=rev
Log:
Revert r291774 which caused buildbot failure.
Modified:
cfe/trunk/lib/CodeGen/BackendUtil.cpp
cfe/trunk/test/CodeGen/thinlto_backend.ll
Modified: cfe/trunk/
Author: dehao
Date: Thu Jan 12 10:29:25 2017
New Revision: 291774
URL: http://llvm.org/viewvc/llvm-project?rev=291774&view=rev
Log:
Pass -fprofile-sample-use to lto backends.
Summary: LTO backend will not invoke SampleProfileLoader pass even if
-fprofile-sample-use is specified. This patch passe
Author: dehao
Date: Tue Jan 3 18:33:23 2017
New Revision: 290919
URL: http://llvm.org/viewvc/llvm-project?rev=290919&view=rev
Log:
Add -plugin-opt=sample-profile for thinLTO build.
Summary: ThinLTO needs to pass down the sample profile file path to linker.
Reviewers: tejohnson, davidxl, mehdi_a
Author: dehao
Date: Wed Dec 14 15:41:04 2016
New Revision: 289715
URL: http://llvm.org/viewvc/llvm-project?rev=289715&view=rev
Log:
Create SampleProfileLoader pass in llvm instead of clang
Summary:
We used to create SampleProfileLoader pass in clang. This makes LTO/ThinLTO
unable to add this pas
Author: dehao
Date: Wed Dec 14 11:22:53 2016
New Revision: 289675
URL: http://llvm.org/viewvc/llvm-project?rev=289675&view=rev
Log:
revert r289670 which breaks bot.
Modified:
cfe/trunk/lib/CodeGen/BackendUtil.cpp
Modified: cfe/trunk/lib/CodeGen/BackendUtil.cpp
URL:
http://llvm.org/viewvc/ll
Author: dehao
Date: Wed Dec 14 10:49:34 2016
New Revision: 289670
URL: http://llvm.org/viewvc/llvm-project?rev=289670&view=rev
Log:
Create SampleProfileLoader pass in llvm instead of clang
Summary:
We used to create SampleProfileLoader pass in clang. This makes LTO/ThinLTO
unable to add this pas
Author: dehao
Date: Fri Oct 7 10:21:29 2016
New Revision: 283564
URL: http://llvm.org/viewvc/llvm-project?rev=283564&view=rev
Log:
Update clang test to accomendate discriminator change in
https://reviews.llvm.org/D25132
Summary: https://reviews.llvm.org/D25132 added discriminator even add -g0.
Author: dehao
Date: Mon Sep 19 11:02:52 2016
New Revision: 281910
URL: http://llvm.org/viewvc/llvm-project?rev=281910&view=rev
Log:
Remove InstructionCombining and its related pass from sample pgo passes as we
can handle "invoke" correctly.
Summary: We previously relies on InstructionCombining p
Author: dehao
Date: Wed Sep 14 12:34:14 2016
New Revision: 281509
URL: http://llvm.org/viewvc/llvm-project?rev=281509&view=rev
Log:
Convert finite to builtin
Summary: This patch converts finite/__finite to builtin functions so that it
will be inlined by compiler.
Reviewers: hfinkel, davidxl, ef
Author: dehao
Date: Thu Jun 23 15:13:10 2016
New Revision: 273606
URL: http://llvm.org/viewvc/llvm-project?rev=273606&view=rev
Log:
Invoke simplifycfg and sroa before instcombine.
Summary: InstCombine needs to be performed after simplifycfg and sroa,
otherwise it may make bad optimization decisi
Author: dehao
Date: Tue Jun 21 14:16:41 2016
New Revision: 273298
URL: http://llvm.org/viewvc/llvm-project?rev=273298&view=rev
Log:
Invoke PruneEH pass before Sample Profile pass.
Summary: We need to call PruneEH pass before AutoFDO pass so that some
EH-related calls can get inlined in Sample Pr
Author: dehao
Date: Fri May 27 11:14:35 2016
New Revision: 271010
URL: http://llvm.org/viewvc/llvm-project?rev=271010&view=rev
Log:
Add instcombine pass if sampleprofile pass is enabled.
Summary: Sample profile pass need to have instcombine pass. A related change is
http://reviews.llvm.org/D1774
Author: dehao
Date: Fri Nov 20 12:29:01 2015
New Revision: 253686
URL: http://llvm.org/viewvc/llvm-project?rev=253686&view=rev
Log:
Add -disable-llvm-passes for frontend debug info tests.
Summary: The frontend debuginfo tests should not invoke llvm passes which
includes add-discriminators that w
danielcdh marked an inline comment as done.
danielcdh added a comment.
http://reviews.llvm.org/D14848
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Yes, as discriminator is a backend pass, it should *not* affect frontend tests.
On Thu, Nov 19, 2015 at 12:36 PM, David Blaikie wrote:
>
>
> On Thu, Nov 19, 2015 at 11:53 AM, Dehao Chen via cfe-commits
> wrote:
>>
>> Author: dehao
>> Date: Thu Nov 19 13:53:1
Author: dehao
Date: Thu Nov 19 13:53:16 2015
New Revision: 253595
URL: http://llvm.org/viewvc/llvm-project?rev=253595&view=rev
Log:
Update clang tests to accomendate discriminator changes.
Summary: The discriminator change in http://reviews.llvm.org/D14738 will fail
these clang tests. Update the
Author: dehao
Date: Fri Oct 30 00:08:43 2015
New Revision: 251691
URL: http://llvm.org/viewvc/llvm-project?rev=251691&view=rev
Log:
Update debug-info-scope test to remove "FIXME", which is fixed in r251689
Modified:
cfe/trunk/test/CodeGen/debug-info-scope.c
Modified: cfe/trunk/test/CodeGen/d
Author: dehao
Date: Tue Sep 29 20:03:10 2015
New Revision: 248869
URL: http://llvm.org/viewvc/llvm-project?rev=248869&view=rev
Log:
Fix the sample profile format that breaks in test
http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/505
http://reviews.llvm.org/D1
41 matches
Mail list logo
