Remove unnecessary extension declarations
authorAlexander Galazin <alexander.galazin@arm.com>
Tue, 8 Jan 2019 15:37:27 +0000 (16:37 +0100)
committerAlexander Galazin <alexander.galazin@arm.com>
Tue, 8 Jan 2019 15:42:08 +0000 (16:42 +0100)
Declarations of SPV_AMD_gpu_shader_half_float
in the precision test are redundant and shouldn't appear

Components: Vulkan

Affects: dEQP-VK.glsl.builtin.*.comparison.*

VK-GL-CTS issue: 1541

Change-Id: I75b57cb3361b683e446effa3ab9e51f441f186d5

external/vulkancts/modules/vulkan/shaderexecutor/vktShaderExecutor.cpp

index 3bbdaaf..a5a1570 100644 (file)
@@ -2058,12 +2058,8 @@ std::string generateSpirv(const ShaderSpec& spec, const bool are16Bit, const boo
                src << "OpCapability StorageBuffer16BitAccess\n"
                        "OpCapability UniformAndStorageBuffer16BitAccess\n";
 
-       if (spec.packFloat16Bit || are16Bit)
-               src << "OpExtension \"SPV_AMD_gpu_shader_half_float\"\n";
-
        if (are16Bit)
-               src << "OpExtension \"SPV_AMD_gpu_shader_half_float\"\n"
-                       "OpExtension \"SPV_KHR_16bit_storage\"\n";
+               src << "OpExtension \"SPV_KHR_16bit_storage\"\n";
 
        src << "%1 = OpExtInstImport \"GLSL.std.450\"\n"
                "OpMemoryModel Logical GLSL450\n"