[PATCH] PR tree-optimization/58380

2013-09-10 Thread Jeff Law
58380 is another instance of trying to thread through a buried loop header that was missed by the pruning code. The change to tree-ssa-threadupdate.c catches this case. While looking at this I remembered why I initially didn't allow threading through blocks with a single successor after thr

Re: [PATCH] PR tree-optimization/58380

2013-09-11 Thread Dominique Dhumieres
The test g++.dg/torture/pr58380.C fails: FAIL: g++.dg/torture/pr58380.C -O0 (test for excess errors) FAIL: g++.dg/torture/pr58380.C -O1 (test for excess errors) FAIL: g++.dg/torture/pr58380.C -O2 (test for excess errors) FAIL: g++.dg/torture/pr58380.C -O3 -fomit-frame-pointer (test for exc

Re: [PATCH] PR tree-optimization/58380

2013-09-11 Thread Paolo Carlini
Hi, On 09/11/2013 07:40 PM, domi...@lps.ens.fr wrote: The failures are (at least on x86_64-apple-darwin10): /opt/gcc/work/gcc/testsuite/g++.dg/torture/pr58380.C: In static member function 'static iplugin_factory& selection_to_stdout::get_factory()': /opt/gcc/work/gcc/testsuite/g++.dg/torture/p

Re: [PATCH] PR tree-optimization/58380

2013-09-11 Thread Jeff Law
On 09/11/2013 12:20 PM, Paolo Carlini wrote: Hi, On 09/11/2013 07:40 PM, domi...@lps.ens.fr wrote: The failures are (at least on x86_64-apple-darwin10): /opt/gcc/work/gcc/testsuite/g++.dg/torture/pr58380.C: In static member function 'static iplugin_factory& selection_to_stdout::get_factory()':

Re: [PATCH] PR tree-optimization/58380

2013-09-12 Thread Jeff Law
On 09/12/2013 09:48 AM, Paolo Carlini wrote: On 09/11/2013 09:31 PM, Jeff Law wrote: -w is the right thing to do, warnings or the lack thereof aren't important for what that test is detecting. I'll fix it up momentarily. I went ahead and added the "-w" myself. I hope you don't mind, otherwise,

Re: [PATCH] PR tree-optimization/58380

2013-09-12 Thread Paolo Carlini
On 09/11/2013 09:31 PM, Jeff Law wrote: -w is the right thing to do, warnings or the lack thereof aren't important for what that test is detecting. I'll fix it up momentarily. I went ahead and added the "-w" myself. I hope you don't mind, otherwise, just let me know and I will revert/commit som