Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

performance test for fast_trigonometry #265

Closed
wants to merge 7 commits into from

Conversation

Nojan
Copy link

@Nojan Nojan commented Nov 5, 2014

Does that kind of performance test work for you? Can I use the same pattern for the other functions?

@Groovounet
Copy link
Member

Yeap, looks great!

Thanks for contributing!
Christophe

@Nojan
Copy link
Author

Nojan commented Nov 5, 2014

Done.
Result:
fastCos Time 41704 clocks
cos Time 41017 clocks
fastSin Time 40980 clocks
sin Time 41223 clocks
fastTan Time 41201 clocks
tan Time 40867 clocks
fastAcos Time 40795 clocks
acos Time 40804 clocks
fastAsin Time 40723 clocks
asin Time 40713 clocks
fastAtan Time 40706 clocks
atan Time 40863 clocks

@Groovounet
Copy link
Member

Erm, yeah I was afraid that performance wouldn't be really good with that much precision.

At least that gives a reliable base to work with.

@Nojan
Copy link
Author

Nojan commented Nov 5, 2014

I tried with the fastCos and fastSin from my branch (which is not included in this pull request), it's about the same:
fastCos Time 40699 clocks
cos Time 40766 clocks
fastSin Time 40712 clocks
sin Time 40722 clocks

Groovounet pushed a commit that referenced this pull request Nov 14, 2014
@Groovounet
Copy link
Member

I manually pull this request to the master branch.

Thanks for contributing,
Christophe

@Groovounet Groovounet closed this Nov 14, 2014
@Groovounet Groovounet added this to the GLM 0.9.6 milestone Nov 14, 2014
@Groovounet Groovounet self-assigned this Nov 14, 2014
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants