Re: [PATCH GCC8][33/33]Fix PR69710/PR68030 by reassociate vect base address and a simple CSE pass

[Michael Meissner <meissner at linux dot vnet dot ibm dot com>]

On Wed, Apr 19, 2017 at 02:08:49PM +0100, Bin.Cheng wrote:
> Hi Michael,
> Thanks for testing it.  Could you have a check if updated patch
> resolves the ICE?
> As for sms-*.c tests, I had difficulty in reproducing it.  Both trunk
> and patched GCC have the same behavior with my local ppc cross
> toolchain, sms-1.c fails and sms-4.c succeeds.

I backed out the original patch #33, and replaced it with the one you sent me,
and it does allow dealII to be built.  I rebuilt all 30 benchmarks in Spec
2006, and now they all build.

I ran a full spec 2006 suite on a little endian power8 that we normally use to
run Spec.  Note, I use different options to build spec that the people that do
the official runs, so the results are in no way official results.  I'm only
providing the differences, and not the spec numbers.

Generally, I consider differences between runs that were less than 2% to be in
the noise level.  There were only two benchmarks that had differences more than
2%:

 * 456.hmmer had an 8% regression
 * 434.zeusmp had a 3% speedup.

In terms of smaller differences, the following had 1% differences:

 * 410.bwaves had a 1% regression
 * 459.GemsFDTD had a 1% regression
 * 437.leslie3d had a 1% speedup
 * 453.povray had a 1.6% speedup

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.vnet.ibm.com, phone: +1 (978) 899-4797

Attachment: 2017-04-19-ivopts-64bit.out.bz2
Description: BZip2 compressed data