-
-
Notifications
You must be signed in to change notification settings - Fork 129
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
enh: add intrinsics with double precision input/output #3844
enh: add intrinsics with double precision input/output #3844
Conversation
This method makes the intrinsics with double precision input/output work but does not throw error if the arguments provided are not double precision. |
{"dexp", "exp"}, | ||
|
||
{"dmin1", "min"}, | ||
{"dmax1", "max"}, | ||
|
||
{"dacos", "acos"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following comment you left, do these function only accept double precision
input? If yes, let's open an issue and add
error not reported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this was intended to make the intrinsics with specific type double precision input/output. However, this works well for intrinsics with input/output as integer like iabs
. Will create an issue explaining the same.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left comments above
403d359
to
29c2164
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good, thanks for doing it.
Towards: #492