Emit memory qualifier once for GLSL targets (#4819)

Fixes #4818

Emit memory qualifier once for GLSL targets.

Co-authored-by: Yong He <yonghe@outlook.com>

3 files changed, 22 insertions, 1 deletions

diff --git a/source/slang/slang-emit-glsl.cpp b/source/slang/slang-emit-glsl.cpp
index 0fb868342..86ad0be6e 100644
--- a/source/slang/slang-emit-glsl.cpp
+++ b/source/slang/slang-emit-glsl.cpp
@@ -228,7 +228,6 @@ void GLSLSourceEmitter::_emitGLSLStructuredBuffer(IRGlobalParam* varDecl, IRHLSL
     HLSLAppendStructuredBufferType                  - Write
     HLSLConsumeStructuredBufferType                 - TODO (JS): Its possible that this can be readonly, but we currently don't support on GLSL
     */
-    _emitMemoryQualifierDecorations(varDecl);
     if (as<IRHLSLStructuredBufferType>(structuredBufferType))
     {
         m_writer->emit("readonly ");
diff --git a/tests/bugs/gh-4818-1.slang b/tests/bugs/gh-4818-1.slang
new file mode 100644
index 000000000..4832bb5fd
--- /dev/null
+++ b/tests/bugs/gh-4818-1.slang
@@ -0,0 +1,11 @@
+//TEST:SIMPLE(filecheck=CHECK): -target glsl -entry computeMain -stage compute
+
+// CHECK: coherent
+// CHECK-NOT: coherent
+
+globallycoherent RWByteAddressBuffer bab;
+RWStructuredBuffer<uint> output;
+void computeMain()
+{
+    output[0] = bab.Load<uint>(0);
+}
\ No newline at end of file
diff --git a/tests/bugs/gh-4818-2.slang b/tests/bugs/gh-4818-2.slang
new file mode 100644
index 000000000..e2a582434
--- /dev/null
+++ b/tests/bugs/gh-4818-2.slang
@@ -0,0 +1,11 @@
+//TEST:SIMPLE(filecheck=CHECK): -target glsl -entry computeMain -stage compute
+
+// CHECK: coherent
+// CHECK-NOT: coherent
+
+globallycoherent StructuredBuffer<uint> sb;
+RWStructuredBuffer<uint> output;
+void computeMain()
+{
+    output[0] = sb.Load(0);
+}
\ No newline at end of file