[PATCHv2] New generic sinf

[Joseph Myers]

On Thu, 2 Nov 2017, Rajalakshmi Srinivasaraghavan wrote:

> BZ#5997 reported sinf performance on x86_64 and sinf had
> already been optimized for x86_64. The proposed patch is
> generic version which now improves performance on other
> arch. So I think 5997 can be marked as fixed.

In my view, bug 5997 reported performance issues with the generic 
implementation and it simply happened that the reporter was using x86_64.  
Bug reports should always be understood globally in that sense: changing 
the implementation used on one platform is not a sufficient fix if the 
problem implementation remains used on other platforms.  (In the case of 
correctness reports for libm functions it can generally be useful to 
understand them to apply to all floating-point formats as well, at least 
where the implementations are similar, but that doesn't really make sense 
for performance reports that are very specific to the characteristics of a 
particular implementation.)

-- 
Joseph S. Myers
joseph@codesourcery.com