[PATCH] Speed up math/test-tgmath2.c

[Steve Ellcey]

On Fri, 2016-11-04 at 17:25 +0000, Joseph Myers wrote:
> On Fri, 4 Nov 2016, Steve Ellcey wrote:
> 
> > 
> > > 
> > > +Â Â return test_cos (vint1, vllong1)
> > > +	Â + test_fabs (vint1, vllong1)
> > > +	Â + test_conj (vint1, vllong1)
> > > +	Â + test_expm1 (vint1, vllong1)
> > > +	Â + test_lrint (vint1, vllong1)
> > > +	Â + test_ldexp (vint1, vllong1)
> > > +	Â + test_atan2 (vint1, vllong1)
> > > +	Â + test_remquo (vint1, vllong1)
> > > +	Â + test_pow (vint1, vllong1)
> > > +	Â + test_fma_1 (vint1, vllong1)
> > > +	Â + test_fma_2 (vint1, vllong1)
> > > +	Â + test_fma_3 (vint1, vllong1)
> > > +	Â + test_fma_4 (vint1, vllong1);
> You don't want to add up test results like that; you should OR them 
> together instead, to avoid the potential for test results adding up
> to 77Â 
> and wrongly becoming UNSUPPORTED.  (In this case of course there
> aren't 
> enough tests to get to 77, but the principle applies that you should
> avoid 
> the test return being a count of something.)

Here is a modified and retested patch.

Steve Ellcey
sellcey@caviumnetworks.com


2016-11-04  Steve Ellcey  <sellcey@caviumnetworks.com>

	* math/test-tgmath2.c: Split up test function.




-------------- next part --------------
A non-text attachment was scrubbed...
Name: tgmath2.patch
Type: text/x-patch
Size: 5399 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20161104/369f402e/attachment.bin>