Re: [PATCH] Fix for PR52009 - Another missed tail merging opportunity

2012-07-05 Thread Richard Guenther
On Wed, Jul 4, 2012 at 8:07 PM, Tom de Vries wrote: > On 31/01/12 22:07, Tom de Vries wrote: >> On 31/01/12 22:05, Tom de Vries wrote: >>> Richard, >>> >> >> Sorry, with patch this time. >> >>> this patch fixes PR52009. >>> >>> Consider this test-case: >>> ... >>> int z; >>> >>> void >>> foo (int

Re: [PATCH] Fix for PR52009 - Another missed tail merging opportunity

2012-07-04 Thread Tom de Vries
On 31/01/12 22:07, Tom de Vries wrote: > On 31/01/12 22:05, Tom de Vries wrote: >> Richard, >> > > Sorry, with patch this time. > >> this patch fixes PR52009. >> >> Consider this test-case: >> ... >> int z; >> >> void >> foo (int y) >> { >> if (y == 6) >> z = 5; >> else >> z = 5; >> }

Re: [PATCH] Fix for PR52009 - Another missed tail merging opportunity

2012-01-31 Thread Tom de Vries
On 31/01/12 22:05, Tom de Vries wrote: > Richard, > Sorry, with patch this time. > this patch fixes PR52009. > > Consider this test-case: > ... > int z; > > void > foo (int y) > { > if (y == 6) > z = 5; > else > z = 5; > } > ... > > Currently, compiling with -O2 gives us this repr

[PATCH] Fix for PR52009 - Another missed tail merging opportunity

2012-01-31 Thread Tom de Vries
Richard, this patch fixes PR52009. Consider this test-case: ... int z; void foo (int y) { if (y == 6) z = 5; else z = 5; } ... Currently, compiling with -O2 gives us this representation at pr51879-7.c.094t.pre: ... # BLOCK 3 freq:1991 # PRED: 2 [19.9%] (true,exec) # .MEMD.17