Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I557cfbf99b4603ec546a3443bd2f5334475980dc

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: I00246ec185fe843f4ca638ca0a4bfa34b8598cde

Fixes multiview query tests

Queries need to be reset prior to use

Components: Vulkan
VK-GL-CTS issue: 1575

Affects:
dEQP-VK.multiview.queries.*

Change-Id: Icf0f712a3391784ee47c00de833f8417bb8337c3
(cherry picked from commit bd913ffd046955e976337ab6fb1e93520b37c375)

Fixes multiview query tests

Queries need to be reset prior to use

Components: Vulkan
VK-GL-CTS issue: 1575

Affects:
dEQP-VK.multiview.queries.*

Change-Id: Icf0f712a3391784ee47c00de833f8417bb8337c3

Fix dependency on unspecified rounding mode

These tests perform an addition whose Rounding mode is unspecified
according to the specification.  This test is not testing addition,
it is testing that the input values are read correctly, so modify
the input values such that they do not require rounding.

Affected tests:
* dEQP-VK.spirv_assembly.instruction.compute.indexing.input.non16basealignment*

Components: Vulkan

VK-GL-CTS issue: 1473
Change-Id: Ifab284d95cc2e36c3a551c30d57deeca7bd8d4b2

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I90db06a96453f6f1ba2095858d490715b12b3b1d

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: Ia670fa92c0d87e0c01f07e156a00d19533faf9f0

Merge vk-gl-cts/vulkan-cts-1.1.1 into vk-gl-cts/vulkan-cts-1.1.2

Change-Id: Ic7e43b929357fc3e076de1105d7e24fb3b92b946

Merge vk-gl-cts/vulkan-cts-1.1.0 into vk-gl-cts/vulkan-cts-1.1.1

Change-Id: I79cdbe3b49745c53437cd85f3022bf9c6123211d

Update vkrunner version

Includes fixes for missing synchronization.

VK-GL-CTS Issue: 1567

Change-Id: I21e8cf172e7bd45d54c0b8e0ef9e2776a85e691c

Add const matrix multiply test

Test case ported from an ANGLE regression test.

New tests:

dEQP-GLES3.functional.shaders.constants.const_mat_multiply*

Components: AOSP

Chromium issue: 912508

Change-Id: I5ecc13af8f8e7a5d5a50126ea609e8ad6fab185b

Add UTF8 abuse tests for extension and layer names

These tests abuse the layer and extension name fields of the instance
creation APIs. The APIs are defined as taking in UTF-8 strings but
all actual strings so far are ASCII, so it is possible that these
inputs are unexpected and may lead to crashes.

New tests:

dEQP-VK.api.device_init.create_instance_extension_name_abuse
dEQP-VK.api.device_init.create_instance_layer_name_abuse

Components: Vulkan

VK-GL-CTS issue: 1510

Change-Id: Iaf370dd737ca8ce8c4fda849a95b951fa12d2ab5

Add graphicsfuzz vkrunner tests

Add a new category for simplified test cases found by GraphicsFuzz that
trigger bugs in widely-used shader compilers so that the bugs will be
fixed and will stay fixed. The test cases do not try to get coverage of
specific features; the tests usually trigger very specific and subtle
bugs and may look strange without this context. Nevertheless, the tests
should clearly pass.

Add one test case. Other test cases to follow.

Components: Vulkan

New Tests: dEQP-VK.graphicsfuzz.pow-vec4

Change-Id: I518cc24b5649e662ee383b36dbe05b55153325c4

Fix missing self dependency in renderpass tests

The dEQP-VK.renderpass.suballocation.formats and
dEQP-VK.renderpass.dedicated_allocation.formats tests were missing self
dependencies in some cases where they read then write one attachment
in one subpass. In cases with overlapping rendering this would give a
read-write-then-read-write situation, which neads a self dependency to
work reliably. Some of the tests were missing the self dependency, some
of them had the color attachment bits set when depending on depth, and
one case had a self dependency where it should not.
This patch adds a self dependency to two test cases that need it,
removes self dependency from a case that does not need it, and fixes
the depth tests to have the correct bits set.

Affects:

dEQP-VK.renderpass.suballocation.formats.*.input.*
dEQP-VK.renderpass.dedicated_allocation.formats.*.input.*
dEQP-VK.renderpass2.suballocation.formats.*.input.*
dEQP-VK.renderpass2.dedicated_allocation.formats.*.input.*

Components: Vulkan

VK-GL-CTS Issue: 1454

Change-Id: I4a9bec3ef6b088ab3a50d7f658867076f68788b9
(cherry picked from commit f659f8c9ad39c179a79a92ee3b66cb83e1dd52ed)

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I3eef5f481cfa4bf5c8bcb0feb69733d492de95d3

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: Idec176bfe0b3a95eb0c557f32015968cbeb914d7

Merge vk-gl-cts/vulkan-cts-1.1.1 into vk-gl-cts/vulkan-cts-1.1.2

Change-Id: I2b85e569622052ec742b1272ac367fc1dccf1733

Merge vk-gl-cts/vulkan-cts-1.1.0 into vk-gl-cts/vulkan-cts-1.1.1

Change-Id: Iaf773ccd948a5707df4d2ae4628a702e739bd8df

Merge vk-gl-cts/vulkan-cts-1.0.2 into vk-gl-cts/vulkan-cts-1.1.0

Change-Id: Idf9225adfcbc231e544516558bd6d794dce40b26

Fix NaN handling in funord tests

Support of NaNs is optional according to the specification, but
these tests were assuming that NaNs are always supported.
Modify these tests to skip check if an operand is a NaN.

Add infrastructure for a new set of tests opfunord_nan*, which
use VK_KHR_shader_float_controls to check whether NaNs are supported.
But since this is a bugfix, do not actually add the new tests.

Affected tests:
* dEQP-VK.spirv_assembly.instruction.compute.opfunord*

Components: Vulkan

VK-GL-CTS issue: 1551
Change-Id: Ic60f13377ff9d2bdc2e10047aab8be0617904670

Fix validation failure in float_controls

Half of float_controls tests read operation arguments from
SSBO (other half generates them in the code). Whent vertex
stage is tested this SSBO must be decorated as NonWritable.
This change resolves this issue by applying NonWritable
decoration to input SSBO for all tested stages.
Operation output is always saved in fragment stage and to
separate SSBO.

Components: Vulkan

VK-GL-CTS issue: 1543

Affects:
dEQP-VK.spirv_assembly.instruction.compute.float_controls.*
dEQP-VK.spirv_assembly.instruction.graphics.float_controls.*

Change-Id: Iaed891d7b45b2a5f02ce1443dc13a40e24fa2086

Update Vulkan CTS Readme

Clarify API level requirements for implementations with
VK_ANDROID_external_memory_android_hardware_buffer support

Components: Vulkan

Change-Id: I8536595678a8d4daff9e16ed3a9e9e25594e372e

Fixes ycbcr image properties requirement

Mipmaps/Arrays are optional for ycbcr images

Components: Vulkan
VK-GL-CTS issue: 1506

Affects:
dEQP-VK.api.info.image_format_properties.*

Change-Id: I795ec1c8b3e44a460ddd0c7c2fe0fe1731794cab
(cherry picked from commit 845394de5b43c71adddc72e7563b50c65e852b27)

Fixes ycbcr image properties requirement

Mipmaps/Arrays are optional for ycbcr images

Components: Vulkan
VK-GL-CTS issue: 1506

Affects:
dEQP-VK.api.info.image_format_properties.*

Change-Id: I795ec1c8b3e44a460ddd0c7c2fe0fe1731794cab

Fix image type in Uniform texel buffer tests

Form the issue:
The Shader Resource and Storage Class Correspondence table
in the Descriptor Set Interface section of the Vulkan 1.1
specification clearly states that a descriptor resource
which is a uniform texel buffer should correspond to
an OpTypeImage(Dim=Buffer,Sampled=1) type.

The current tests in the binding_model category
generate OpTypeSampledImage for uniform texel buffer
resources instead, which I believe is incorrect.

This can be fixed by changing the GLSL declaration from
samplerBuffer to textureBuffer

Components: Vulkan

VK-GL-CTS public issue: 139

Affects:
dEQP-VK.binding_model.*uniform_texel_buffer.*

Change-Id: Ie1beb05cac69a6cf1220bbd47c2d7876eed7ff26

Protected Memory Fixes - Barrier Layouts & Flags

This change removes errors reported by core validation layer.

Affects: dEQP-VK.protected_memory.*

Components: Vulkan

Change-Id: Ic258ba2af077b2dc119db0d177613558464e0c9a

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I46729dcd60ae4c533dabacb17fbeecc92f709bb0

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: Ibfebb592de5aff940b3167f5475139cec99ef6f5

Update VK_KHR_vulkan_memory_model to spec version 3

Components: Vulkan
Affects: dEQP-VK.memory_model.*
Change-Id: I3ba8a275092f0a2fcfef02cdabf2ab51750f74f5

Fixes depth load_op in multiview tests

Depth load_op was set to dont care before drawing with depth test.

Components: Vulkan
VK-GL-CTS issue: 1557

Affects:
dEQP-VK.multiview*.depth.*

Change-Id: Ia853a84fa7c96b641a2dd91075d015a90a372d35

Truncate random coordinates in late fragment tests

The late-fragment tests generate random triangles.  Unfortunately,
the reference rasteriser does not respect the sub-pixel precision
of the physical device (it assumes 8 bits) which means that on
some platforms the reference rasterisation does not agree with
implementation.

This patch rounds the random coordinates to the reported
sub-pixel precision.  It was decided that in the short term this
was a less invasive change than modifying the reference rasteriser.

Affects: dEQP-VK.renderpass*.subpass_dependencies.late_fragment_tests.*

Components: Vulkan

VK-GL-CTS issue: 1549

Change-Id: I50f20284d9d2cb22e73564d6f71c0a3342547b68

Reenable coverage of opaque type indexing

These tests appear to have working code for testing various scenarios
that was disabled. This adds coverage of 1D samplers, non-constant SSBO
indexing, and SSBO indexing in tessellation and geometry stages.

Components: Vulkan
VK-GL-CTS Issue: 1544
New Tests: dEQP-VK.glsl.opaque_type_indexing.sampler.*.{i,u,}sampler1d
           dEQP-VK.glsl.opaque_type_indexing.sampler.*.{i,u,}sampler1darray
           dEQP-VK.glsl.opaque_type_indexing.sampler.*.sampler1dshadow
           dEQP-VK.glsl.opaque_type_indexing.sampler.*.sampler1darrayshadow
           dEQP-VK.glsl.opaque_type_indexing.ssbo.uniform_*
           dEQP-VK.glsl.opaque_type_indexing.ssbo.dynamically_uniform_*
           dEQP-VK.glsl.opaque_type_indexing.ssbo_storage_buffer_decoration.uniform_*
           dEQP-VK.glsl.opaque_type_indexing.ssbo_storage_buffer_decoration.dynamically_uniform_*
           dEQP-VK.glsl.opaque_type_indexing.ubo.*_geometry
           dEQP-VK.glsl.opaque_type_indexing.ubo.*_tess_ctrl
           dEQP-VK.glsl.opaque_type_indexing.ubo.*_tess_eval
           dEQP-VK.glsl.opaque_type_indexing.ssbo.*_geometry
           dEQP-VK.glsl.opaque_type_indexing.ssbo.*_tess_ctrl
           dEQP-VK.glsl.opaque_type_indexing.ssbo.*_tess_eval

Change-Id: I16ed787be3057b1d77c89d9684472f1e74a3b870

Test swizzles of 2D texture lookup coordinates

Add new tests that verify 2D textures with given lookup coordinate
swizzles are rendered correctly.

New tests:
dEQP-VK.texture.swizzle.texture_coordinate.*

Affects:
dEQP-VK.texture.swizzle.*

Components: Vulkan

VK-GL-CTS issue: 1212

Change-Id: I8d1a3ef060542232ffa016342728d9b5fb01902a

Touch watchdog more often for precision tests.

Have a test that takes ~2.5 minutes here using ARM
emulation:

01-15 17:58:54.903  3909  3926 I dEQP    : Test case 'dEQP-VK.glsl.builtin.precision.reflect.mediump_compute.vec4'..
01-15 18:01:28.631  3909  3926 I dEQP    :   Pass (Pass)

Since this is still below the overall 300 seconds limit
for a test I thought just increasing the watchdog touch
frequency would be okay.

We probably strictly need about 1000, but for leaving
some time before/after this for the driver to do stuff
(like compiling a shader), I took something lower.

Affected tests:
dEQP-VK.glsl.builtin.precision.*

Component: Vulkan

VK-GL-CTS issue: 1560

Change-Id: I450ddac39635c857ee1966d86ae16085082d242b

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I7ca6e37c293b217b4e95f475e171e142c33ac181

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: I99ab3d97b302f2b9d82ec0ce8a05e716d1aec54a

Added PointSize in shader used for drawing points

Follow-up change of 76a3efb8583de58358570c912e4388a780a2cb39.

A SPIR-V assembly shader with undefined PointSize was being used with
point list topology.

Affects:

dEQP-VK.subgroups.*

Components: Vulkan

VK-GL-CTS issue: 1197,1552

Change-Id: I10404d8fe45b9fb7484d1b90b984e66ca8569de4

Add AHB to Android allowed extensions list

Affects: dEQP-VK.api.info.android.no_unknown_extensions
VK-GL-CTS Issue: 1559
Components: Vulkan
Google Bug: b/120907290

Change-Id: Id1f2d63cbd51e7b3e51106f8f0214cd8d95c199c

Fix logging of 8-bit integer values

The values were 'streamed' using << to std::stream, so they
were treated as characters, instead of integer values.

Components: Vulkan

VK-GL-CTS Issue: 1505

Affects:
dEQP-VK.spirv_assembly.type.scalar.*8.*

Change-Id: I34e697eb5b1c554b98a2fe556368bd6485fb406f

Fix incorrect use of copyImageToBuffer in TextureRenderer::renderQuad()

The last parameter is an access flag, not a pipeline stage one.
As the default value is already correct, removing the last
parameter is enough.

Fixes https://gitlab.khronos.org/Tracker/vk-gl-cts/issues/1511

Components: Vulkan

VK-GL-CTS issue: 1511
Affects: dEQP-VK.texture.*

Change-Id: Ib261482d4a9fae911fbda6ad5a45548a02430c1a
(cherry picked from commit 92491133746a3d1715a683dad7f5516096c79d8f)

Allow coarse derivatives to be calced by fine func

OpDPdxCoarse (OpDPdyCoarse) are allowed to be calculated by same
formulas as OpDPdxFine (OpDPdyFine).  Also due to OpFwidthCoarse
is described as sum of OpDPdxCoarse, OpDPdyCoarse allow it to be
calculated by OpFwidthFine.

Components: vulkan

VK-GL-CTS issue: 1452

Affects: dEQP-VK.spirv_assembly.instruction.graphics.float16.derivative_*.opdpdxcoarse

Change-Id: Ib91d813626e579309d3d9176d52b0954efa920d4

Add 1.1.3.0 to known conformance versions

Components: Vulkan

Affects: dEQP-VK.api.driver_properties.properties

Change-Id: I521e0a02b6b26065e7da30b006511cdaf7707418

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: Ic27d89a0685072c89ba6c0ab226f7a4c7459f98a

Correct support conditions for float16s

My previous fix missed out the graphics versions of these tests, so add
the correct conditions under which these tests can be supported as well.

Components: Vulkan
VK-GL-CTS issue: 1499
Affects:
dEQP-VK.spirv_assembly.instruction.compute.float16.opconstantcomposite.*
dEQP-VK.spirv_assembly.instruction.graphics.opspecconstantop.fconvert16_*
dEQP-VK.spirv_assembly.instruction.graphics.float16.opconstant.*

Change-Id: I7ac48027b4160bd82484e7d8d780d8a1870babbf
(cherry picked from commit f2ed80c3d2f8b4f2e782f71cc3755fe06ab52b33)

Remove one method of specifying features in vktSpvAsm

The features can either be specified as a vector of strings (most of
which won't work), or as a VulkanFeatures struct, which is properly
checked. Remove the vector of strings method and change everything to
use the VulkanFeatures.

Components: Vulkan
Affects: dEQP-VK.spirv_assembly.*

Change-Id: Ia4f51f9e64dde129cb577982f2a17fa529c4c961
(cherry picked from commit f58817ce095d2a0dda0b5d2a57f82428b3462f38)

Remove incorrect vertexPipelineStoresAndAtomics dependencies

Components: Vulkan
Affects: dEQP-VK.pipeline.push_descriptor.graphics.*
         dEQP-VK.spirv_assembly.instruction.graphics.variable_pointers.*
         dEQP-VK.glsl.opaque_type_indexing.ssbo.*
         dEQP-VK.glsl.opaque_type_indexing.ssbo_storage_buffer_decoration.*

Change-Id: I68a6a42aac647e61b6d58f2cdc74b0a5b9ea9718

Correct support conditions for float16s

My previous fix missed out the graphics versions of these tests, so add
the correct conditions under which these tests can be supported as well.

Components: Vulkan
VK-GL-CTS issue: 1499
Affects:
dEQP-VK.spirv_assembly.instruction.compute.float16.opconstantcomposite.*
dEQP-VK.spirv_assembly.instruction.graphics.opspecconstantop.fconvert16_*
dEQP-VK.spirv_assembly.instruction.graphics.float16.opconstant.*

Change-Id: I7ac48027b4160bd82484e7d8d780d8a1870babbf

Remove one method of specifying features in vktSpvAsm

The features can either be specified as a vector of strings (most of
which won't work), or as a VulkanFeatures struct, which is properly
checked. Remove the vector of strings method and change everything to
use the VulkanFeatures.

Components: Vulkan
Affects: dEQP-VK.spirv_assembly.*

Change-Id: Ia4f51f9e64dde129cb577982f2a17fa529c4c961

Add tests for vkAcquireNextImage2KHR

This change adds two tests for vkAcquireNextImage2KHR.
* "basic2" test is based on existing "basic" test. This test
was generalized so that it can be executed using both
vkAcquireNextImage and vkAcquireNextImage2.
* New "device_group" test also does the same thing as "basic"
tests but it is rendering triangle over 600 frames using
device_group extension.
This change triggers layer errors (4 for each frame) but it
seams this is because of AcquireNextImage2 not beeing supported
by layers (https://github.com/KhronosGroup/
Vulkan-LoaderAndValidationLayers/issues/2502).

VK-GL-CTS issue: 1369

Components: Vulkan

Affects:
dEQP-VK.wsi.*.swapchain.render.*

Change-Id: I31b4ff99990a2b478cbcbbf4a8f7978edf2163c2

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: Ibbb2f02096cee66991004e8b14dedd9c4c618c59

Update glslang

Component: Framework

Change-Id: I35639832f859e7eb4d62c887eee465555f04b877

Clear Core Validation Errors for Protected Memory

Affects: dEQP-VK.protected_memory.*

Components: Vulkan

VK-GL-CTS issue: 1539

Change-Id: I19c51330394f2fccc7a7a01fed3395bb66a5ab3f

Cause vkrunner tests to fail if scripts not found

Rather than simply ignoring the test throw an InternalError.

Components: Vulkan
VK-GL-CTS Issue: 1548
Affects: dEQP-VK.vkrunner-example.*

Change-Id: Ib29692c792acf49f428e042fb9cc951dd32130dd
(cherry picked from commit 255f67d1d9dc6616058335ca8a0597b92a3bfa27)

Allow a tolerance for results of sqrt

The tolerance added to the test is around 5 ULP.

Components: Vulkan
VK-GL-CTS Issue: 1546
Affects: dEQP-VK.vkrunner-example.sqrt_*

Change-Id: Idfb6f8485d10a29cf0c4c46355df1df1008298df
(cherry picked from commit 215b2e3f4424e4568cb19c290d82fe7c48cb1b8b)

Added test for backwards subpass self dependency

There is a plan to change spec to allow subpass self dependency
to go backwards in non-framebuffer stages. This change adds
a test for one such case where geometry shader writes to an
indirect buffer, which in turn is used by an indirect draw
in the same subpass.

Affects:

dEQP-VK.renderpass*.subpass_dependencies.self_dependency.*

Components: Vulkan

VK-GL-CTS issue: 1446
Change-Id: If1d1f5fd7ad5938d54baf4c70dfc76ca0f332972

Fix dup tests to not free in-use semaphores/fences

The dup{,2,3} tests were subject to a race condition in which a
semaphore may be deleted before a VkQueue which was dependent on it had
completed execution.

This change moves the vkQueueWaitIdle call into the same scope as the
semaphores/fences that the queue is dependent on.

VK-GL-CTS issue: 1538

Components: Vulkan

Affects: dEQP-VK.api.external.{fence,semaphore}.opaque_fd.dup*

Change-Id: I73cf541c9c184790df44b68b8a065787634847d9
(cherry picked from commit 732a8b6800972b271735cf9b865a3612fb56dfc4)

Fix dup tests to not free in-use semaphores/fences

The dup{,2,3} tests were subject to a race condition in which a
semaphore may be deleted before a VkQueue which was dependent on it had
completed execution.

This change moves the vkQueueWaitIdle call into the same scope as the
semaphores/fences that the queue is dependent on.

VK-GL-CTS issue: 1538

Components: Vulkan

Affects: dEQP-VK.api.external.{fence,semaphore}.opaque_fd.dup*

Change-Id: I73cf541c9c184790df44b68b8a065787634847d9

pipeline timestamps test: fix too-large VkBufferCopy

The pipeline timestamps test intends to copy the entire buffer contents
from source to destination, but the copy is done for 512 bytes, which
currently exceeds the size of the buffer, 256. That is not valid
behavior.

Components: Vulkan
Affects: dEQP-VK.pipeline.timestamp.transfer_tests.transfer_stage_with_copy_buffer_method
VK-GL-CTS Issue: 1535

Change-Id: I1ab1ef3a94f4d22702d37c86b6aa2c21c0919a7c

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I53a8d2676ea31656cd43414e20d203961fbdac76

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: I7a9d2154ae19bdac61feed815f69691914a59bd4

Set correct range for OpVectorTimesScalar tests

The min[] and max[] values should be based on the result in double precision, not after it has been converted to fp16type.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_2.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_3.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_4.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_2.opvectortimesscalar*
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_3.opvectortimesscalar*
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_4.opvectortimesscalar*

Components: Vulkan

VK-GL-CTS issue: 1532
Change-Id: I30810c8c954ee899bb05602b1f8805f82ac021b3
(cherry picked from commit 12b26bd9762dd56068522cc0a17b26c9e31df1e2)

Fix errors in SPIR-V 16bit_storage tests

Commit 04ba8ac400a3 was wrongly setting a condition that made
all the *16_to_16* tests to fail. Also, in the last rebase of CL#2458,
there were a rename of an argument of OpFunction that was not updated.

Affects: dEQP-VK.spirv_assembly.instruction.graphics.16bit_storage.*

Components: Vulkan
VK-GL-CTS issue: 1234

Change-Id: Ia83bedefa33d3e37b62336ebadb2bc992e0ddb2a
(cherry picked from commit 56e8d4e2318dcd0bceddca4d182649f3a03be7ab)

Remove NaN handling in float16 tests

Support of NaNs is optional according the specification.
Tests always have expected that NaNs are always supported.
To be compatible with the specification skip check of NaN
input floats.

Updated tests:
 * dEQP-VK.spirv_assembly.instruction.compute.float16.logical.opfunord*
 * dEQP-VK.spirv_assembly.instruction.graphics.float16.logical.opfunord*
 * dEQP-VK.spirv_assembly.instruction.compute.float16.logical.opford*
 * dEQP-VK.spirv_assembly.instruction.graphics.float16.logical.opford*

Affected tests:
 * dEQP-VK.spirv_assembly.instruction.compute.float16.logical.opisnan*
 * dEQP-VK.spirv_assembly.instruction.graphics.float16.logical.opisnan*
 * dEQP-VK.spirv_assembly.instruction.compute.float16.logical.opisinf*
 * dEQP-VK.spirv_assembly.instruction.graphics.float16.logical.opisinf*

Components: Vulkan

VK-GL-CTS issue: 1528

Change-Id: Iadd91ed24c78e15ff458355262b009015b8a8e4a

Set correct range for OpVectorTimesScalar tests

The min[] and max[] values should be based on the result in double precision, not after it has been converted to fp16type.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_2.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_3.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.compute.float16.arithmetic_4.opvectortimesscalar
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_2.opvectortimesscalar*
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_3.opvectortimesscalar*
dEQP-VK.spirv_assembly.instruction.graphics.float16.arithmetic_4.opvectortimesscalar*

Components: Vulkan

VK-GL-CTS issue: 1532
Change-Id: I30810c8c954ee899bb05602b1f8805f82ac021b3

Allow a tolerance for results of sqrt

The tolerance added to the test is around 5 ULP.

Components: Vulkan
VK-GL-CTS Issue: 1546
Affects: dEQP-VK.vkrunner-example.sqrt_*

Change-Id: Idfb6f8485d10a29cf0c4c46355df1df1008298df

Fix incorrect use of copyImageToBuffer in TextureRenderer::renderQuad()

The last parameter is an access flag, not a pipeline stage one.
As the default value is already correct, removing the last
parameter is enough.

Fixes https://gitlab.khronos.org/Tracker/vk-gl-cts/issues/1511

Components: Vulkan

VK-GL-CTS issue: 1511
Affects: dEQP-VK.texture.*

Change-Id: Ib261482d4a9fae911fbda6ad5a45548a02430c1a

Cause vkrunner tests to fail if scripts not found

Rather than simply ignoring the test throw an InternalError.

Components: Vulkan
VK-GL-CTS Issue: 1548
Affects: dEQP-VK.vkrunner-example.*

Change-Id: Ib29692c792acf49f428e042fb9cc951dd32130dd

Propagate required features down to the constructor

VariablePointers wasn't fully propagated down to the test constructor

Components: Vulkan

VK-GL-CTS issue: 1540

Affects:
dEQP-VK.spirv_assembly.instruction.compute.pointer_parameter.workgroup_memory_variable_pointers

Change-Id: I4d29248ad4c2f38cdda5c31be1be781331433399

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: If836faad2ea632456cf922f123ad1adb8eeed326

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: I5302ce80a95a036db240f3097f806ffbfad6fe43

Remove unnecessary extension declarations

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

Merge vk-gl-cts/aosp-deqp-dev into vk-gl-cts/master

Change-Id: I3dd5a5a6c3f33307283624b2bc40cd93716f9785

Run validation before optimization.

The optimizer runs is own pass of the validator as a safety check.
However, that call to the validator does not get the any validatoin options passed in.
This causes problems with the memory layout tests.

The solution we will be to diable the call to the validator in the
optimizer.  Instead the will be an explict call to validate the binary
before optimization starts, reusing functions that already exist to do
that.

At the same time, the run of the validator that is done with
--deqp-validation=enable will be moved so that it runs after
optimization.  This way it will validate the output instead of the
input, and it will not be a redundant check.

VK-GL-CTS issue: 1503

Components: Vulkan, Framework

Affects: dEQP-VK.ssbo.layout.random.relaxed.*

Change-Id: Ib52fa97505855316739e07e472d23fcab4ac2beb

Update common.py to get the latest EGL registry.

Update common.py in order to get
EGL_EXT_gl_colorspace_display_p3_passthrough extension.

Components: EGL
VK-GL-CTS Issue: 1534

Change-Id: Ie24de656e9b88317b8e2fdc5b07d4e5ede796d0e

Merge "Merge commit aosp/master into deqp-dev" into deqp-dev

Merge remote-tracking branch 'khronos/master' into deqp-dev

Change-Id: Ibacf4a76415fb5cdb0d35506adbae5f6f9e63364

Merge vk-gl-cts/vulkan-cts-1.1.3 into vk-gl-cts/master

Change-Id: I67bea781c4ee93c08da8ab6a45b33820672a62d5

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/vulkan-cts-1.1.3

Change-Id: I46453ca0e9531975079099a6f06e9076a20accf0

Merge vk-gl-cts/vulkan-cts-1.1.1 into vk-gl-cts/vulkan-cts-1.1.2

Change-Id: I93423c80882209c8c41d4bd9f3d403037ec9ad01

Merge vk-gl-cts/vulkan-cts-1.1.0 into vk-gl-cts/vulkan-cts-1.1.1

Change-Id: Ie0a15653a8e00e8dc4acb07c0371dad770c5ee9f

Fix Visual Studio 2013 build of VkRunner

There were build errors related to undefined inline attribute and
undefined offsetof() function.

Change-Id: I111225756c1c950d73c4886fe1c2d3d203b567c2
Components: Vulkan
VK-GL-CTS: 1530
(cherry picked from commit 0b091875702f6ac418928267917987196f0b1854)

Fix Visual Studio 2013 build of VkRunner

There were build errors related to undefined inline attribute and
undefined offsetof() function.

Change-Id: I111225756c1c950d73c4886fe1c2d3d203b567c2
Components: Vulkan
VK-GL-CTS: 1530

Fix 0b prefix in conditional rendering tests

Binary integers prefixes 0b breaks Visual Studio 2013
build.  Replace them with substitute constants.

Components: Vulkan

VK-GL-CTS issue: 1529

Change-Id: I58e9c1e18ba61f2248ec4ff7f29b1bf29eba8207

Fix image aspects in multiview tests

The image aspects were not being set for the renderpass2 versions of
these tests either.

Component: Vulkan
Affects: dEQP-VK.multiview.*

Change-Id: I4062742caeffb220d0c74e61a5e489bbf4534e49
(cherry picked from commit 132adab7998788b20232794962215026a097b500)

Test different data types for workgroup memory

Workgroup memory was being tested only for 32bit values. New
tests were added to cover signed and unsigned 8/16/32/64-bit
integers and 16/32/64-bit floats.

New tests:

dEQP-VK.spirv_assembly.instruction.compute.workgroup_memory.*

Components: Vulkan

VK-GL-CTS issue: 1469
Change-Id: I3f699108cd8b87abc621cfd49bb6dad094833604

Update KC CTS

Components: Framework
VK-GL-CTS issue: 1475

Change-Id: Ic241a395eb96a99c4278ae37f71c3b8b41385d30

Refactor Sync and Image tests for EGL 1.5.

Refactor Sync and Image tests for EGL 1.5 by templatizing them to allow
running the tests for both the base and extension EGL calls.

Affects:
dEQP-EGL.functional.fence_sync.*
dEQP-EGL.functional.reusable_sync.*
dEQP-EGL.functional.image.api.invalid_create_image

Components: EGL

Google bug: 80239516

Change-Id: I665c4f0ce03fd09dbf05900dca203899d73b2568

Check for exact spec version for VK_KHR_vulkan_memory_model

This extension is provisional and its Features struct is expected to change soon.
Add a version check to protect against querying the struct and running tests with
an incompatible driver.

Components: Vulkan
Affects: dEQP-VK.memory_model.*
Change-Id: Iedd08fa6433af4c59e18c58aa39c5366ad72570e

Fix image aspects in multiview tests

The image aspects were not being set for the renderpass2 versions of
these tests either.

Component: Vulkan
Affects: dEQP-VK.multiview.*

Change-Id: I4062742caeffb220d0c74e61a5e489bbf4534e49

Fix memory requirements for draw_clear tests

The memory requirements for the draw_clear subtests of
the conditional_rendering group required the memory
bound for the conditional rendering predicate buffer
to be both host visible and device local.

It's not a requirement that all implementation support
this combinations so all the draw_clear tests were
skipped unnecessarily.

Since the device local requirement was unnecessary,
this CL just removes it.

Affects:
dEQP-VK.conditional_rendering.draw_clear.*

Change-Id: I10d289a7666a97d665707d5e49ec7c8846d2997d
Components: Vulkan
(cherry picked from commit 11bfc063db2b32fbda21b17d232739469c9738a1)

Fix memory requirements for draw_clear tests

The memory requirements for the draw_clear subtests of
the conditional_rendering group required the memory
bound for the conditional rendering predicate buffer
to be both host visible and device local.

It's not a requirement that all implementation support
this combinations so all the draw_clear tests were
skipped unnecessarily.

Since the device local requirement was unnecessary,
this CL just removes it.

Affects:
dEQP-VK.conditional_rendering.draw_clear.*

Change-Id: I10d289a7666a97d665707d5e49ec7c8846d2997d
Components: Vulkan

Update Vulkan CTS Readme to point to 1.1.4

Components: Vulkan

Change-Id: Ibef8d751890c6536087ea6f09dd1ee421e3f868d

Disable 0b-prefixed integers to be used

Binary integers prefixes 0b breaks Visual Studio 2013 build.
Add validation check to avoid usage of 0b prefixed integers.

Components: Vulkan

VK-GL-CTS issue: 1529

Change-Id: I8bac5b5788adac544b72ca44b74f847b1d3c96cf

Start 1.1.4 mustpass

Components: Vulkan

Change-Id: I86860cd62ba12a94b13bda7c7cd7345e5866b080

Merge vk-gl-cts/vulkan-cts-next-dev into vk-gl-cts/master

Change-Id: If7f438b4268baae507cad11799537acaad9937dc

Don't fail DisplayProperties2KHR with no display

Instead, we should mark this test as not supported.

VK-GL-CTS issue: 1527
Components: Vulkan

Change-Id: I3b23ee27b6095b9dd1a4cdec01f0b92046aa8bc5
Affects: dEQP-VK.wsi.display.get_display_properties2

Merge vk-gl-cts/aosp-deqp-dev into vk-gl-cts/master

Change-Id: I16ba69b95f84cbb26e9d082f7a7fbb3180a092e5

Merge vk-gl-cts/master into vk-gl-cts/vulkan-cts-next-dev

Change-Id: Id1fe64ddc8d2ac42c8d27a71f65656422a523e3b

Merge vk-gl-cts/vulkan-cts-1.1.2 into vk-gl-cts/master

Change-Id: I3f84de46fe99fe2345041f61d4ddd9ea1763267a

Be more careful about Block decorations on structs

In spirv/SPIR-V@9701a9f9 we changed the SPIR-V spec to say that
block-decorated structures could not be nested.  This is needed for
sorting out certain variable pointers issues inside a SPIR-V parser.
Unfortunately, the mentioned group of CTS tests violates this rule.
Their %input_buffer type is a struct containing two instances of
%outer_struct and both %input_buffer and %outer_struct are decorated
Block.  Fortunately, only one of the two struct types is actually
required for an input at any given type so we just need to be more
careful about the decorations.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.variable_pointers.complex_types_compute.*

Components: Vulkan

VK-GL-CTS issue: 1518
Change-Id: I87a832fab94a9c06317d20b32afedf16b52ab2f2
(cherry picked from commit 1dbdfe34947fdb3e6d24226a58f02d0b8a515da8)