This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] fix PR67700
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Tobias Grosser <tobias at grosser dot es>
- Cc: Sebastian Pop <s dot pop at samsung dot com>, 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 12:39:08 -0700
- 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>
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.