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]

Reply via email to