This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt
- From: Richard Biener <rguenther at suse dot de>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Jakub Jelinek <jakub at redhat dot com>, schwab at linux-m68k dot org
- Date: Mon, 8 Jun 2015 12:47:11 +0200 (CEST)
- Subject: Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt
- Authentication-results: sourceware.org; auth=none
- References: <551564D0 dot 2090308 at mentor dot com> <551E8A1F dot 5050908 at mentor dot com> <552E69ED dot 7020601 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1504201400400 dot 20496 at zhemvz dot fhfr dot qr> <55548A19 dot 3000302 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1505261232500 dot 30088 at zhemvz dot fhfr dot qr> <55700C12 dot 9000801 at mentor dot com> <557571B5 dot 1060903 at mentor dot com>
On Mon, 8 Jun 2015, Tom de Vries wrote:
> On 04/06/15 10:28, Tom de Vries wrote:
> > > I'm ok with the patch and count on you to fix eventual fallout ;)
> > >
> >
> > Great, will do.
>
> And here is the fallout:
> * PR66442 - [6 regression] FAIL: gcc.dg/autopar/pr46885.c (test for
> excess errors)
>
> There are two problems in try_transform_to_exit_first_loop_alt:
> 1. In case the latch is not a singleton bb, the function should return
> false rather than true.
> 2. The check for singleton bb should ignore debug-insns.
>
> Attached patch fixes these problems.
>
> Bootstrapped and reg-tested on x86_64.
>
> Verified by Andreas to fix the problem on m68k.
>
> OK for trunk?
Ok.
Thanks,
Richard.