This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PATCH] fix PR67700
- From: Sebastian Paul Pop <s dot pop at samsung dot com>
- To: "'H.J. Lu'" <hjl dot tools at gmail dot com>, 'Tobias Grosser' <tobias at grosser dot es>
- Cc: 'GCC Patches' <gcc-patches at gcc dot gnu dot org>, 'Sebastian Pop' <sebpop at gmail dot com>, aditya dot k7 at samsung dot com, 'Richard Biener' <richard dot guenther at gmail dot com>
- Date: Mon, 28 Sep 2015 17:48:41 -0500
- Subject: RE: [PATCH] fix PR67700
- Authentication-results: sourceware.org; auth=none
- References: <1443213563-16853-1-git-send-email-s dot pop at samsung dot com> <560674D3 dot 2050001 at grosser dot es> <CAMe9rOrL18Ftn7NiqE9JxO2_=Epff1Jw=iv2qhwzEmabEjax+A at mail dot gmail dot com>
I fixed this in a follow-up patch.
Sebastian
-----Original Message-----
From: H.J. Lu [mailto:hjl.tools@gmail.com]
Sent: Monday, September 28, 2015 2:39 PM
To: Tobias Grosser
Cc: Sebastian Pop; GCC Patches; Sebastian Pop; aditya.k7@samsung.com; Richard Biener
Subject: Re: [PATCH] fix PR67700
On Sat, Sep 26, 2015 at 3:34 AM, Tobias Grosser <tobias@grosser.es> wrote:
> On 09/25/2015 10:39 PM, Sebastian Pop wrote:
>>
>> The patch makes the detection of scop parameters in
>> parameter_index_in_region a
>> bit more conservative by discarding scalar variables defined in function
>> of data
>> references defined in the scop.
>>
>> 2015-09-25 Aditya Kumar <aditya.k7@samsung.com>
>> Sebastian Pop <s.pop@samsung.com>
>>
>> PR tree-optimization/67700
>> * graphite-sese-to-poly.c (parameter_index_in_region):
>> Call
>> invariant_in_sese_p_rec.
>> (extract_affine): Same.
>> (rewrite_cross_bb_scalar_deps): Call update_ssa.
>> * sese.c (invariant_in_sese_p_rec): Export. Handle vdefs
>> and vuses.
>> * sese.h (invariant_in_sese_p_rec): Declare.
>>
>> * testsuite/gcc.dg/graphite/run-id-pr67700.c: New.
It breaks bootstrap on x86:
https://gcc.gnu.org/ml/gcc-regression/2015-09/msg00382.html
../../src-trunk/gcc/sese.c: In function Ãbool
invariant_in_sese_p_rec(tree, sese)Ã:
../../src-trunk/gcc/sese.c:781:12: error: unused variable ÃvdefÃ
[-Werror=unused-variable]
if (tree vdef = gimple_vdef (stmt))
^
--
H.J.