Re: [PATCH] Discard Scops for which entry==exit

2015-07-02 Thread Tobias Grosser
On 07/02/2015 05:37 PM, Aditya K wrote: A test case (gcc/testsuite/gcc.dg/graphite/pr18792.c) came up when we removed `graphite-scop-detection.c:limit_scops'. The test case is a scop where entry==exit, BB5 (*#) -> BB6 (#); BB6 -> BB5; In this case BB2 is out of the scop. This is basically an e

RE: [PATCH] Discard Scops for which entry==exit

2015-07-02 Thread Aditya K
.1.0.243.g30d45f7 > Date: Thu, 2 Jul 2015 09:53:25 +0200 > From: tob...@grosser.es > To: hiradi...@msn.com; seb...@gmail.com > CC: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] Discard Scops for which entry==exit > > On 06/30/2015 05:47 PM, Aditya K wrote: >> Hi Tobias, &

Re: [PATCH] Discard Scops for which entry==exit

2015-07-02 Thread Tobias Grosser
On 06/30/2015 05:47 PM, Aditya K wrote: Hi Tobias, A test case (gcc/testsuite/gcc.dg/graphite/pr18792.c) came up when we removed `graphite-scop-detection.c:limit_scops'. The test case is a scop where entry==exit, BB5 (*#) -> BB6 (#); BB6 -> BB5; In this case BB2 is out of the scop. This is bas

RE: [PATCH] Discard Scops for which entry==exit

2015-06-30 Thread Aditya K
  >, shape=box, style="setlinewidth(0)"] 0 -> 2; 2 -> 8; 2 -> 3; 3 -> 4; 3 -> 7; 4 -> 9; 9 -> 5; 5 -> 6; 6 -> 5; 7 -> 1; 8 -> 4; } -Aditya > Date: Tue, 30 Jun 2015 08:11:01 +0200 > From: tob...@grosser.

Re: [PATCH] Discard Scops for which entry==exit

2015-06-30 Thread Richard Biener
On Tue, Jun 30, 2015 at 8:11 AM, Tobias Grosser wrote: > On 06/30/2015 02:09 AM, Sebastian Pop wrote: >> >> On Mon, Jun 29, 2015 at 3:04 PM, Aditya Kumar wrote: >>> >>> In this patch we discard the scops where entry and exit are the same BB. >>> This is an effort to remove graphite-scop-detection

Re: [PATCH] Discard Scops for which entry==exit

2015-06-29 Thread Tobias Grosser
On 06/30/2015 02:09 AM, Sebastian Pop wrote: On Mon, Jun 29, 2015 at 3:04 PM, Aditya Kumar wrote: In this patch we discard the scops where entry and exit are the same BB. This is an effort to remove graphite-scop-detection.c:limit_scops. Removing the limit_scops function introduces correctness

Re: [PATCH] Discard Scops for which entry==exit

2015-06-29 Thread Sebastian Pop
On Mon, Jun 29, 2015 at 3:04 PM, Aditya Kumar wrote: > In this patch we discard the scops where entry and exit are the same BB. > This is an effort to remove graphite-scop-detection.c:limit_scops. > Removing the limit_scops function introduces correctness regressions. > We are making relevant chan

[PATCH] Discard Scops for which entry==exit

2015-06-29 Thread Aditya Kumar
In this patch we discard the scops where entry and exit are the same BB. This is an effort to remove graphite-scop-detection.c:limit_scops. Removing the limit_scops function introduces correctness regressions. We are making relevant changes in incremental steps to fix those bugs, and finally we int