-
Notifications
You must be signed in to change notification settings - Fork 359
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
Basics.clamp doesn't passthrough NaN #1080
Comments
Thanks for reporting this! To set expectations:
Finally, please be patient with the core team. They are trying their best with limited resources. |
and because
with both
If you trick the compiler to avoid the optimization, you get
|
If
clamp
is givenNaN
as third argument it should return it unchanged, but instead it returns the given upper limit:I don't understand why this happens.
clamp
is defined as (source)Since both comparisons should be
False
,number
should be returned unchanged, but that doesn't happen.I can't test
gt
myself, but its corresponding>
does returnFalse
as expected:Maybe related to #1050
The text was updated successfully, but these errors were encountered: