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

[FEA] Refactor and strip cudf::column_device_view and cudf::mutable_column_device_view to device-only code #18229

Open
lamarrr opened this issue Mar 11, 2025 · 0 comments · May be fixed by #17968
Labels
feature request New feature or request

Comments

@lamarrr
Copy link
Contributor

lamarrr commented Mar 11, 2025

Is your feature request related to a problem? Please describe.
We can't use the column_device_view.cuh header in JITIFY as it includes host code. This includes code from upstream dependencies like thrust and RMM. Some of the C++ standard headers are also not supported in device code.

Describe the solution you'd like
We want to strip both column_device_view and mutable_column_device_view down to a minimal version that can be included in the JITIFY kernels and then have them build upon the minimal version.
PR #17968 already introduces a jit::mutable_column_device_view that can serve as the foundation for this work.

Describe alternatives you've considered

  • Directly using the headers, this won't work due to JITIFY's inability to include non-device code
@lamarrr lamarrr added the feature request New feature or request label Mar 11, 2025
@lamarrr lamarrr linked a pull request Mar 11, 2025 that will close this issue
3 tasks
@lamarrr lamarrr changed the title [FEA] Refactor and strip cudf::column_device_view and cudf::mutable_column_device_view [FEA] Refactor and strip cudf::column_device_view and cudf::mutable_column_device_view to device-only code Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant