From a83f7c3ed8d0f956f7ae774bd6919705cdb431ec Mon Sep 17 00:00:00 2001
From: Jeremy Kemp <jeremy@jeremykemp.co.uk>
Date: Thu, 26 Mar 2020 17:56:26 +0000
Subject: [PATCH] c11_atomics: do not overallocate local memory for embedded
 devices #690. (#691)

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.
---
 test_conformance/c11_atomics/test_atomics.cpp | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/test_conformance/c11_atomics/test_atomics.cpp b/test_conformance/c11_atomics/test_atomics.cpp
index e005e9c7..f12f9554 100644
--- a/test_conformance/c11_atomics/test_atomics.cpp
+++ b/test_conformance/c11_atomics/test_atomics.cpp
@@ -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()