Vulkan: Add GL_EXT_separate_shader_objects support

src/libANGLE/renderer/vulkan/vk_caps_utils.cpp

@@ -345,6 +345,9 @@ void RendererVk::ensureCapsInitialized() const
     // Implemented in the translator
     mNativeExtensions.shaderNonConstGlobalInitializersEXT = true;
 
+    // Implemented in the front end
+    mNativeExtensions.separateShaderObjects = true;
+
     // Vulkan has no restrictions of the format of cubemaps, so if the proper formats are supported,
     // creating a cube of any of these formats should be implicitly supported.
     mNativeExtensions.depthTextureCubeMapOES =

src/libANGLE/validationES3.cpp

@@ -2014,17 +2014,11 @@ bool ValidateGetProgramBinary(const Context *context,
     return ValidateGetProgramBinaryBase(context, program, bufSize, length, binaryFormat, binary);
 }
 
-bool ValidateProgramParameteri(const Context *context,
-                               ShaderProgramID program,
-                               GLenum pname,
-                               GLint value)
+bool ValidateProgramParameteriBase(const Context *context,
+                                   ShaderProgramID program,
+                                   GLenum pname,
+                                   GLint value)
 {
-    if (context->getClientMajorVersion() < 3)
-    {
-        context->validationError(GL_INVALID_OPERATION, kES3Required);
-        return false;
-    }
-
     if (GetValidProgram(context, program) == nullptr)
     {
         return false;
@@ -2062,6 +2056,20 @@ bool ValidateProgramParameteri(const Context *context,
     return true;
 }
 
+bool ValidateProgramParameteri(const Context *context,
+                               ShaderProgramID program,
+                               GLenum pname,
+                               GLint value)
+{
+    if (context->getClientMajorVersion() < 3)
+    {
+        context->validationError(GL_INVALID_OPERATION, kES3Required);
+        return false;
+    }
+
+    return ValidateProgramParameteriBase(context, program, pname, value);
+}
+
 bool ValidateBlitFramebuffer(const Context *context,
                              GLint srcX0,
                              GLint srcY0,

src/libANGLE/validationES3.h

@@ -31,6 +31,11 @@ bool ValidateES3TexImageParametersBase(const Context *context,
                                        GLenum type,
                                        GLsizei imageSize,
                                        const void *pixels);
+
+bool ValidateProgramParameteriBase(const Context *context,
+                                   ShaderProgramID program,
+                                   GLenum pname,
+                                   GLint value);
 }  // namespace gl
 
 #endif  // LIBANGLE_VALIDATION_ES3_H_

src/tests/gl_tests/ProgramPipelineTest.cpp

@@ -49,6 +49,24 @@ TEST_P(ProgramPipelineTest, GenerateProgramPipelineObject)
     }
 }
 
+// Verify that program pipeline errors out without GL_EXT_separate_shader_objects extension.
+TEST_P(ProgramPipelineTest, GenerateProgramPipelineObjectEXT)
+{
+    GLuint pipeline;
+    glGenProgramPipelinesEXT(1, &pipeline);
+    if (!IsGLExtensionEnabled("GL_EXT_separate_shader_objects"))
+    {
+        EXPECT_GL_ERROR(GL_INVALID_OPERATION);
+    }
+    else
+    {
+        EXPECT_GL_NO_ERROR();
+
+        glDeleteProgramPipelinesEXT(1, &pipeline);
+        EXPECT_GL_NO_ERROR();
+    }
+}
+
 class ProgramPipelineTest31 : public ProgramPipelineTest
 {
   protected: