-
Notifications
You must be signed in to change notification settings - Fork 10
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
Make NaN != NaN #8
Comments
Not desirable because it would impact performance on all equality checks, because at least a check of the biasedexponent is required. |
It is standardized IEEE 754 behavior. It is not only desirable, but mandatory for compliance, if perfect compliance is the goal. However, it could be an opportunity to fix mistakes of the committee, if that is desired. Edit: I realize I poked a dead issue, but I'm going to leave this here for posterity and informational purposes |
It's an open issue, so I'm glad you commented. I didn't make the change because it seemed to have no support until now. |
No description provided.
The text was updated successfully, but these errors were encountered: