Test viewport with zero height

New tests were added to verify that the driver does not crash and that
triangle primitives are not drawn when using a viewport with zero
height.

New tests:

dEQP-VK.draw.zero_viewport_height.*

Components: Vulkan

VK-GL-CTS issue: 1048

Change-Id: I886f604944ad6224e31eb70a72917e505d4e11c5

Alignment tests of 16bit types in structs

Add tests:
dEQP-VK.spirv_assembly.instruction.compute.16bit_storage.uniform_16struct_to_32struct.*
dEQP-VK.spirv_assembly.instruction.compute.16bit_storage.uniform_32struct_to_16struct.*
dEQP-VK.spirv_assembly.instruction.graphics.16bit_storage.uniform_16struct_to_32struct.*
dEQP-VK.spirv_assembly.instruction.graphics.16bit_storage.uniform_32struct_to_16struct.*
dEQP-VK.spirv_assembly.instruction.compute.16bit_storage.struct_mixed_types.*
dEQP-VK.spirv_assembly.instruction.graphics.16bit_storage.struct_mixed_types.*

Components: Vulkan

VK-GL-CTS issue: 1061

Change-Id: I9f206431cd313c736bc12b4b8a002426ffbf6398

Fix random shader generator

Put parentheses around the RHS when converting L == R into
abs(L - R) < 0.001.

Affects: dEQP-GLES2.functional.shaders.random.*
         dEQP-GLES3.functional.shaders.random.*

Components: OpenGL

VK-GL-CTS issue: 286

Change-Id: I95d43ed40944c2815d7ce464ac4dc4993f87a4e9

Add missing SConvert, FConvert and UConvert tests

Different variations of SConvert, FConvert and UConvert weren't properly
tested in compute and graphics shaders.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.sconvert.*
dEQP-VK.spirv_assembly.instruction.compute.uconvert.*

New tests:

dEQP-VK.spirv_assembly.instruction.compute.sconvert.int32_to_*int16
dEQP-VK.spirv_assembly.instruction.compute.sconvert.int64_to_*
dEQP-VK.spirv_assembly.instruction.compute.sconvert.uint*
dEQP-VK.spirv_assembly.instruction.compute.sconvert.*vec*
dEQP-VK.spirv_assembly.instruction.compute.uconvert.uint32_to_uint16
dEQP-VK.spirv_assembly.instruction.compute.uconvert.uint64_to_*
dEQP-VK.spirv_assembly.instruction.compute.fconvert.*
dEQP-VK.spirv_assembly.instruction.graphics.sconvert.*
dEQP-VK.spirv_assembly.instruction.graphics.uconvert.*
dEQP-VK.spirv_assembly.instruction.graphics.fconvert.*

Components: Vulkan

VK-GL-CTS issue: 880

Change-Id: Id84c7a20749cd15e84a9c85993c1e6574712889e

Added shader cache to speed up runtime

Added shader cache which hashes incoming shader sources and saves the
resulting binaries on disk, avoiding unneccessary recompilation.

On testing it was found that the conformance suite calls shader
compilation about 500k times while only about 30k shaders are unique.
When all shaders are prebuilt on disk, complete runtime is reduced by
about 30%.

The shader cache also affects vk-build-programs. If shaders already
exist and shader validation is not enabled, vk-build-programs runs about
95% faster.

The timings above are based on debug builds on a desktop PC. Release
builds get less benefit, but this may be helpful on mobile platforms.

The shader cache does not affect targets which do not include spir-v
tools.

Affects:

dEQP-VK.*

Components: Framework, Vulkan

VK-GL-CTS issue: 899

Change-Id: I9b1ea915b3c9d94be85adb17161c12f453175d78

Add GL_ARB_draw_indirect extension

Add wrapper functions defined by this extension.

Components: Framework

VK-GL-CTS issue: 1087

Affects:
GTF-GL3*.gtf31.GL3Tests.primitive_restart.primitive_restart_draw_elements
GTF-GL3*.gtf32.GL3Tests.vertex_array_bgra.vertex_array_bgra_draw_calls

Change-Id: I0e13535a099b6599be3127685c473cbab120b0d4

Test for return values of vkGet*ProcAddr

This change tests vkGet*ProcAddr. It also removes special
case for vkGetDeviceProcAddr which now is considered by a
framework as a device-level function.

Components: Framework, Vulkan

VK-GL-CTS issue: 948

Affects:
dEQP-VK.api.version_check.entry_points
dEQP-VK.*

Change-Id: I629bceaae25534e5eff9550034e7af0ab2668aa9

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

Change-Id: I4ebb0dcf96e37806528ee00bb78c586706224680

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

Change-Id: I752717cf0180806e0745a0a7d055e3471a2b203b

Request proper features in spirv_asembly tests

image_sampler and ubo_padding tests used stores to SSBOs without
requesting proper features. In this change implementation of
ShaderTestUtil was modified to enable possibility to create test
for individual stages. Now proper features could be requested
before performing the tests for specified stages.

Components: Vulkan

VK-GL-CTS issue: 1049

Affects:
dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.*
dEQP-VK.spirv_assembly.instruction.graphics.ubo_padding.*
dEQP-VK.spirv_assembly.instruction.graphics.*

Change-Id: I03441b385097340faa72b18e17f09a07eba49efb

Test QueueSubmit with null SubmitInfo and fence

This test matches
VkLayerTest.TwoQueueSubmitsOneQueueNullQueueSubmitWithFence found in
layer validation tests which fails on some shipping drivers.

New tests:

dEQP-VK.api.command_buffers.submit_two_buffers_one_buffer_null_with_fence

Components: Vulkan

VK-GL-CTS issue: 836

Change-Id: I57eb96bb724406dafc522041c95ac99fd0e103b4

Add glu::BufferOffsetAsPointer utility

Clang 6.x adds a warning that doing arithmetic involving a NULL
pointer is a GNU extension. This affects the common GL pattern
    (deUint8*)DE_NULL + offset
used to pass an offset into a bound buffer object as a client pointer
parameter.

This change replaces that pattern with a call to a new utility
function glu::BufferOffsetAsPointer(), which will hopefully be
warning-free on all compilers.

Change-Id: I8be939297b02c44091441c71ae75e45ceab30639
Components: Framework, OpenGL, AOSP

Add tests for VK_EXT_post_depth_coverage

Additional tests for sample mask with depth test without extension

New tests:
dEQP-VK.pipeline.multisample.sample_mask_with_depth_test.*

Components: Vulkan
VK-GL-CTS issue: 888

Change-Id: I2cecb9f9db52ca471fea9a8630923dab68730e59

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

Change-Id: Ia97d8543cbf11fa2a525cdb96d4c23a644645a61

Merge "Merge vk-g-cts/opengl-cts-4.6.0 into vk-gl-cts/master"

Merge vk-g-cts/opengl-cts-4.6.0 into vk-gl-cts/master

Change-Id: Ibf1a7adea71d5d7c94205bcc83e5200b6c86b91c

vktApiFeatureInfo.cpp: fix one 'Wshadow' warning on GCC

Components: Vulkan

VK-GL-CTS Issue: 1101

Affects: None

Change-Id: Iba44a66378f12543d54967afba25f496f6bc3ab4

Fixes cap check for 16bit storage stress test

Missing cap check for StorageUniformBufferBlock16

Affects:
dEQP-VK.spirv_assembly.instruction.compute.16bit_storage.uniform_16_to_16.stress_test

Components: Vulkan

VK-GL-CTS issue: 1091

Change-Id: Ifaf31eab31ca2b8fe9cf23c94087cf25542bd8ed

Use common utility function for command buffer begin and end

Command buffer begin and end is used in almost every test case. Some test
groups already implement an utility function for these. Others do it
inline. These are now replaced with common utility functions.

Affects:

dEQP-VK.compute.*
dEQP-VK.fragment_operations.*
dEQP-VK.geometry.*
dEQP-VK.image.*
dEQP-VK.pipeline.*
dEQP-VK.protected_memory.*
dEQP-VK.query_pool.*
dEQP-VK.renderpass.*
dEQP-VK.sparse_resources.*
dEQP-VK.spirv_assembly.*
dEQP-VK.synchronization.*
dEQP-VK.tessellation.*
dEQP-VK.clipping.*
dEQP-VK.ycbcr.query.*
dEQP-VK.glsl.*
dEQP-VK.api.*
dEQP-VK.device_group.*
dEQP-VK.draw.*
dEQP-VK.dynamic_state.*
dEQP-VK.memory.*
dEQP-VK.multiview.*
dEQP-VK.pipeline.*
dEQP-VK.robustness.*
dEQP-VK.ssbo.*
dEQP-VK.binding_model.shader_access.*
dEQP-VK.texture.*
dEQP-VK.wsi.*
dEQP-VK.ycbcr.*
dEQP-VK.rasterization.*
dEQP-VK.ubo.*

Components: Vulkan, Framework

VK-GL-CTS issue: 1022

Change-Id: I84bec62fb2d0a5ce408a85cd243e7f629acc5867

Fix off indentation in vktApiFeatureInfo.cpp

Components: Vulkan

VK-GL-CTS Issue: 1102

Affects: None

Change-Id: Id05617e0bc2895de1ae67ea3925f4300c5d55d3f

Fixed rounding issue in spirv_assembly tests

The test assumes that the host system and the device use the same
rounding mode. When this is not the case, the derived rounding errors
make the test fail unexpectedly. By rounding the input values down we
avoid differences in the test output caused by the rounding modes in
use.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.loop_control.dependency_infinite

Components: Vulkan

VK-GL-CTS issue: 1097

Change-Id: I3678e38cbe5c6d57cecef91e7398df0d04763851

Merge vk-gl-cts/opengl-es-cts-3.2.4 into vk-gl-cts/master

Change-Id: Id1abb58d7a7c93277531a74ca38eaf5d5e6d7550

Fix regression after adding run time report (4758c8d4)

qpXmlWriter_endDocument was not called after the end of test
case timing XML block, leaving XML writer in bad state.

This causes assert on subsequent XML blocks being started.

VK-GL-CTS Issue: 1086

Components: Framework

Change-Id: I8bf2fd48517a5389671389dcb0883e168a587da7
Affects: all

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

Change-Id: Ibcd4407ae531a32d85f7e6f6fc263247a1f25da3

Move khr_debug tests to NoContext

As implemented, the KHR-GL4*.khr_debug.* tests can terminate the EGL
display used by other tests. As recommended, move these tests to the
KHR-NoContext.* group.

Since the test names are already changing, fix the spelling of
'receiving' in one test case.

Components: OpenGL

VK-GL-CTS Issue: 1062

Renamed tests:
KHR-GL4*.khr_debug.* -> KHR-NoContext.gl43.khr_debug.*

Change-Id: I766131f77206cad1cb34955426a81cf10b50e159

Fix SSBO tests that assume VS is invocated once per vertex

Some implementations may execute vs shaders more than
once per vertex as allowed per spec.

Affects:
KHR-GLES31.core.shader_storage_buffer_object.advanced-usage-sync-vsfs
KHR-GLES31.core.shader_storage_buffer_object.basic-atomic-case3-vsfs
KHR-GLES31.core.shader_storage_buffer_object.basic-atomic-case4-vsfs

Components: OpenGL ES
VK-GL-CTS issue: 949

Change-Id: I40d4ef540c9868db43554f898b92c333a79eac18

Fix compute shader in KHR-NoContext.es32.robustness.getnuniform

Patch removes unnecessary difference in compute shader between
desktop and ES profiles, now both output to SSBO in same manner.

Problem with the shader on ES was that some compilers detected
declared shared variables as dead and optimized them away which
made uniforms inactive.

Components: OpenGL ES
VK-GL-CTS issue: 1036

Affects:
KHR-NoContext.es32.robustness.getnuniform

Change-Id: I8a4aeed926faa03a62c7973fe21ef979fb61bf53

RGB10_A2UI renderbuffer case requires integer data

Most of the cases in KHR-GL40.internalformat.renderbuffer.* test are
float-based, and thus the shaders as well as the functions used to read
the pixels are based on floats.

But for this specific format, we need to use integer data, both when the
shader writes out the colors as well as when we read it back with
readPixels(). And later we need to convert the results to floats in
order to compare with the expected results (which are float-based
values).

Components: OpenGL

VK-GL-CTS issue: 1001

Affects:
KHR-GLES*.core.internalformat.renderbuffer.*
KHR-GL*.internalformat.renderbuffer.*

Change-Id: I44c051817672614fb1ed6ed1927f917afcaadc96

Try to use a separated FBO for copy operation

If the FBO can not be created because the format under testing does not
allow it (FBO not complete), then the copy will be done from the main
FBO.

Components: OpenGL

VK-GL-CTS issue: 1000

Affects:
KHR-GLES*.core.internalformat.copy_tex_image.*
KHR-GL*.internalformat.copy_tex_image.*

Change-Id: I64f05a9d542abcc2abbeb865e5d638bfa64a0ad3

Add tests for OpSwitch operator

This create tests for SPIR-V OpSwitch with different integer types.

It provides tests using 32-bit and 64-bit signed and unsigned integers.
But it is designed to easily extend to other types, like 16-bit
integers.

New tests:
dEQP-VK.spirv_assembly.type.scalar.*.switch_*

VK-GL-CTS issue: 995

Components: Vulkan

Change-Id: I4940f5853c81dff0191c3d352ddbf439eb0ea560

Check VkPhysicalDevicePushDescriptorPropertiesKHR

This adds test for getting VkPhysicalDevicePushDescriptorPropertiesKHR
using vkGetPhysicalDeviceProperties2. The test also verifies
maxPushDescriptors against spec requiremnts.

Components: Vulkan

VK-GL-CTS Issue: 1053

Affects:
dEQP-VK.api.info.get_physical_device_properties2.properties

Change-Id: I44ee1b0a3005d149cc7fb3c60414a412e36c45a4

Remove debugs from vktSpvAsmGraphicsShaderTestUtil.cpp

Remove debug messages represented by two SPIR-V instructions:
 * OpName
 * OpMemberName

Affects:
 * dEQP-VK.spirv_assembly.instruction.graphics.module.*

Components: Vulkan

VK-GL-CTS issue: 963

Change-Id: I3091f6f8772aecfdf6f7d799660dfc3175d6476f

Fix validation errors in pipeline tests

All validation errors found in dEQP-VK.pipeline.* were fixed.

Affects:

dEQP-VK.pipeline.stencil.*
dEQP-VK.pipeline.blend.*
dEQP-VK.pipeline.image.*
dEQP-VK.pipeline.sampler.*
dEQP-VK.pipeline.multisample.*
dEQP-VK.pipeline.vertex_input.*
dEQP-VK.pipeline.input_assembly.*
dEQP-VK.pipeline.timestamp.*
dEQP-VK.pipeline.cache.*
dEQP-VK.pipeline.render_to_image.*
dEQP-VK.pipeline.framebuffer_attachment.*

Components: Vulkan

VK-GL-CTS issue: 1057

Change-Id: I494ee517dbba529957593577f3927937fdab37a2

Fix invalid command pool create flag in image tests

Many image tests use a command pool that has been created using a
parameter of type VkCommandPoolResetFlagBits instead of
VkCommandPoolCreateFlags. The flag was changed to a proper type while
still retaining the original bit value.

Affects:

dEQP-VK.image.store.*
dEQP-VK.image.load_store.*
dEQP-VK.image.format_reinterpret.*
dEQP-VK.image.qualifiers.restrict.*

Components: Vulkan

VK-GL-CTS issue: 576

Change-Id: I99ecfafa6d320d262fec74508ad39ae3d2cb374c

Combined duplicate code

There were several places where sparse binding and buffer to
image copies were done. These are now using common functions for
better code maintenance.

Affects:

dEQP-VK.glsl.*texture*
dEQP-VK.texture.*
dEQP-VK.spirv_assembly.*.compute.*

Components: Vulkan

VK-GL-CTS issue: 996

Change-Id: I53b01555ed6ef6e502b227e3cf026f2b05fd8f2a

Fix invalid memory scope in SPIR-V tests

Existing SPIR-V assembly tests using CrossDevice memory scope in atomic
operations were modified to use Device scope instead.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.opatomic.load
dEQP-VK.spirv_assembly.instruction.compute.opatomic.store
dEQP-VK.spirv_assembly.instruction.compute.opatomic_storage_buffer.load
dEQP-VK.spirv_assembly.instruction.compute.opatomic_storage_buffer.store

Components: Vulkan

VK-GL-CTS issue: 991

Change-Id: I30a2db6e4614296f08e2ed2c2717315aef0afb6e

Fix XCreateWindow BadMatch with depth 32 visuals

For the common case of a depth 24 root window, when creating a window
with a visual of depth 32, attempting to use a depth of CopyFromParent
will result in a BadMatch error.

Fix this by passing the visual's depth to XCreateWindow explicitly --
fortunately the VisualInfo structure was already queried.

Components: Framework

VK-GL-CTS issue: 938

Change-Id: I54b142b6105c7f328e76d7ea0f9b01e635380e4e
(cherry picked from commit af57c99056538393e99925d6c3bee0253889a7c3)

Account for differences in nan with fp16 comparison

When comparing fp16 values it's not possible to use a bitwise
comparison because there can be many different bit patterns
representing nan values, and all nans are equal. This CL updates
the computeCheckBuffersFloats function which compares two
buffers of fp16 values to take into account different forms
of nan.

Affects:

dEQP-VK.spirv_assembly.instruction.compute.16bit_storage.uniform_16_to_16.stress_test

Components: Vulkan

VK-GL-CTS issue: 1018

Change-Id: I77013b49ce070d9615b287eae24dd32731b314b1
(cherry picked from commit 92d5909ab99bc97dc2ae4dd67e1cfa592dafb246)

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

Change-Id: Ia487f499b5e677146b0422c17a6aa00439ed3525

Add more context for SPIR-V build log outputs.

This can help newcomers write new SPIR-V tests.

VK-GL-CTS issue: 1029

Change-Id: I9673847a599839c61b15fe438b0e28b1c3294c32

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

Change-Id: I391929e9887ffa793eefacead5444fff103acbbb

Test using gl_ViewportIndex in a vertex shader

Needs VK_EXT_shader_viewport_index_layer extension. Test works by
setting up N viewports forming a grid and each pair of triangles is
assigned to one of the viewports. Colors are used to distinguish each
pair, and by consequence, each cell in the grid.

This test has some similarities with how the multi_viewport scissors
test in vulkan/fragment_ops/.

New tests:
dEQP-VK.draw.shader_viewport_index.vertex_shader_*

VK-GL-CTS issue: 1043

Components: Vulkan

Change-Id: I311012126f0f4acc18cc31350e16f1c36dd35c62

Test waiting for fences without waitAll

Two command buffers and two fences are created. Both command buffers
are submitted to a queue, and vkWaitForFences with waitAll set to
VK_FALSE is used to wait for either one of the fences to be signaled.

New tests:

dEQP-VK.synchronization.basic.fence.multi_waitall_false

Components: Vulkan

VK-GL-CTS issue: 1037

Change-Id: I78762b2561e8d28a9a4278a054cbe169af10366a

Test push constant ranges with overlapping shader stages

New tests were added to exercise push constant ranges where multiple
shader stages overlap. Offset and size of each push constant is
selected arbitrarily. Some ranges are only partially updated.

New tests:

dEQP-VK.pipeline.push_constant.graphics_pipeline.overlap_2_shaders_vert_frag
dEQP-VK.pipeline.push_constant.graphics_pipeline.overlap_3_shaders_vert_geom_frag
dEQP-VK.pipeline.push_constant.graphics_pipeline.overlap_4_shaders_vert_tess_frag
dEQP-VK.pipeline.push_constant.graphics_pipeline.overlap_5_shaders_vert_tess_geom_frag

Components: Vulkan

VK-GL-CTS issue: 1009

Change-Id: Ibcbf7f6f21a4827c3089719f06e02dde138f183b

Fix test with invalid viewport size

Viewport dimensions and scissor extent were changed to match the values
used in VkPositiveLayerTest.DestroyPipelineRenderPass.

Affects:

dEQP-VK.api.pipeline.renderpass.destroy_pipeline_renderpass

Components: Vulkan

VK-GL-CTS issue: 1045

Change-Id: Ie39c516237cf3a14078a83b0c51dc401c30f7672

Test pipeline layout lifetime

This test has the same functionality as
VkLayerTest.DescriptorSetCompatibility, which breaks some shipping
drivers in the validation tests.

The test creates a pipeline layout and uses that layout to create a
pipeline. The pipeline layout object is destroyed immediately after
binding the pipeline to a command buffer.

The test passes as long as no crash happens.

New tests:

dEQP-VK.api.pipeline.pipeline_layout.lifetime.graphics
dEQP-VK.api.pipeline.pipeline_layout.lifetime.compute

Components: Vulkan

VK-GL-CTS issue: 824

Change-Id: I0b72ad681528388577678b4e5dfe96eef6f3486e

Merge vk-gl-cts/opengl-cts-4.6.0 into vk-gl-cts/master

Change-Id: Ia75cc2e00a0502ab5b2d3fa83261cd9cf20839ba

Merge vk-gl-cts/opengl-es-cts-3.2.4 into vk-gl-cts/master

Change-Id: I87b9b049c6e21dc45629284cf78f02fc57a0317c

Merge vk-gl-cts/opengl-es-cts-3.2.3 into vk-gl-cts/opengl-es-cts-3.2.4

Change-Id: Iff05f0c3ed7725c4cff850649a83b78d98b341e1

Add pipeline cache corner cases

Two new test cases were added. The first case creates a pipeline cache
using valid previously retrieved data, but sets the initial cache size
to zero.

The second case creates multiple pipeline caches using previously
retrieved data, which is valid apart from cache header version, vendor
ID, device ID and pipeline cache ID.

New tests:

dEQP-VK.pipeline.cache.misc_tests.zero_size_test
dEQP-VK.pipeline.cache.misc_tests.invalid_blob_test

Components: Vulkan

VK-GL-CTS issue: 1012
VK-GL-CTS public issue: 81

Change-Id: Ie28b10f97adc2e4b2d1a2239ec5c6c1d75416087

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

Change-Id: I768567546e111ed9af5e35293d4af079453abccf

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

Change-Id: Id0a9cedc7822a3b3206e162f217f790d087c3392

Fix GL CTS Readme

Update the README to state the correct version
of the required Android API

Components: OpenGL

Public issue: 86

Change-Id: I60d7b6952957ac6803864a16eed82c9f28e07e88

Add runtime time report of a CTS run

New section have been added to file TestResults.qpa.
"#beginTestsCasesTime" - section contains a time summary for groups.
bottleneck_report.py - script which helps to find bottlenecks in CTS

Components: Framework

VK-GL-CTS issue: 879

Change-Id: I73773d820ddcd0f5b3c50c2128b8961cd1227b0e

Fetch new tags when fetching new Git sources

Change-Id: I1a906ebf2698eac33c3acbf1da74df9642b775a2
Component: Framework
(cherry picked from commit 204d597504c94fa52ed0f5a2c21895482236d5cc)

Fetch new tags when fetching new Git sources

Change-Id: I1a906ebf2698eac33c3acbf1da74df9642b775a2
Component: Framework

Use sys.executable instead of "python"

When Python scripts call other python scripts, prefer using whatever
interpreter was used for the topmost call instead of "python". In some
environments "python" is already pointing to a Python 3 interpreter.

Components: Framework

Change-Id: Ifba9b2965901ffcf9c1897ccabe5fa183f359707

Handle gcc 7 warnings

- "-Wimplicit-fallthrough" warns on case conditions that
  fallthrough. GCC does have a way to accept conditions that have a
  comment about falling through inside the cases, but the codebase has
  other cases where a single comment explains the fallthrough for the
  whole switch. This warning is being ignored. It was not possible to
  use "-Wno-error=WARNING" because older GCCs fail -- but they do
  accept "-Wno-WARNING" where WARNING is unknown (since at least gcc
  4.6.4).

- "-Wint-in-bool-context" warns (among other things) about result of
  multiplication being used in boolean context. Code was changed to
  use de::max() instead.

- "-Wmaybe-uninitialized" warned about some attributes of a struct not
  being initialized depending on the case. Code was changed to
  initialize the struct in question.

Components: Framework, Vulkan

Change-Id: Iabb26f01e047353c6e2a704ab27b673e85b185be

Fix internal formats conversions

This change resolves Windows compilation errors that
were triggered by the use of snprintf. This function was
replaced with StringTemplate.
This change also fixes few warnings.

Components: OpenGL

Affects:
KHR-GLES3.copy_tex_image_conversions.*

Change-Id: I70c9fdaac3f971e563378a2bcffc65b5386ca914

Replaced deprecated windows version detect

The old windows version reporting API has been deprecated and has been
lying since Windows 8.1 - all windows versions since that version have
been reported as Windows 8.0. (It is possible to add manifest files to
the executable to accept later windows versions, but that would require
the UUID of every existing and future versions of Windows).

There is no direct replacement, as Microsoft would prefer applications
to be written against available features instead of relying on Windows
version numbers.

The official workaround is to look at the version numbers in system DLL
files. In addition, the registry contains several strings which tell the
version number. This change attempts to do both, in case future Windows
versions change this behavior.

Windows can also lie about the bit-ness of the operating system; reading
the windows version from registry from a 32bit application may report
the operating system as 32 bit even if it's actually 64 bit. Separate
64-bit detect function is also included. (Hypothetical future 128 bit
Windows will likely report as 64 bit, but I'll leave that problem to the
future generations).

Affects:

dEQP-VK.info.platform

Components: Framework

VK-GL-CTS issue: 1044

Change-Id: I80062d6eefb14b2ce04818a6c3176db0b172419e

Restrict alpha range for rgba5551 surfaces

The tests were blending against an rgba5551 surface and expecting alpha
to be quantized to 1 bit between every drawcall. The spec does not
forbid implementations to use higher precision internal representations
where this quantisation may not occur to the same extent. This change
restricts fragment outputs to be either 0 or 1, so the issue is avoided

Components: OpenGL

VK-GL-CTS issue: 988

Affects:
dEQP-EGL.functional.render.*rgba5551*

Change-Id: Id8fa5068931124d79ae7fa1b9b8a41845c73e507

Test layout(location) qualifier with opaque types

layout(location) is tested with images and samplers.

Components: OpenGL

VK-GL-CTS issue: 870

Affects:
KHR-GL45.layout_location.*
KHR-GLES31.shaders.layout_location.*

Change-Id: I3abacfe2fbc93860cfd7744f77733189db531dce

teglImageFormatTests: call glFinish after creating the buffer in the client API

According to section 3.7.3.2 (Order of Rendering Operations Between
Contexts) of the EGL spec, the application is responsible for
synchronization between different rendering contexts even in the same
thread.

Some of the tests execute a sequence of operations like this (see
MultiContextRenderTests::init):

1. ctx0: create image
2. ctx0: read from image
3. ctx0: clear image
4. ctx1: recreate image
5. ctx0: read from image
6. ctx1: read from image

This change adds a call to glFinish between steps 4 and 5 that is needed
for correctness. Note that other parts of the same file already call
glFinish, e.g. the various Render actions.

Components: EGL
VK-GL-CTS issue: 857
Affects:
dEQP-EGL.functional.image.render_multiple_contexts.gles2_*_read_pixels

Change-Id: I4dc50f7516374405596653d7006165819b95b6fa

Take account of alignment for data to skip

When making the gradient for packed pixels tests, in some cases it is
required to generate data that will be skip by the operations, like when
using GL_UNPACK_SKIP_IMAGES.

But also the images need to be aligned with the proper
GL_UNPACK_ALIGNMENT, so the data to skip need to account for this
alignment too.

Besides this main fix, a minor fix included is to not increase the
buffer size to account for the skipped images, as the depth already
considered them.

Components: OpenGL

VK-GL-CTS issue: 1017

Affects:
KHR-GLES3.packed_pixels.*
KHR-GL3*.packed_pixels.*
KHR-GL4*.packed_pixels.*

Change-Id: I2fc0626ad2c5bb900db1c67dd1b06bf8a7f25ef9

Merge "Fix ShaderUniformIntegerFunctionTest to not have bool precision"

Fix ShaderUniformIntegerFunctionTest to not have bool precision

Bug: 71875560
Components: AOSP
Affects: dEQP-GLES31.functional.shaders.builtin_functions.uniform.*
Change-Id: I25325aa74090ec253da33ba2677ac281e3a6e01c

eglGetFrameTimestamps: Don't require reads done to be monotonic. am: e8634c9254
am: 24f69aed49

Change-Id: I793d5340447fc5fea2e7e6dfcdc11a4b174812bb

eglGetFrameTimestamps: Don't require reads done to be monotonic.
am: e8634c9254

Change-Id: I68606ee22f516db51b60db205cd7ebb28cc563f6

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

Change-Id: Ib0f3fa2db421ec19b464cb42230a08ae26d9860e

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

Change-Id: I25a00d37394a47b90d7cdf0f51738e71f151f4f2

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

Change-Id: I4453b674dba334a6c547fc694be9639f47226ebe

eglGetFrameTimestamps: Don't require reads done to be monotonic.

Reads done is usually monotonic, but only under the assumption
that every buffer of the surface is being consumed by the same
unit. It is possible for a surface to switch between being an
overlay and being composited by the GPU. It's also possible a
driver will maintain a shadow copy of a frame that will further
affect when reads from a buffer might finish.

Remove the overly strict check.

Bug: 70334851, 74184639
Test: --deqp-case=dEQP-EGL*get_frame_timestamps*
Change-Id: I8e242a9fd649720b4d8059797c71163329b7b92c

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

Change-Id: I45c9f0676f47400c617f7ba1fa6d31812fb5cff6

Update glslang, SPIR-V commits

Components: Framework

Affects: dEQP-VK.*

Change-Id: If5830108278597a28f8043dd947bed3c20c9688d

Add fixed clipping tests to 1.0.3 mustpass

Change-Id: I53fa94455924d1c03a961ce4f35076dc8fc354d5
Components: Vulkan

Allow pop-free behavior in line clipping tests

