Skip to content
This repository was archived by the owner on Apr 28, 2023. It is now read-only.

Commit 84760e6

Browse files
author
Sven Verdoolaege
committed
Fix and reactivate 1-D reduction tests
Since these (disabled) tests were introduced, the representation of a grid has been changed to be of a type CudaDim which requires at least one dimension.
1 parent 3e6e532 commit 84760e6

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

test/test_tc_mapper.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -139,30 +139,30 @@ TEST_F(TcMapper1DReductionTest, DISABLED_Reduction1Dv0) {
139139
Check(A, mappingOptions, 0);
140140
}
141141

142-
TEST_F(TcMapper1DReductionTest, DISABLED_Reduction1Dv1) {
142+
TEST_F(TcMapper1DReductionTest, Reduction1Dv1) {
143143
auto mappingOptions = tc::CudaMappingOptions::makeNaiveCudaMappingOptions()
144144
.tile(0)
145-
.mapToBlocks({})
145+
.mapToBlocks({1})
146146
.mapToThreads({16});
147147
LOG(INFO) << mappingOptions << endl;
148148
at::Tensor A = at::CUDA(at::kFloat).rand({M});
149149
Check(A, mappingOptions, 1);
150150
}
151151

152-
TEST_F(TcMapper1DReductionTest, DISABLED_Reduction1Dv2) {
152+
TEST_F(TcMapper1DReductionTest, Reduction1Dv2) {
153153
auto mappingOptions = tc::CudaMappingOptions::makeNaiveCudaMappingOptions()
154154
.tile(0)
155-
.mapToBlocks({})
155+
.mapToBlocks({1})
156156
.mapToThreads({16});
157157
LOG(INFO) << mappingOptions << endl;
158158
at::Tensor A = at::CUDA(at::kFloat).rand({M});
159159
Check(A, mappingOptions, 2);
160160
}
161161

162-
TEST_F(TcMapper1DReductionTest, DISABLED_Reduction1Dv3) {
162+
TEST_F(TcMapper1DReductionTest, Reduction1Dv3) {
163163
auto mappingOptions = tc::CudaMappingOptions::makeNaiveCudaMappingOptions()
164164
.tile(0)
165-
.mapToBlocks({})
165+
.mapToBlocks({1})
166166
.mapToThreads({16});
167167
LOG(INFO) << mappingOptions << endl;
168168
at::Tensor A = at::CUDA(at::kFloat).rand({M});

0 commit comments

Comments
 (0)