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

Commit 43aa4ee

Browse files
authored
Merge pull request #301 from facebookresearch/fix-memory-promotion-order-in-test
do not promote to registers before promoting to shared in tests
2 parents dfed608 + 2a1caa1 commit 43aa4ee

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/test_cuda_mapper_memory_promotion.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ class MapperMemoryPromotion2DHelper : public TestMapper {
6666
CudaMappingOptions::makeNaiveCudaMappingOptions()
6767
.tile(tileSizes) // passing more tiling values triggers a check...
6868
.useSharedMemory(false) // do not auto-promote
69-
.usePrivateMemory(true);
69+
.usePrivateMemory(false);
7070
return makeMappedScop(tc, mappingOptions, problemSizes);
7171
}
7272
};
@@ -86,7 +86,7 @@ def fun(float(N,M,K,L) A, float(N,M,K,L) B) -> (C) {
8686
auto mappingOptions = CudaMappingOptions::makeNaiveCudaMappingOptions()
8787
.tile(tileSizes)
8888
.useSharedMemory(false) // do not autopromote
89-
.usePrivateMemory(true);
89+
.usePrivateMemory(false);
9090
auto mscop = makeMappedScop(
9191
tc,
9292
mappingOptions,

0 commit comments

Comments
 (0)