Skip to content

Commit 38d1813

Browse files
committed
[Project] Minor fixes for CI
1 parent 8ac77be commit 38d1813

File tree

4 files changed

+3
-3
lines changed

4 files changed

+3
-3
lines changed

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ project(cubool LANGUAGES CXX)
77
# Exposed to the user build options
88
option(CUBOOL_WITH_CUDA "Build library with cuda backend (default)" ON)
99
option(CUBOOL_WITH_SEQUENTIAL "Build library with cpu sequential backend (fallback)" ON)
10-
option(CUBOOL_WITH_NAIVE "Build library with naive and naive-shared dense matrix multiplication" ON)
10+
option(CUBOOL_WITH_NAIVE "Build library with naive and naive-shared dense matrix multiplication" OFF)
1111
option(CUBOOL_BUILD_TESTS "Build project unit-tests with gtest" ON)
1212

1313
set(CUBOOL_VERSION_MAJOR 1)

cubool/sources/cuda/details/device_allocator.cuh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ namespace cubool {
6161
template <typename U>
6262
__host__ explicit DeviceAllocator(const DeviceAllocator<U> &other) : mInstanceRef(other.mInstance) { }
6363

64-
__host__ ~DeviceAllocator() = default;
64+
~DeviceAllocator() = default;
6565

6666
__host__ bool operator!=(const DeviceAllocator<T> &other) const {
6767
return &mInstanceRef != &other.mInstanceRef;

cubool/tests/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ target_link_libraries(test_matrix_kronecker PUBLIC testing)
3232
add_executable(test_matrix_ewiseadd test_matrix_ewiseadd.cpp)
3333
target_link_libraries(test_matrix_ewiseadd PUBLIC testing)
3434

35-
if (CUBOOL_WITH_NAIVE)
35+
if (CUBOOL_WITH_TESTS_NAIVE)
3636
add_executable(test_matrix_dense_vs_naive test_matrix_dense_vs_naive.cpp)
3737
target_link_libraries(test_matrix_dense_vs_naive PUBLIC testing)
3838
target_link_libraries(test_matrix_dense_vs_naive PUBLIC naive_gpu)
File renamed without changes.

0 commit comments

Comments
 (0)