Re: Split out some tests from builtins-20.c

2015-10-19 Thread Jeff Law
On 10/15/2015 07:18 AM, Richard Sandiford wrote: Stripping unnecessary sign ops at the gimple level means that we're no longer able to optimise: if (cos(y<10 ? -fabs(x) : tan(x<20 ? -x : -fabs(y))) != cos(y<10 ? x : tan(x<20 ? x : y))) link_error (); because we're currently not a

Re: Split out some tests from builtins-20.c

2015-10-15 Thread Mike Stump
On Oct 15, 2015, at 1:38 PM, Richard Sandiford wrote: > Mike Stump writes: >> On Oct 15, 2015, at 12:47 PM, Richard Sandiford >> wrote: >>> I can see that argument if people are only taking work items from >>> the PR database. But it's possible (likely even) that people will >>> independently

Re: Split out some tests from builtins-20.c

2015-10-15 Thread Richard Sandiford
Mike Stump writes: > On Oct 15, 2015, at 12:47 PM, Richard Sandiford > wrote: >> I can see that argument if people are only taking work items from >> the PR database. But it's possible (likely even) that people will >> independently find a problem like this and just fix it, if the missed >> opti

Re: Split out some tests from builtins-20.c

2015-10-15 Thread Mike Stump
On Oct 15, 2015, at 12:47 PM, Richard Sandiford wrote: > I can see that argument if people are only taking work items from > the PR database. But it's possible (likely even) that people will > independently find a problem like this and just fix it, if the missed > optimisation happens to be impo

Re: Split out some tests from builtins-20.c

2015-10-15 Thread Richard Sandiford
Mike Stump writes: > On Oct 15, 2015, at 6:18 AM, Richard Sandiford > wrote: >> Stripping unnecessary sign ops at the gimple level means that we're >> no longer able to optimise: >> >> if (cos(y<10 ? -fabs(x) : tan(x<20 ? -x : -fabs(y))) >> != cos(y<10 ? x : tan(x<20 ? x : y))) >>link_

Re: Split out some tests from builtins-20.c

2015-10-15 Thread Mike Stump
On Oct 15, 2015, at 6:18 AM, Richard Sandiford wrote: > Stripping unnecessary sign ops at the gimple level means that we're > no longer able to optimise: > > if (cos(y<10 ? -fabs(x) : tan(x<20 ? -x : -fabs(y))) > != cos(y<10 ? x : tan(x<20 ? x : y))) >link_error (); > > because we're