Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

refactor: replaces the math functions with float based function from the standard namespace #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deepsource-autofix[bot]
Copy link

Calling math functions from the C libray that only accept double with float arguments causes an implicit type promotion of the float argument. Type promoting float to double costs extra space in memory, it also costs extra instructions for the conversion from float and lastly vectorisation of float is a lot more efficient compared to double.

…m the standard namespace

Calling math functions from the C libray that only accept `double` with `float` arguments causes an implicit type promotion of the `float` argument. Type promoting `float` to `double` costs extra space in memory, it also costs extra instructions for the conversion from `float` and lastly vectorisation of `float` is a lot more efficient compared to `double`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants