This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC][PATCH] Introduce -fdump*-folding
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Martin Liška <mliska at suse dot cz>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Wed, 3 May 2017 01:14:43 -0700
- Subject: Re: [RFC][PATCH] Introduce -fdump*-folding
- Authentication-results: sourceware.org; auth=none
- References: <668c62b5-9603-47c5-9bfe-266d0d369a1f@suse.cz>
On Wed, May 3, 2017 at 1:10 AM, Martin Liška <mliska@suse.cz> wrote:
> Hello
>
> Last release cycle I spent quite some time with reading of IVOPTS pass
> dump file. Using -fdump*-details causes to generate a lot of 'Applying pattern'
> lines, which can make reading of a dump file more complicated.
Yes the folding message can get annoying. Especially when it is not
clear what is being folded which is the case a lot of the time I have
seen the message.
Thanks,
Andrew
>
> There are stats for tramp3d with -O2 and -fdump-tree-all-details. Percentage number
> shows how many lines are of the aforementioned pattern:
>
> tramp3d-v4.cpp.164t.ivopts: 6.34%
> tramp3d-v4.cpp.091t.ccp2: 5.04%
> tramp3d-v4.cpp.093t.cunrolli: 4.41%
> tramp3d-v4.cpp.129t.laddress: 3.70%
> tramp3d-v4.cpp.032t.ccp1: 2.31%
> tramp3d-v4.cpp.038t.evrp: 1.90%
> tramp3d-v4.cpp.033t.forwprop1: 1.74%
> tramp3d-v4.cpp.103t.vrp1: 1.52%
> tramp3d-v4.cpp.124t.forwprop3: 1.31%
> tramp3d-v4.cpp.181t.vrp2: 1.30%
> tramp3d-v4.cpp.161t.cunroll: 1.22%
> tramp3d-v4.cpp.027t.fixup_cfg3: 1.11%
> tramp3d-v4.cpp.153t.ivcanon: 1.07%
> tramp3d-v4.cpp.126t.ccp3: 0.96%
> tramp3d-v4.cpp.143t.sccp: 0.91%
> tramp3d-v4.cpp.185t.forwprop4: 0.82%
> tramp3d-v4.cpp.011t.cfg: 0.74%
> tramp3d-v4.cpp.096t.forwprop2: 0.50%
> tramp3d-v4.cpp.019t.fixup_cfg1: 0.37%
> tramp3d-v4.cpp.120t.phicprop1: 0.33%
> tramp3d-v4.cpp.133t.pre: 0.32%
> tramp3d-v4.cpp.182t.phicprop2: 0.27%
> tramp3d-v4.cpp.170t.veclower21: 0.25%
> tramp3d-v4.cpp.029t.einline: 0.24%
>
> I'm suggesting to add new TDF that will be allocated for that.
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Thoughts?
> Martin