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

[BUG]: nvrtc Excessive error pragma for --expt-relaxed-constexpr #594

Closed
1 task done
kroburg opened this issue Aug 28, 2024 · 4 comments
Closed
1 task done

[BUG]: nvrtc Excessive error pragma for --expt-relaxed-constexpr #594

kroburg opened this issue Aug 28, 2024 · 4 comments
Labels
type: bug Something isn't working

Comments

@kroburg
Copy link

kroburg commented Aug 28, 2024

Is this a duplicate?

Type of Bug

Compile-time Error

Describe the bug

Hello.

I'm trying to use cuco with nvrtc. And it fails with error

catastrophic error: #error directive: "Support for relaxed constexpr is required (nvcc flag --expt-relaxed-constexpr)"
  #error "Support for relaxed constexpr is required (nvcc flag --expt-relaxed-constexpr)"

Please consider thread from jitify. Looks like it is not required for nvrtc.

I'm not sure if I can have success with nvrtc anyway.
I have to include cuco/static_map.cuh to compose static_map_ref for kernel params like that: decltype(((const map_t*)nullptr)->ref(cuco::find));
This is because I don't known how to craft StorageRef template argument.

From my expirience including such a "wide" header as cuco/static_map.cuh will not work with nvrtc because of host api calls.

How to Reproduce

NA

Expected behavior

cuco compiles using nvrtc.

Reproduction link

No response

Operating System

No response

nvidia-smi output

No response

NVCC version

NA

@kroburg kroburg added the type: bug Something isn't working label Aug 28, 2024
@kroburg
Copy link
Author

kroburg commented Aug 28, 2024

Well, I got it. It will not work anyway :-/

/tmp/sis_cuda_package.tmp.0caa4e15e025276df64cd21d90ac80e4/cccl/thrust/thrust/detail/execute_with_dependencies.h(63): error: A function explicitly marked as a __host__ function is not allowed in JIT mode
    _CCCL_HOST execute_with_dependencies(super_t const& super, Dependencies&&... deps)
           ^

@PointKernel
Copy link
Member

PointKernel commented Aug 30, 2024

#595 improves the situation by disabling the relaxed constexpr requirements in cuco

@kroburg
Copy link
Author

kroburg commented Aug 31, 2024

Great! Thank you.

@PointKernel
Copy link
Member

Closing as resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants