Re: PATCH: fix infinite loop in CSE

2011-12-19 Thread Sandra Loosemore
On 12/18/2011 05:21 AM, Eric Botcazou wrote: OK everywhere without the cap and with the same test on ARG2: If a testcase doesn't need fancy options, it must go in gcc.c-torture/compile. OK. I've checked in this version, which addresses both those issues. -Sandra 2011-12-19 Sandra Loosemor

Re: PATCH: fix infinite loop in CSE

2011-12-18 Thread Eric Botcazou
> This patch fixes the specific problem in the test case by skipping over > equivalences that would rewrite to exactly the same expression as on the > current iteration. But, it's not clear that there can't also be cycles > of length > 1. I don't see much point in getting fancy here (I assume > t

Re: PATCH: fix infinite loop in CSE

2011-12-12 Thread Sandra Loosemore
On 12/12/2011 08:00 PM, Andrew Pinski wrote: On Mon, Dec 12, 2011 at 6:53 PM, Sandra Loosemore wrote: The test case in the attached patch gets stuck in an infinite loop in find_comparison_args in CSE when compiled for MIPS at -O2. This bug has been present at least as far back as GCC 4.5 and

Re: PATCH: fix infinite loop in CSE

2011-12-12 Thread Andrew Pinski
On Mon, Dec 12, 2011 at 6:53 PM, Sandra Loosemore wrote: > The test case in the attached patch gets stuck in an infinite loop in > find_comparison_args in CSE when compiled for MIPS at -O2.  This bug has > been present at least as far back as GCC 4.5 and probably much earlier than > that. > > The

PATCH: fix infinite loop in CSE

2011-12-12 Thread Sandra Loosemore
The test case in the attached patch gets stuck in an infinite loop in find_comparison_args in CSE when compiled for MIPS at -O2. This bug has been present at least as far back as GCC 4.5 and probably much earlier than that. The problem is that the inner loop over equivalences in the hash tabl