[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 a[

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 > > 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 > 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 >> >> PR tree-optimization/51491 >> * tree-ssa-ccp.c (insert_clobber_before_stack_restore): New function. >> (ccp_fold_stmt): Use insert_clobber_befor

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 > fo

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 > 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 below patch applied.

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 > 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 dejagnu i

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 >> 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. > >