ahmed/OpenCL-CTS
1
0
Fork 0
mirror of https://github.com/KhronosGroup/OpenCL-CTS.git synced 2026-03-19 06:09:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

c11_atomics: do not overallocate local memory for embedded devices #690. (#691)

Browse Source 
The spec states that the minimum amount of local memory for embedded devices is 1KB. This change clamps work group sizes to 1024 for embedded devices, and sets the number of local variables per thread to 1.

Fixes #690.
This commit is contained in:
Jeremy Kemp
2020-03-26 17:56:26 +00:00
committed by GitHub
parent f38ed4c0b3
commit a83f7c3ed8
1 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
test_conformance/c11_atomics/test_atomics.cpp
View File

@@ -1761,7 +1761,16 @@ public:
if (MemoryOrder() == MEMORY_ORDER_SEQ_CST)
return 1;
if (LocalMemory())
return 32 * 1024 / 8 / CurrentGroupSize() - 1; //32KB of local memory required by spec
{
if (gIsEmbedded)
{
if (CurrentGroupSize() > 1024)
CurrentGroupSize(1024);
return 1; //1KB of local memory required by spec. Clamp group size to 1k and allow 1 variable per thread
}
else
return 32 * 1024 / 8 / CurrentGroupSize() - 1; //32KB of local memory required by spec
}
return 256;
}
virtual std::string SingleTestName()