testsuite typo fixed in [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-18 Thread Hans-Peter Nilsson
From: Tom de Vries tom_devr...@mentor.com Date: Thu, 15 Dec 2011 10:23:27 +0100 bootstrapped and reg-tested (ada inclusive) on x86_64. Time to update your inspection script: there's a missing sanity check for the number of passes. I suggest you re-run regtests for the main change with the

Re: testsuite typo fixed in [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-18 Thread Hans-Peter Nilsson
From: Hans-Peter Nilsson h...@axis.com Date: Sun, 18 Dec 2011 09:32:36 +0100 Typos like this cause the test-run to be aborted for the current .exp file, In this case, some 3000 test-cases went missing, judging by the gcc Summary output. Correction: not just the .exp file but the whole

Re: testsuite typo fixed in [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-18 Thread Tom de Vries
On 18/12/11 09:47, Hans-Peter Nilsson wrote: From: Hans-Peter Nilsson h...@axis.com Date: Sun, 18 Dec 2011 09:32:36 +0100 Typos like this cause the test-run to be aborted for the current .exp file, In this case, some 3000 test-cases went missing, judging by the gcc Summary output.

Re: [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-16 Thread Richard Henderson
On 12/15/2011 01:23 AM, Tom de Vries wrote: PR tree-optimization/51491 * tree-ssa-ccp.c (insert_clobber_before_stack_restore) (gsi_prev_dom_bb_nondebug, insert_clobbers_for_var): New function. (ccp_fold_stmt): Use insert_clobbers_for_var after a successful

Re: [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-15 Thread Tom de Vries
On 13/12/11 14:13, Jakub Jelinek wrote: On Tue, Dec 13, 2011 at 01:26:42PM +0100, Tom de Vries wrote: 2011-12-13 Tom de Vries t...@codesourcery.com PR tree-optimization/51491 * tree-ssa-ccp.c (insert_clobber_before_stack_restore): New function. (ccp_fold_stmt): Use

[PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-13 Thread Tom de Vries
Jakub, I have a patch for PR51491. Consider the following test-case, compiled with gcc -O2 -S f.c -fdump-tree-all: ... int f(void) { int tt = 0; int t = 4; { int a[t]; tt = g(a); tt += a[0]; } { int a[4]; tt += g(a); tt += a[0]; } return tt; } ... The vla

Re: [PATCH, PR51491] add CLOBBER before __builtin_stack_restore when converting vla alloca_with_align to array decl

2011-12-13 Thread Jakub Jelinek
On Tue, Dec 13, 2011 at 01:26:42PM +0100, Tom de Vries wrote: 2011-12-13 Tom de Vries t...@codesourcery.com PR tree-optimization/51491 * tree-ssa-ccp.c (insert_clobber_before_stack_restore): New function. (ccp_fold_stmt): Use insert_clobber_before_stack_restore after a