zhouyuan commented on code in PR #11275:
URL:
https://github.com/apache/incubator-gluten/pull/11275#discussion_r2609113257
##########
cpp/velox/CMakeLists.txt:
##########
@@ -439,6 +439,7 @@ if(ENABLE_GPU)
${VELOX_BUILD_PATH}/_deps/nvtx3-src/c/include
${VELOX_BUILD_PATH}/_deps/nvcomp_proprietary_binary-src/include
${VELOX_BUILD_PATH}/_deps/rapids_logger-src/include
+ /usr/local/cuda/include/cccl
Review Comment:
@bdice thanks for the inputs. Yes, I think we ran into version mismatch for
rapids rmm and cuda. Initially in Gluten we prepared a docker env with
cuda-12.8 pre-installed, everything works and the CMake piece was targeting for
the old env. However with the recent `cudf-25.12` change, It does not compile,
hence I'm experimenting on how to fix this. In my local env, i will need to
bump to use `cuda-13.1` otherwise there will be issues on some header
definition. I also tried `cuda-12.9` and `cuda-13.0` - does not work
```
/__w/incubator-gluten/incubator-gluten/dev/../ep/build-velox/build/velox_ep/_build/release/_deps/rmm-src/cpp/include/rmm/detail/cuda_memory_resource.hpp:23:49:
error: 'synchronous_resource_with' is not a member of 'cuda::mr'
23 | inline constexpr bool resource_with =
cuda::mr::synchronous_resource_with<Resource, Properties...>;
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]