Existing line clipping tests were modified to allow both popping and
pop-free behavior. The tests support both strict and non strict lines.

Affects:

dEQP-VK.clipping.clip_volume.clipped.wide_lines_axis_aligned
dEQP-VK.clipping.clip_volume.clipped.wide_lines_diagonal

Components: Vulkan

VK-GL-CTS issue: 98

Change-Id: I30301327076ddbd42986b7a58bd3f49786fe993a

Add tests for findLSB/findMSB applied to a uniform value.

Ensure that findLSB/findMSB work correctly when:

Argument is a uniform value, of various precisions
Result is written out directly
Result is compared to a fixed value

Components: AOSP
Bug: 71875560
New Tests: dEQP-GLES31.functional.shaders.builtin_functions.uniform.*

Merged-In: I0dc83b291d1ecaed5e451b76e4b41827116218c6
Change-Id: Iafd784deaf22dd86b6bfc955af967e4482d9e95c

Write only R channel on GL_DEPTH_COMPONENT and OpenGL ES

The affected tests fuzzily compare the results of using sized depth
textures (GL_DEPTH_COMPONENT{16,24,32}) with unsized depth texture
(GL_DEPTH_COMPONENT), which is used as reference.

According to OpenGL ES, the formers write the depth in R channel, (D,
0.0, 0.0, 1.0), while the later writes the depth in the three RGB
channels (D, D, D, 1.0).

Roughly speaking, the image in the former case is red-ish, while the
second is grayscale, and thus the comparison fails.

As only the depth is interested, just modify the test to write the red
channel in the last case.

To allow comparing them, let's just use one channel in the reference.

Components: OpenGL

VK-GL-CTS issue: 999

Affects:
KHR-GLES2.core.internalformat.texture2d.depth*
KHR-GL4*.internalformat.texture2d.depth*

Change-Id: I0bd9eea951eec0dcf91b072c476fbe0ae5292df1

Add test for VkSetHdrMetadata

Affects: dEQP-VK.wsi.*.colorspace.*

Change-Id: I767b19cd94d7387d7df19b3d85553f62ad6f891b
Components: Vulkan

Tests for layered resolve attachments

Add tests:
 * dEQP-VK.renderpass.multisample_resolve.layers_3.*
 * dEQP-VK.renderpass.multisample_resolve.layers_6.*
 * dEQP-VK.renderpass.suballocation.multisample_resolve.layers_3.*
 * dEQP-VK.renderpass.suballocation.multisample_resolve.layers_6.*

Update tests:
 * dEQP-VK.renderpass.multisample_resolve.*
 * dEQP-VK.renderpass.suballocation.multisample_resolve.*

Components: Vulkan

VK-GL-CTS issue: 960

Change-Id: Ic52f813d5590b7a17301016af193256eae1f85b5

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

Change-Id: I3a70f61143bc2f7f2b1e3ad51a6179a776b6dfcc

Use de::min instead of std::min

std::min was used without including algorithm, which breaks build on
at least VS2017

Components: OpenGL ES
VK-GL-CTS issue: 1054

Affects:
dEQP-GLES2.functional.texture.size.*
dEQP-GLES3.functional.texture.size.*

Change-Id: Id3edb237931d6957095cf6b8fcb38359422decd3

Test creating empty descriptor set layouts

Two new test cases were added. The first case creates a descriptor set
layout with zero bindings and without optional flags. The second case
creates a descriptor set layout with zero bindings and uses the
VK_DESCRIPTOR_SET_LAYOUT_CREATE_PUSH_DESCRIPTOR_BIT_KHR flag.

The test passes as long as the descriptor set layout is created
successfully.

New tests:

dEQP-VK.api.descriptor_set.descriptor_set_layout.empty_set.normal
dEQP-VK.api.descriptor_set.descriptor_set_layout.empty_set.push_descriptor

Components: Vulkan, Framework

VK-GL-CTS issue: 1013
VK-GL-CTS public issue: 80

Change-Id: I75abe4e3f0f14902a4d7178e7e763ccdec90dc8c

Extend coverage in subgroups: vote, builtinMaskVar

Affects:
dEQP-VK.subgroups.vote.*
dEQP-VK.subgroups.builtin_mask_var.*

 Add tests:
 dEQP-VK.subgroups.vote.*_frag_helper

Components: Vulkan

VK-GL-CTS issue: 965, 799

Change-Id: I32fe052e4cbfc6d1dfefb9b3be7d1e4d54124b16

Use common utility function for queue submit and wait

Queue submit and wait is being used in almost every test case. Some
test groups already implement an utility function to create a fence,
submit commands and wait for the fence. Others do it inline. These
are now replaced with a single utility function.

Affects:

