Bug 65506 - [5 Regression] FAIL: gcc.dg/pr29215.c scan-tree-dump-not gimple "memcpy"
Summary: [5 Regression] FAIL: gcc.dg/pr29215.c scan-tree-dump-not gimple "memcpy"
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: testsuite (show other bugs)
Version: 5.0
: P3 normal
Target Milestone: 5.0
Assignee: Jakub Jelinek
URL:
Keywords:
: 65529 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-03-21 18:20 UTC by John David Anglin
Modified: 2015-03-25 13:20 UTC (History)
1 user (show)

See Also:
Host: hppa*-*-*
Target: hppa*-*-*
Build: hppa*-*-*
Known to work:
Known to fail:
Last reconfirmed: 2015-03-23 00:00:00


Attachments
Gimple dump (374 bytes, text/plain)
2015-03-21 18:20 UTC, John David Anglin
Details
gcc5-pr65506.patch (413 bytes, patch)
2015-03-23 14:05 UTC, Jakub Jelinek
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John David Anglin 2015-03-21 18:20:27 UTC
Created attachment 35088 [details]
Gimple dump

spawn /home/dave/gnu/gcc/objdir/gcc/xgcc -B/home/dave/gnu/gcc/objdir/gcc/ /home/
dave/gnu/gcc/gcc/gcc/testsuite/gcc.dg/pr29215.c -fno-diagnostics-show-caret -fdi
agnostics-color=never -O2 -fdump-tree-gimple -S -o pr29215.s
PASS: gcc.dg/pr29215.c (test for excess errors)
FAIL: gcc.dg/pr29215.c scan-tree-dump-not gimple "memcpy"

This is revision 221548.  Revision 221505 was okay.
Comment 1 Richard Biener 2015-03-23 09:46:40 UTC
Ok, that's probably caused by

2015-03-20  Richard Biener  <rguenther@suse.de>

        PR middle-end/64715
        * tree-chrec.c (chrec_fold_poly_cst): Use useless_type_conversion_p
        for type comparison and gcc_checking_assert.
        (chrec_fold_plus_poly_poly): Likewise.
        (chrec_fold_multiply_poly_poly): Likewise.
        (chrec_convert_1): Likewise.
        * gimplify.c (gimplify_expr): Remove premature folding of
        &X + CST to &MEM[&X, CST].

and should be fixed by adjusting the testcase to scan not the gimple dump
but the ccp1 dump.  Can you try that?
Comment 2 Jakub Jelinek 2015-03-23 14:05:32 UTC
Created attachment 35111 [details]
gcc5-pr65506.patch

Indeed, have verified this with the cross-compiler and the attached patch should cure this.
Comment 3 Richard Biener 2015-03-23 14:43:52 UTC
Looks good to me.
Comment 4 dave.anglin 2015-03-23 14:47:45 UTC
On 2015-03-23 10:05 AM, jakub at gcc dot gnu.org wrote:
> Indeed, have verified this with the cross-compiler and the attached patch
> should cure this.
I have applied the patch for testing but hit pr65522 regression. 
Restarted build without ada.
Will take about a day to confirm fix.

Dave
Comment 5 Jakub Jelinek 2015-03-23 20:05:14 UTC
Author: jakub
Date: Mon Mar 23 20:04:43 2015
New Revision: 221607

URL: https://gcc.gnu.org/viewcvs?rev=221607&root=gcc&view=rev
Log:
2015-03-23  Jakub Jelinek  <jakub@redhat.com>

	PR testsuite/65506
	* gcc.dg/pr29215.c: Dump and analyze ccp1 dump instead of
	gimple dump.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/pr29215.c
Comment 6 Jakub Jelinek 2015-03-23 20:05:48 UTC
Fixed.
Comment 7 Oleg Endo 2015-03-23 21:32:08 UTC
*** Bug 65529 has been marked as a duplicate of this bug. ***
Comment 8 Alan Lawrence 2015-03-25 13:20:46 UTC
This test was also failing for target arm-none-eabi, also fixed by Jakub's r221607.