Overloading functions to allow custom allocators#56
Overloading functions to allow custom allocators#56MikaelHuppunen wants to merge 3 commits intokstppd:masterfrom
Conversation
|
@MikaelHuppunen this looks good and passes the CI test locally on my machine. However would you still want to add a small unit test somewhere? |
| split::tools::loop_compact<<<1, BLOCKSIZE, 0, s>>>(input, output, rule); | ||
| } | ||
|
|
||
| template <typename T, typename U, typename Rule, typename ALLOCATOR, size_t BLOCKSIZE = 1024, size_t WARP = WARPLENGTH> |
There was a problem hiding this comment.
same question for all these pretty much
There was a problem hiding this comment.
The functions would then break if ALLOCATOR is defined but BLOCKSIZE and WARP are not (or the other way around depending on the order of the arguments).
Is there an existing unit test that tests extractPatternLoop and extractAllKeysLoop? I could then do that but with a custom allocator. |
@MikaelHuppunen yup there are a couple of test cases here |
|
This is the diff on the meson file diff --git a/meson.build b/meson.build
index 5c10eb6..fd37248 100644
--- a/meson.build
+++ b/meson.build
@@ -30,6 +30,7 @@ insertion_mechanism = executable('insertion', 'unit_tests/insertion_mechanism/ma
tombstoneTest = executable('tbPerf', 'unit_tests/benchmark/tbPerf.cu', dependencies :gtest_dep)
realisticTest = executable('realistic', 'unit_tests/benchmark/realistic.cu', dependencies :gtest_dep)
hybridGPU = executable('hybrid_gpu', 'unit_tests/hybrid/main.cu',dependencies :gtest_dep )
+customAllocatorTest = executable('custom_allocator', 'unit_tests/custom_allocator/unit.cu',dependencies :gtest_dep )
#Test-Runner
@@ -48,3 +49,4 @@ test('hybridCPU_Test', hybridCPU)
test('hybridGPU_Test', hybridGPU)
test('TbTest', tombstoneTest)
test('RealisticTest', realisticTest)
+test('CustomAllocator', customAllocatorTest)in an attempt to integrate your unit test there but fails to compile due to |
No description provided.