dEQP-VK.api.buffer_view.access.*
dEQP-VK.api.command_buffers.*
dEQP-VK.api.copy_and_blit.*
dEQP-VK.api.fill_and_update_buffer.*
dEQP-VK.api.get_memory_commitment.*
dEQP-VK.api.smoke.*
dEQP-VK.api.image_clearing.*
dEQP-VK.binding_model.shader_access.*
dEQP-VK.device_group.*
dEQP-VK.draw.*
dEQP-VK.dynamic_state.*
dEQP-VK.fragment_operations.*
dEQP-VK.geometry.*
dEQP-VK.image.*
dEQP-VK.memory.binding.*
dEQP-VK.memory.pipeline_barrier.*
dEQP-VK.multiview.*
dEQP-VK.pipeline.*
dEQP-VK.query_pool.occlusion_query.*
dEQP-VK.query_pool.statistics_query.*
dEQP-VK.rasterization.*
dEQP-VK.renderpass.multisample_resolve.*
dEQP-VK.renderpass.multisample.*
dEQP-VK.renderpass.sampleread.*
dEQP-VK.renderpass.sparserendertarget.*
dEQP-VK.glsl.*
dEQP-VK.spirv_assembly.*
dEQP-VK.ssbo.*
dEQP-VK.synchronization.internally_synchronized_objects.*
dEQP-VK.synchronization.op.*queue.*
dEQP-VK.tessellation.*
dEQP-VK.texture.*
dEQP-VK.ubo.*
dEQP-VK.ycbcr.*

Components: Vulkan

VK-GL-CTS issue: 1015

Change-Id: I27c05269b1c546c7079f8197f84cdc7f2c8bda17

Add tests for vkGetDeviceQueue2

Add tests verifying behavior of vkGetDeviceQueue2 when the provided
flags match and do not match.

Components: Vulkan

New tests: dEQP-VK.api.device_init.create_device_queue2*

Change-Id: I0654a0e4e20e56571220268e65f54a923bec2ff8

Merge "Test effect of transform feedback object switching." into deqp-dev

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

Change-Id: I4e0d4ac0cab7b8cc9ffd010bc6cc53c60b485e88

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

Change-Id: I30ffdaf2b3e30f6ac490ef4bf4e5169d98f0c66c

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

Change-Id: Ibf8696558e773e47e04e0f14e7b0deb566eaa5ce

Unfreeze Vulkan CTS 1.0.3 mustpass

Change-Id: Icee32789f1a86ee873f20139787222d416456bfd

Add a note to the README

Explain that the verification scrpt will report a warning
for a correctly generated git-log

Components: Vulkan

Change-Id: Ie8e0ab090f50c776150f9a7f4ce7d69583186953

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

Change-Id: Ia75241c78980e6bac6358ade016d4849090f8444

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

Change-Id: I28a337678dc76cf49f090fc0ee304b5d7a137cf0

Validation errors fix in copying 3d images

A minor chages to fix validation errors.
Two tests are not passing currently!

Affected tests:
dEQP-VK.api.copy_and_blit.dedicated_allocation.image_to_image.3d_images.*

Components: Vulkan

VK-GL-CTS issue: 208

Change-Id: Id3c5ec869d52db6b1229f45bbeacf24e35e75e48
(cherry picked from commit c23ba94d7978ba84f9ffd2e75b6afbf8e3395633)

Fix: Memory alignment

Affects:
dEQP-VK.memory.mapping*

Components: Vulkan

VK-GL-CTS issue: 1039

Change-Id: Ia5093148fa145c9eecc03fe3b76102cdf77ad472

Spec relaxation for protected YCbCr Implicit chroma reconstruction

Incorporates spec relaxation in the protected memory YCbCr tests which
describes that when doing implicit reconstruction with nearest chroma
filtering, an implementation may behave as if both xChromaOffset and
yChromaOffset were VK_CHROMA_LOCATION_MIDPOINT.

Tests affected:
dEQP-VK.protected_memory.interaction.ycbcr.*

Components: Vulkan

VK-GL-CTS issue: 1032

Change-Id: I25ef589ed05b1b167475eb5b6d782b261b1b3532

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

Change-Id: I43efda824345475ca1305ed3fb2c68e0adb2cc29

Validation errors fix in copying 3d images

A minor chages to fix validation errors.
Two tests are not passing currently!

Affected tests:
dEQP-VK.api.copy_and_blit.dedicated_allocation.image_to_image.3d_images.*

Components: Vulkan

VK-GL-CTS issue: 208

Change-Id: Id3c5ec869d52db6b1229f45bbeacf24e35e75e48

Set usable instance version in external memory tests

These tests were using vkEnumerateInstanceVersion() to get
instance version. This failed, when Vulkan 1.1 Loader is used with
Vulkan 1.0 ICD: the function was returning version 1.1,
so test was not requesting any needed instance extensions, as these
were assumed to be already in core.

This is out of sync with dispatch code dispatching to
1.0 + KHR extension functions, when calling entrypoints like
vkGetPhysicalDeviceExternalBufferProperties(). As no KHR extensions
were requested during instance creation, the behavior is undefined.

The dispatch code is using context.getUsedApiVersion() utility
to guess Vulkan version, so same utility should be used for instance
creation, avoiding raw calls to vkEnumerateInstanceVersion().

Components: Vulkan

VK-GL-CTS Issue: 1047, 854

Affects:
dEQP-VK.api.external.memory.*
dEQP-VK.api.external.fence.*
dEQP-VK.api.external.semaphore.*

Change-Id: Iecf9bb086e04c6d88118fee9c4867ac5647d1ab7