This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR46761] graphite: fix testing of boundary wrapping
- From: Sebastian Pop <sebpop at gmail dot com>
- To: Alexander Monakov <amonakov at ispras dot ru>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 13 Dec 2010 09:39:25 -0600
- Subject: Re: [PR46761] graphite: fix testing of boundary wrapping
- References: <alpine.LNX.2.00.1012131624320.5687@monoid.intra.ispras.ru>
On Mon, Dec 13, 2010 at 07:40, Alexander Monakov <amonakov@ispras.ru> wrote:
> Hi,
>
> As indicated by the testcase, sometimes Graphite would generate wrong region
> guards if UB_ONE overflows, but does not become zero (i.e. it is signed).
> The patch changes the corresponding test to use TREE_OVERFLOW flag.
>
> Bootstrapped and regtested on x86_64-linux, OK for trunk?
>
> 2010-12-13 Alexander Monakov <amonakov@ispras.ru>
>
> PR middle-end/46761
> * graphite-clast-to-gimple.c (graphite_create_new_loop_guard): Use
> TREE_OVERFLOW_P to test overflow.
>
> testsuite:
> * gcc.dg/graphite/pr46761.c: New.
>
Ok. Thanks for fixing this,
Sebastian