-
Notifications
You must be signed in to change notification settings - Fork 218
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
Use correct intrinsics for rem #1041
Conversation
@vchuravy where should i add some tests? |
Uhm here https://github.com/JuliaGPU/CUDA.jl/blob/master/test/device/intrinsics/math.jl, but the content seems to have gotten lost? |
Let me land #1042 first xD |
Codecov Report
@@ Coverage Diff @@
## master #1041 +/- ##
=======================================
Coverage 79.91% 79.91%
=======================================
Files 117 117
Lines 7629 7629
=======================================
Hits 6097 6097
Misses 1532 1532 Continue to review full report at Codecov.
|
@vchuravy is this okay? |
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 the right thing to do
Thanks! |
Fixes #1040