Drop mgpuMemHostRegisterMemRef's dependence on LLVM Support. This
method is the only one in CUDA runtime wrappers library that creates
a dependence on libLLVMSupport due to its use of SmallVector and
ArrayRef. The code can be as easily/compactly written without those ADT.
The dependence on LLVMSupport adds a significant amount of additional
complexity for external things that want to link this library in (both
statically or as a shared object) since libLLVMSupport includes numerous
other objects that are sensitive to C++ compiler version and ABI.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks, makes sense to keep the runtime as self contained as possible. Should this also reflect in cmake somewhere?
mlir/lib/ExecutionEngine/CudaRuntimeWrappers.cpp | ||
---|---|---|
14–15 | Is this still needed? |
Comment Actions
I checked the cmake file and I couldn't find anything to remove for mlir_cuda_runtime.
mlir/lib/ExecutionEngine/CudaRuntimeWrappers.cpp | ||
---|---|---|
14–15 | Thanks, removed. |
Is this still needed?