Fix amber vs cts device feature requirements
authorJari Komppa <jari.komppa@siru.fi>
Mon, 15 Nov 2021 08:25:36 +0000 (10:25 +0200)
committerMatthew Netsch <quic_mnetsch@quicinc.com>
Mon, 6 Dec 2021 22:58:12 +0000 (22:58 +0000)
This change unifies the device requirements between cts and amber for
the affected tests.

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

Components: Vulkan
VK-GL-CTS issue: 3304

Change-Id: I93f840751b40d145f89f847612e36733a18ac971

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

index 3a94c69..95b1bfb 100644 (file)
@@ -6884,12 +6884,18 @@ public:
 
                                        if (ctx.isPackFloat16b == true)
                                        {
-                                               requirements.push_back("Storage16BitFeatures.storageBuffer16BitAccess");
                                                fileName += "_32bit";
                                        }
+                                       else
+                                       {
+                                               requirements.push_back("Storage16BitFeatures.storageBuffer16BitAccess");
+                                       }
                                }
                        }
 
+                       requirements.push_back("VK_KHR_16bit_storage");
+                       requirements.push_back("VK_KHR_storage_buffer_storage_class");
+
                        group->addChild(cts_amber::createAmberTestCase(ctx.testContext, "mat3", "Square matrix 3x3 precision tests", dataDir, fileName + "_mat_3x3.amber", requirements));
                        group->addChild(cts_amber::createAmberTestCase(ctx.testContext, "mat4", "Square matrix 4x4 precision tests", dataDir, fileName + "_mat_4x4.amber", requirements));
                }