I fixed this in a follow-up patch. Sebastian
-----Original Message----- From: H.J. Lu [mailto:hjl.to...@gmail.com] Sent: Monday, September 28, 2015 2:39 PM To: Tobias Grosser Cc: Sebastian Pop; GCC Patches; Sebastian Pop; aditya...@samsung.com; Richard Biener Subject: Re: [PATCH] fix PR67700 On Sat, Sep 26, 2015 at 3:34 AM, Tobias Grosser <tob...@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...@samsung.com> >> Sebastian Pop <s....@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.
