This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] fix PR67700


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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]