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: [PATCH] Fix PR70484, RTL DSE using wrong dependence check

From: Bernd Schmidt <bernds_cb1 at t-online dot de>
To: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <rguenther at suse dot de>
Cc: gcc-patches at gcc dot gnu dot org
Date: Fri, 1 Apr 2016 17:16:46 +0200
Subject: Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
Authentication-results: sourceware.org; auth=none
References: <alpine dot LSU dot 2 dot 11 dot 1604011106140 dot 13384 at t29 dot fhfr dot qr> <20160401091906 dot GR3017 at tucnak dot redhat dot com> <alpine dot LSU dot 2 dot 11 dot 1604011134060 dot 13384 at t29 dot fhfr dot qr> <20160401150549 dot GY3017 at tucnak dot redhat dot com>

On 04/01/2016 05:05 PM, Jakub Jelinek wrote:

with my usual pair of rtl,yes checking bootstraps/regtests (x86_64-linux
and i686-linux, former one with ada, latter without), both without your
patch and with the patch.
Without the patch got 66555 successful replace_reads, with your patch
only 65971, that is 1% difference.  I guess that is acceptable level,
though for GCC 7 we should try to improve that (either at the GIMPLE level,
or do something better at the RTL level).

I was also running before/after tests on my set of .i files, and so farthe effect seems negligible.



Bernd

References:
- [PATCH] Fix PR70484, RTL DSE using wrong dependence check
  - From: Richard Biener
- Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
  - From: Jakub Jelinek
- Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
  - From: Richard Biener
- Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
  - From: Jakub Jelinek

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