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: Fix PR 65177: diamonds are not valid execution threads for jump threading

From: Jeff Law <law at redhat dot com>
To: Sebastian Pop <sebpop at gmail dot com>, Richard Biener <richard dot guenther at gmail dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 20 Mar 2015 12:24:19 -0600
Subject: Re: Fix PR 65177: diamonds are not valid execution threads for jump threading
Authentication-results: sourceware.org; auth=none
References: <20150318223541 dot GA17168 at f1 dot c dot bardezibar dot internal> <CAFiYyc04U3Yvji=jp+A3shgR+fgZp4fF9A_C2KVGeTd6uDF47A at mail dot gmail dot com> <20150319195402 dot GA21421 at f1 dot c dot bardezibar dot internal> <CAFiYyc11dWMb2_=n3ZogxW7zUAdzGZ=b=cpxeD6d5RCjN7CR6Q at mail dot gmail dot com> <20150320181716 dot GB21421 at f1 dot c dot bardezibar dot internal>

On 03/20/2015 12:17 PM, Sebastian Pop wrote:

Richard Biener wrote:

On Thu, Mar 19, 2015 at 8:54 PM, Sebastian Pop <sebpop@gmail.com> wrote:

Richard Biener wrote:

please instead fixup after copy_bbs in duplicate_seme_region.


Thanks for the review.
Attached patch that does not modify copy_bbs.
Fixes make check in hmmer and make check RUNTESTFLAGS=tree-ssa.exp

Full bootstrap and regtest in progress on x86_64-linux.  Ok for trunk?


Looks good to me.  Please also add a testcase.


Attached.  I will wait to commit until I have a green light from Jeff.

Sorry, dealing with some non-technical stuff right now. I should havetime to look at this over the weekend.


jeff

References:
- Fix PR 65177: diamonds are not valid execution threads for jump threading
  - From: Sebastian Pop
- Re: Fix PR 65177: diamonds are not valid execution threads for jump threading
  - From: Richard Biener
- Re: Fix PR 65177: diamonds are not valid execution threads for jump threading
  - From: Sebastian Pop
- Re: Fix PR 65177: diamonds are not valid execution threads for jump threading
  - From: Richard Biener
- Re: Fix PR 65177: diamonds are not valid execution threads for jump threading
  - From: Sebastian Pop

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