This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Enable jump threading on paths meeting hot paths

From: Jeff Law <law at redhat dot com>
To: Jan Hubicka <hubicka at ucw dot cz>
Cc: Richard Biener <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 2 Feb 2017 10:37:10 -0700
Subject: Re: Enable jump threading on paths meeting hot paths
Authentication-results: sourceware.org; auth=none
References: <20170125181107.GB14317@kam.mff.cuni.cz> <CAFiYyc12K=xk_XLp2bQf5VE47WpcybN6DF_zSvWT22ecw03FZw@mail.gmail.com> <20170126100425.GA60443@kam.mff.cuni.cz> <CAFiYyc3kXUfmB09zJVVUX=O7wLmfPag-M_Tv3uOhh8nCfwwfWQ@mail.gmail.com> <20170126103916.GB60443@kam.mff.cuni.cz> <2e81a817-bda6-207d-bbee-16f05a5ee42f@redhat.com> <20170202134919.GA91024@kam.mff.cuni.cz>

On 02/02/2017 06:49 AM, Jan Hubicka wrote:

Hi,
it seems I forgot to send the updated patch. Here it is.
We now dump info like:
Checking profitability of path:  5 (16 insns) 3 (2 insns) 34 (2 insns) 33 (4 insns) 32 (1 insns) 10 (3 insns) 6
  Control statement insns: 16
  Overall: 12 insns
  Registering FSM jump thread: (6, 10) incoming edge;  (10, 32)  (32, 33)  (33, 34)  (34, 3)  (3, 5)  (5, 16) nocopy; (5, 16)

path is printed backwards. It is how the loop process it.

Yea. I've always found that confusing, particularly since the otherthreaders go the other direction.

One thing on the todo list is to stop converting between the tworecording formats and remove the forward/backwards threadingentanglement in tree-ssa-threadupdate.c.

Anyway, the patch looks fine to me as well (please fix the nits Richipointed out).


jeff

References:
- Re: Enable jump threading on paths meeting hot paths
  - From: Jan Hubicka

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]