Skip to content
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

limits: Suppress long double device code warning with MSVC #178

Merged
merged 1 commit into from
May 28, 2020

Conversation

stotko
Copy link
Owner

@stotko stotko commented May 28, 2020

The probably last long-standing warning is thrown by MSVC with the CUDA backend enabled stating that long double will be treated as double in device code. This is caused by MSVC's implementation of HUGE_VALL which is expands to (long double)INFINITY. Suppress this warning by using HUGE_VAL for MSVC instead which avoids explicitly casting to long double.

@stotko stotko added the bug label May 28, 2020
@stotko stotko added this to the 1.3.0 milestone May 28, 2020
@codecov
Copy link

codecov bot commented May 28, 2020

Codecov Report

Merging #178 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   97.81%   97.81%           
=======================================
  Files          28       28           
  Lines        1833     1833           
=======================================
  Hits         1793     1793           
  Misses         40       40           
Impacted Files Coverage Δ
src/stdgpu/impl/limits_detail.h 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3caec37...a5e6ba0. Read the comment docs.

@stotko stotko merged commit d005edc into master May 28, 2020
@stotko stotko deleted the limits_msvc_cuda branch May 28, 2020 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant