-
Notifications
You must be signed in to change notification settings - Fork 606
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
Fix CUDA 10.0 compilation #2917
Conversation
!build |
CI MESSAGE: [2324287]: BUILD STARTED |
CI MESSAGE: [2324292]: BUILD STARTED |
CI MESSAGE: [2324292]: BUILD FAILED |
- moves isHWDecoderSupported to CUDA 11 ifdef as it uses HasHwDecoder function not available for CUDA 10 Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
ab4ef8a
to
ff91cc4
Compare
CI MESSAGE: [2324541]: BUILD STARTED |
CI MESSAGE: [2324543]: BUILD STARTED |
CI MESSAGE: [2324287]: BUILD FAILED |
CI MESSAGE: [2324543]: BUILD PASSED |
CI MESSAGE: [2324541]: BUILD FAILED |
CI MESSAGE: [2324541]: BUILD PASSED |
@@ -74,7 +74,7 @@ class nvJPEGDecoder : public Operator<MixedBackend>, CachedDecoderImpl { | |||
nvjpeg2k_thread_(1, | |||
spec.GetArgument<int>("device_id"), | |||
spec.GetArgument<bool>("affine")) { | |||
#if NVJPEG_VER_MAJOR >= 11 | |||
#if NVJPEG_VER_MAJOR >= 11 && NVJPEG_VER_MINOR >= 1 |
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.
Technically this is wrong, because for 12.0 the condition would fail.
#if NVJPEG_VER_MAJOR >= 11 && NVJPEG_VER_MINOR >= 1 | |
#if NVJPEG_VER_MAJOR > 11 || (NVJPEG_VER_MAJOR == 11 && NVJPEG_VER_MINOR >= 1) |
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.
Fixed
@@ -558,7 +558,7 @@ class nvJPEGDecoder : public Operator<MixedBackend>, CachedDecoderImpl { | |||
|
|||
// only when we have ROI info check if nvjpegDecodeBatchedSupportedEx supports it | |||
if (nvjpeg_decode) { | |||
#if NVJPEG_VER_MAJOR >= 11 | |||
#if NVJPEG_VER_MAJOR >= 11 && NVJPEG_VER_MINOR >= 1 |
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.
#if NVJPEG_VER_MAJOR >= 11 && NVJPEG_VER_MINOR >= 1 | |
#if NVJPEG_VER_MAJOR > 11 || (NVJPEG_VER_MAJOR == 11 && NVJPEG_VER_MINOR >= 1) |
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.
Fixed
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
CI MESSAGE: [2336625]: BUILD STARTED |
CI MESSAGE: [2336626]: BUILD STARTED |
CI MESSAGE: [2336626]: BUILD FAILED |
CI MESSAGE: [2336625]: BUILD FAILED |
CI MESSAGE: [2336625]: BUILD PASSED |
CI MESSAGE: [2336626]: BUILD PASSED |
function not available for CUDA 10
Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com
Why we need this PR?
Pick one, remove the rest
What happened in this PR?
Fill relevant points, put NA otherwise. Replace anything inside []
moves isHWDecoderSupported to CUDA 11 ifdef as it uses HasHwDecoder function not available for CUDA 10
nvml.h
NA
CI build for CUDA 10.0
NA
JIRA TASK: [NA]