Bug 531915 Comment 27 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to sanketh from comment #23)
> (In reply to Tooru Fujisawa [:arai] from comment #22)
> > Have you checked the performance impact by switching to fdlibm sin/cos/tan? (see bug 933257 comment #139 and bug 933257 comment #159 for the previous performance impact)
> 
> We did a few basic experiments early on, like we saw a 10% slowdown on the box2d benchmark in Jetstream. But, now that these patches are beginning to take shape, we are going to do more benchmarking.
> 
> Thanks for the references to the comments, I will keep the ni to redo those experiments and post the data here.

We did more experiments and the data is included in the email to dev-platform [0]. Following the discussion there and offline, we might do more experiments, and we'll post the details here when we have it.

[0]: https://groups.google.com/a/mozilla.org/g/dev-platform/c/0dxAO-JsoXI/m/eEhjM9VsAgAJ
(In reply to sanketh from comment #23)
> (In reply to Tooru Fujisawa [:arai] from comment #22)
> > Have you checked the performance impact by switching to fdlibm sin/cos/tan? (see bug 933257 comment #139 and bug 933257 comment #159 for the previous performance impact)
> 
> We did a few basic experiments early on, like we saw a 10% slowdown on the box2d benchmark in Jetstream. But, now that these patches are beginning to take shape, we are going to do more benchmarking.
> 
> Thanks for the references to the comments, I will keep the ni to redo those experiments and post the data here.

We did more experiments and the data is included in the [email to dev-platform](https://groups.google.com/a/mozilla.org/g/dev-platform/c/0dxAO-JsoXI/m/eEhjM9VsAgAJ). Following the discussion there and offline, we might do more experiments, and we'll post the details here when we have it.

Back to Bug 531915 Comment 27