This recommendation is part of the open catalog of best practice rules for performance that is automatically detected and reported by Codee.
Issue
Not calling the most suitable mathematical function for a given data type results in unnecessary data type conversions and higher precision operations (which take more time).
Relevance
In C, there are several versions of the same mathematical function for different types. For example, the square root function is available for floats, doubles and long doubles through sqrtf, sqrt and sqrtl, respectively. Oftentimes, the developer who is not careful will not use the function matching the data type. For instance, most developers will just use “sqrt” for any data type, instead of using sqrtf when the argument is float.
The type mismatch does not cause a compiler error because of the implicit type conversions. However, this practice will result in suboptimal code due to expensive calculations with unnecessary high precision and data type conversions.
Actions
Replace the mathematical function call with its alternative version matching the data type.
Code example
The following code uses the sin function to calculate the sin of the float variable x and store the result in the variable res:
void example(float x) {
float res = sin(x);
}
This causes a conversion of x to a double that is passed to function sin and another of the result which has type double into a float to be stored into the variable res. These two conversion operations can be prevented by using the most appropriate function sinf which matches the float data type:
void example(float x) {
float res = sinf(x);
}
Related resources
References
Building performance into the code from day one with Codee
