Re: Remove undefined behaviour from builtins-20.c

2015-10-16 Thread Richard Biener
On Thu, Oct 15, 2015 at 3:18 PM, Richard Sandiford wrote: > builtins-20.c had: > > if (cos((y*=2, -fabs(tan(x/-y != cos((y*=2,tan(x/y > link_error (); > > which is undefined behaviour. The test expected that y had the same > value in x/y and x/-y, but gimplification actually impleme

Remove undefined behaviour from builtins-20.c

2015-10-15 Thread Richard Sandiford
builtins-20.c had: if (cos((y*=2, -fabs(tan(x/-y != cos((y*=2,tan(x/y link_error (); which is undefined behaviour. The test expected that y had the same value in x/y and x/-y, but gimplification actually implements the "obvious" interpretation, multiplying y by 2, using it for one