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

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)

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

Change-Id: I2a34679ba766c3637c4937318ba5260193307f5f

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

Change-Id: I3755b4a43827bf4184df7d21382b3a7e6639c67e

Fix Block decorations in 8-bit storage tests

The Vulkan environment spec allows two types of SSBO declarations:

 - Storage class of Uniform with the BufferBlock decoration
 - Storage class of StorageBuffer with the Block decoration

All of the 8-bit storage tests, as far as I can see, have chosen to use
the StorageBuffer storage class.  However, most of the SSBO declarations
are missing the Block decoration entirely.  A few mistakenly have a
BufferBlock decoration.

Affects:

dEQP-VK.spirv_assembly.instruction.graphics.8bit_storage.*

Components: Vulkan

VK-GL-CTS issue: 1519
Change-Id: Id78cae9c1813cd04daa894779aa3fbec57240eb8

Fix further aspect masks in renderpass2 tests

Components: Vulkan
Affects: dEQP-VK.renderpass.suballocation.multisample.*
         dEQP-VK.renderpass2.suballocation.multisample.*

Change-Id: I52262321f00b2f7ffbe47efa4e455c80b6c5d251

Simplify YCbCr result verification

Rather than write all the checks out three times only do it once.

Components: Vulkan
Affects: dEQP-VK.ycbcr.copy.*

Change-Id: I20fb18862c5db88b865da6121c09ac3486882c54

Fix initial YCbCr image layout

When initialising linear tiling images they should be created with
LAYOUT_PREINITIALIZED rather then UNDEFINED.

Components: Vulkan
Affects: dEQP-VK.ycbcr.copy.*

Change-Id: Ie9136ce45cfa1e9f18e83c4655cc762d2211274e

Fix validation of occlusion query tests

The vertex buffer usage was incorrectly set to storage buffer and the
flushing/invalidation was incorrect.

Change-Id: Ic2ff15fae3bf8dcbbf2e5d4d197f775755e933ba
Components: Vulkan
Affects: dEQP-VK.query_pool.occlusion_query.*

Fix more memory flush/invalidation cases

Flushing or invalidating memory based on buffer sizes is not safe
because of the restriction around non-coherent atom size. Use the flush
and invalidate helpers to resolve this problem, fixing many validation
errors.

Components: Vulkan
Affects: dEQP-VK.draw.*
         dEQP-VK.image.*
         dEQP-VK.rasterization.*
         dEQP-VK.pipeline.push_descriptor.*

Change-Id: I7ad4cbc23529d24dad7dc80f29d5451d9157f74c

Validation fixes for device_init tests

The validator requires that the queue properties be queried before
devices and queues are created.

Components: Vulkan
Affects: dEQP-VK.api.device_init.*

Change-Id: Iddf9487703c65890eb0c97acc991d6e0a5c6a72b

Fix denorm tests in float_controls

When binary operations on denorms are executed against Inf or Nan
SignedZeroInfNanPreserve mode should always be enabled as some
implementations may perform optimizations that assume that
arguments are not Nan or Inf. Before this change DenormPreserve
and DenormFlushToZero tests had only those modes enabled. Now
SignedZeroInfNanPreserve is added when second operand is Inf or Nan.

Components: Vulkan

VK-GL-CTS issue: 1517

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

Change-Id: I0b5e0ed010bde200154f58f9fe194fdf7c565803

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

Update glslang

Components: Framework

Change-Id: Idc6041b4dc36737e9c3af9d9985ecab2f8bda811

Merge commit aosp/master into deqp-dev

Change-Id: I074db19a2a5e6e1898a13045bd71e268fed5ffe6

Revert "DO NOT MERGE - Merge pie-platform-release (PPRL.181205.001) into master"

This reverts commit 876c479eb58a96c7d655e42d8afd2aeacf751da2, reversing
changes made to bb802e221330136944030f00dbc7121c15408d6f.

Change-Id: I5830a4475069c496f4542cd779b6b7344eab7c17

Update SPIR-V Tools

Components: Framework

Change-Id: I8b4acf9c7f144f745a2e0ff79ca489eb55edba4e

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

Change-Id: I2931dfff2ea3c8df38fdf56b8df9f6e146d1a920

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

Change-Id: Ic63a11b7b47f13bb1ab350afbeeec9059da4ff9d

Remove multiple Offset decorations

Fixes unintentional duplication of the Offset decoration
for the first struct member

Components: Vulkan

Affects: dEQP-VK.spirv_assembly.instruction.compute.opcopymemory.struct

Change-Id: Id13a8f15a72d7dd32d13fb52a494d57349cea289

Added tests to intentionally mismatch opImageType

These tests intentionally mismatch the opImageType of the shader with
the source image types. The results of execution are ignored, but the
driver and/or graphics device is not expected to crash.

For the graphics tests, the source image format is changed, and
for the compute tests the shader is changed.

New tests:

dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.*.optypeimage_mismatch.*
dEQP-VK.spirv_assembly.instruction.compute.image_sampler.*.optypeimage_mismatch_*

Components: Vulkan

VK-GL-CTS issue: 1420

Change-Id: I6f86986fda108b0dafc1a10d1790414d7bd5a2ba

Fix multiple ArrayStride decorations in float_controls

Most of float_controls tests that were executed on fragment stage
had an extra ArrayStride decorations for the same arrays. This change
corrects SPIR-V generation so that second set of ArrayStride
decorations is added only when tested operation input type is
different from operation result type.

Components: Vulkan

VK-GL-CTS issue: 1522

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

Change-Id: I3e72414e00bd82ff7a9975ff9148e9889bb32246

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

Change-Id: Ib44f63dc9bbc3da62777baea5bbf22b19151aec8

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

Change-Id: I295ae6f8a1549a687bf58eabc8fa38052a2bad48

Optimizing reading in host_read_host_write.1048576

To optimize reading the "for loop byte by byte" was replaced
by memcpy for memory equal and larger then 1 MB.

Components: Vulkan
VK-GL-CTS issue: 1515
Affects:
dEQP-VK.memory.pipeline_barrier.host_read_host_write.1048576

Change-Id: I02f2c3c90d2fc554597f8892a77da6106062452d

Remove duplicate tests under dEQP-VK.renderpass*

Some of the tests are located under both, dEQP-VK.renderpass* and
dEQP-VK.renderpass*.suballocation.* test groups. This removes
the duplicate cases directly under dEQP-VK.renderpass* and moves
the test cases that use sub-allocation into
dEQP-VK.renderpass*.suballocation.* test group.

Components: Vulkan

VK-GL-CTS Issue: 1491

Affects:

dEQP-VK.renderpass.multisample.*                       (removed)
dEQP-VK.renderpass.multisample_resolve.*               (removed)
dEQP-VK.renderpass.subpass_dependencies.*              (removed)
dEQP-VK.renderpass.sampleread.*                        (removed)
dEQP-VK.renderpass2.sampleread.*                       (removed)
dEQP-VK.renderpass.sparserendertarget.*                (removed)
dEQP-VK.renderpass2.sparserendertarget.*               (removed)
dEQP-VK.renderpass.unused_attachment.*                 (removed)
dEQP-VK.renderpass2.unused_attachment.*                (removed)
dEQP-VK.renderpass.suballocation.sampleread.*          (added)
dEQP-VK.renderpass2.suballocation.sampleread.*         (added)
dEQP-VK.renderpass.suballocation.sparserendertarget.*  (added)
dEQP-VK.renderpass2.suballocation.sparserendertarget.* (added)
dEQP-VK.renderpass.suballocation.unused_attachment.*   (added)
dEQP-VK.renderpass2.suballocation.unused_attachment.*  (added)

Change-Id: I99613fb5a0074877932974039ac01dfbdb80dc0d

Add more conditional rendering tests

Components: Vulkan

VK-GL-CTS Issue: 1485

New Tests: dEQP-VK.conditional_rendering.draw_clear.*

Change-Id: I9b58ab0e5c4d7519b452b25199ee2de0f090e3a1

Fixes binding model test format mismatch

Storage texel buffer had a different format in
the shader (R32I) than in the app (R32UI).

Components: Vulkan
VK-GL-CTS issue: 1514

Affects:
dEQP-VK.binding_model.descriptorset_random.*

Change-Id: Iffbffbd002a77387dbe008e53ebb81d3f28838f5

Fixes AHB test out of memory error

Handles out of memory as not supported

Components: Vulkan
VK-GL-CTS issue: 1507

Affects:
dEQP-VK.api.external.memory.android_hardware_buffer.image_formats.*

Change-Id: If46d2fd7a446f8348aa11bb1636635c7d1f3660a

Remove Clip/CullDistance when not needed

Although the tests don't need clip or cull distances the capabilities
are enabled, which means that features would need to be enabled. Remove
the capabilities.

Components: Vulkan
Affects: dEQP-VK.spirv_assembly.instruction.*.float_controls.*
         dEQP-VK.spirv_assembly.instruction.compute.multiple_shaders.*

Change-Id: If8e18b1608f4d8d2782e9010c531a4e20119b414

Add tests using VkRunner, precompiling the scripts

This is an alternative version of change 2660 that precompiles the
GLSL or SPIR-V assembly in the VkRunner test scripts to SPIR-V binary
during the build phase so that they can be executed faster and without
any external glslang or spirv-tools dependencies.

This adds VkRunner as an external dependency and then adds some
example test cases to use it. VkRunner is a tool designed to make it
easier to create shader tests for Vulkan by using a simple scripting
language which combines the shaders used for a pipeline along with a
description of the inputs and expected outputs.

VkRunner is fetched from an external repository so the fetch_sources
script needs to be run after merging the patch.

VkRunner tests can be added to any group, the tests need to be a child
class of the VkRunnerTestCase class.

Some VkRunner tests are provided as example, but they are excluded from
the mustpass:

   dEQP-VK.vkrunner-example.*

Shader test files are under external/vulkancts/data/vulkan/vkrunner/ path.
The example shader_test files are under example/ subfolder.

VK-GL-CTS: 1147
Change-Id: I1c5fc5f672a360800bfe10afbf96ca8fd07b38e6

Add some watchdog touches in AHB image format tests

Timing on these tests is marginal if the device runs into memory trouble
and needs to spend a lot of time reclaiming to satisfy the allocations.

Change-Id: If397b95b3be02553b3bca2baf43b21b245136b6d
Affects: dEQP-VK.api.external.memory.android_hardware_buffer.image_formats
Components: Vulkan

Fix API usage errors in fragment operation tests

Validation layers catch multiple API usage errors
in fragment operation tests.

Fixes API usage errors caused by unsafe memory flush
and unsupported combinations of image memory barrier
dstAccessMask and dstStageMask.

Components: Vulkan

Affects: dEQP-VK.fragment_operations.*

VK-GL-CTS issue: 1471

Change-Id: I8647478082e9ca74a267bef96fd3e806c6b80dc4

Fixed incorrect load/store ops for depth in multiview test

The test expected the initial depth values to be preserved after the end of the
render pass, which wasn't guaranteed with the given load/store ops.

Components: Vulkan
VK-GL-CTS issue: 1227
Affects: dEQP-VK.multiview.depth.*

Change-Id: I1a42394c8d0f329b43ddde9f40e4a993122eddf9

Add subpass external dependency to pushDescriptor tests

Back to back render passes are used with no pipeline barrier
or external subpass dependency to synchronise the output of
the first to the input of the second.

This change adds an external subpass dependency to ensure that
the render passes are synchronised.

Component: Vulkan
Affects: dEQP-VK.pipeline.push_descriptor.*input_attachment

VK-GL-CTS issue: 1467

Change-Id: Ie42ec31ab393439e7a4c312cfe6dcc40fde82c25

Reduce devices created in device_group tests

The device_group tests could sometimes hit timeouts on
slower CPUs. This change reduces the number of objects
created in these tests to match the reduced number used
in the device and instance tests.

Affects:
dEQP-VK.api.object_management.
multithreaded_per_thread_resources.device_group
dEQP-VK.api.object_management.
multithreaded_shared_resources.device_group

Components: Vulkan

VK-GL-CTS issue: 1512
Change-Id: Icb68643ff530a8b1a1a7daa9fc93d8db8bc5faf9

Remove duplicate identical make*MemoryBarrier

This change adds a new util, vkBarrierUtil, which contains barrier
creation functions. The framework had 8 identical copies of these
functions. Apart from the identical ones, couple other barrier
making functions were also moved into the new util to have them
in the same place. The two makeImageBarrier functions were merged
using default parameters.

The affects-list is overly broad, but not easily measurable since the
changes may affect tests outside the directly edited files.

Affects:

dEQP-VK.*

Components: Vulkan, Framework

VK-GL-CTS issue: 1472

Change-Id: I761f9885bb6bce9a64526e498b9182c5064551ed

Allow SRGB conversion for store tests to generate reference image.

If the driver supports feature VK_FORMAT_FEATURE_STORAGE_IMAGE_BIT for
SRGB format then SRGB coversion is required to generate correct reference
values. Stores tests are using view format (SRGB format) for reference
image, and to generate correct reference image SRGB conversion is required.

Affects:
dEQP-VK.image.mutable.2d.*

Components: vulkan

VK-GL-CTS issue: 1501

Change-Id: I2ca657d53142f84f27390845539e84a6ea5e2d63

Fix flag dEQP-VK.*.record_simul_use_primary

Flag VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT
should be used in VkCommandBufferBeginInfo when
a buffer is resubmitted to a queue while it is
in the pending state,

Components: Vulkan

Affects: dEQP-VK.api.command_buffers.record_simul_use_primary

VK-GL-CTS Issue: 1500

Change-Id: Id527cc2fa54de2675514f89b5f44af221f42bd80

DEQP: Fix the build targets without SPIRV tools

VK-GL-CTS issue: 1516

Affects: Any build without SPIRV tools

Components: Framework

Change-Id: I7bdbcb94a0877e51d2ae39018fb80c84b2924607

Add tests for varying naming

Some platforms were reported to incorrectly link varyings
between shader stages by varying name (OpName) instead
of location ID. These new tests make sure the name or
the lack of it doesn't affect the end results.

New tests:

dEQP-VK.spirv_assembly.instruction.graphics.varying_name.*

Components: Vulkan

VK-GL-CTS issue: 1407
Change-Id: I04aaaf7e0addce6f608875f1646ea71ad249a65e

Fix subgroup shuffle tests to do testing

Modify the subgroup shuffle tests to modify the indices so that they're
in range, rather than just rejecting all the values that were out of
range. In my testing of the old method, every test was skipped so the
tests always passed.

Components: Vulkan
VK-GL-CTS issue: 1490
Affects: dEQP-VK.subgroups.shuffle.*

Change-Id: I51d4511aef60c37f39c634be9dbeaff9aca4717e

Remove duplication from subgroup shuffle tests

The code was very repetitious, meaning edits to the tests had to be done
in numerous places. Consolidate all the different copies together so
that they are easier to fix.

Components: Vulkan
Affects: dEQP-VK.subgroups.shuffle.*

Change-Id: I95d864c72dcd4e070f26bc73cf97920c9a1fe088

Update code comments around --deqp-renderdoc

No workarounds are needed or implemented at this moment.

Components: Vulkan, Framework

Change-Id: I967f211fb64088b75a2e32bbeceb4fa029cf2efd
Affects: None.

Correct copyright header

Components: Vulkan, Framework

Change-Id: I58107231871c5997d2feaf38e6633ed426288c00
Affects: None

DO NOT MERGE - Merge pie-platform-release (PPRL.181205.001) into master

Bug: 120502534
Change-Id: Ibae9975bacb16812d4ee4d0602e151e8683921f3

Don't fail DisplayModeProperties2 with no display

Instead, we should mark this test as not supported.

VK-GL-CTS issue: 1513
Components: Vulkan

Change-Id: I16a6dcb9f10997b5bbc24ffd2c2d7b0300f43f4c
Affects: dEQP-VK.wsi.display.get_display_mode_properties2

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

Change-Id: I53fafcbc72de554b0a8485c83f9f16e5a2fc595c

Set the correct "unused" attachment initialLayout

Before the renderpass the "unused" attachment image is cleared
to a known value and left in the VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL
layout. However, the initialLayout field for the "unused"
attachment is set to VK_IMAGE_LAYOUT_UNDEFINED, which might
cause the image contents to be perturbed when the image
is transitioned from VK_IMAGE_LAYOUT_UNDEFINED to the
finalLayout of VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL.

I understand that part of the purpose of these
"unused_attachment" tests is to ensure "unused" attachments
aren't modified by the renderpass. However, there is nothing
in the Vulkan spec that says the layout of "unused"
attachments should be ignored. This was discussed in issue:
https://github.com/KhronosGroup/Vulkan-Docs/issues/821

And the conclusion is that the layout must: not be ignored
for "unused" attacments.

Affects:
dEQP-VK.renderpass*.unused_attachment.*

Components: Vulkan

VK-GL-CTS issue: 1129

Change-Id: Ia6db607538cb191bc8ca6e03f8da9c88bac031c4

Add framework function createInstanceWithExtensions

Several tests have their own implementation for creating an
instance with extension. Combined these into a framework
function.

Affects:

dEQP-VK.api.device_init.*
dEQP-VK.api.info.*
dEQP-VK.compute.basic.*
dEQP-VK.memory.allocation.*
dEQP-VK.sparse_resources.*
dEQP-VK.ycbcr.format.*

Components: Vulkan

VK-GL-CTS issue: 1470
Change-Id: Idf40f65b18f5f3c84313f22ef023f2ca9aa6ea75

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

Change-Id: I6b9a7542624d1899ed00f694e7f87c1a40252280

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

Change-Id: I02792923c734edfd6749ff9aea25516c5c4e2a80

Add support for RenderDoc in Vulkan tests

This add --deqp-renderdoc=enable commandline, which adds
start frame / end frame signalization using RenderDoc 'in-application' API.

Components: Vulkan, Framework

VK-GL-CTS Issue: 1444

Change-Id: Ib9e8a978cecb96784405ec32495fcb2b156540be

Skip fconvert16 if float16 not supported

The SPIR-V instruction test for float16 wasn't checking for the float16
capability correctly so didn't work properly on implementations that
don't support it.

Components: Vulkan
VK-GL-CTS issue: 1499
Affects:
    dEQP-VK.spirv_assembly.instruction.compute.opspecconstantop.fconvert16

Change-Id: Id8dfc596acda41da670b2d6c08bde9920df99268

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

Change-Id: Ic9da2ef7da15b1d5e72ed892c753a26e084fcd49

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

Change-Id: I0351d45fc4e1069de4ab67dae4df820eb76e5785

Merge "Move APK install/uninstall to tradefed target_preparer"

Update glslang

Components: Framework, Vulkan

Change-Id: Ic2ecbd6915b8f4d6ecfc6a291df0791ddf5167e8

Add precision error margin to cos() tests for float_controls

On some old Intel hardware, Mesa doesn't return exactly 1.0 for cos(0).
This change implements the precision error margin for the affected cos() tests,
following Vulkan spec, "Precision and Operation of SPIR-V Instructions".

Components: Vulkan

Affects: dEQP-VK.spirv_assembly.instruction.*.float_controls.*.cos_denorm_flush_to_zero*

VK-GL-CTS issue: 1498

Change-Id: I2ff07616f706d4214517917276aeb26d2d875325
Signed-off-by: Samuel Iglesias Gonsálvez <siglesias@igalia.com>

Remove unsupported image load store tests

CL https://gerrit.khronos.org/#/c/2639/ added more
formats to the dEQP-VK.image.mutable.* tests. Variants
of these tests use image load store. However, the new
formats added are not all compatible with image load
store as defined by:
https://www.khronos.org/registry/vulkan/specs/1.1/html/vkspec.html#spirvenv-image-formats

This CL removes test variants that use image load store
and the invalid formats from the mustpass list.

Affected tests:
dEQP-VK.image.mutable.*
dEQP-VK.image.swapchain_mutable.*

Components: Vulkan

VK-GL-CTS issue: 1495

Change-Id: I49867913ba5c4e50b4f3d85d7c6787dc60113e63

Fix image feature checks for swapchain_mutable.*

This fix mirrors what was already done in
https://gerrit.khronos.org/#/c/2799/ for the other
image.mutable.* tests. Without this fix the
DE_FATAL("Invalid download method") assert will fire
for some of the sub tests and the test won't be
properly skipped when the platform doesn't support
the format.

Components: Vulkan

Affects: dEQP-VK.image.swapchain_mutable.*

VK-GL-CTS Issue: 1295

Change-Id: I009baa8697ae09f7e9b8001b90159658ced4c62a

Validation: Use correct aspects in renderpass2

Many of the tests were not setting up the aspect mask for input
attachments, meaning that the renderpass2 tests (for which it is
required) were failing validation.

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

Change-Id: I7ca22cf1958a31137d1fb32ef84f944704354fb9

Extend OpCompositeInsert test

Extended OpCompositeInsert test with vector and nested
struct cases.

Affects:

dEQP-VK.spirv_assembly.instruction.*.composite_insert.*

Components: Vulkan

VK-GL-CTS issue: 1458

Change-Id: Icef6dd3161581ddb8b7413190b77833aa02461f1

Fix stray whitespace in external memory tests
am: d1de07835b

Change-Id: I5b0de064757de64913dff55dc274d6e9213cf20f

Fix dEQP-GLES3.functional.fbo.multiview.*
am: 5f9c92ae8f

Change-Id: Ib3c8b65a753246a3b0f062dd0bc1cbbafaafac30

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

Change-Id: I993281855ca306c2d1050c3ef712e639609b8e88

Move APK install/uninstall to tradefed target_preparer

We shouldn't be doing this by hand with a modern tradefed. The previous
implementation was also insanely wasteful -- we'd uninstall and
reinstall the target package every 1K tests.

V3: Adjust testrunner unit tests to not expect installs/uninstalls to
happen
V4: Remove time claim. There is still a significant pause between batches.

Change-Id: Ida67984ce45e60eb025fabfd997c7a9015496333
(cherry picked from commit c2b068c5a09f883f3a6af0e7d4aff639e8f8a070)
Merged-In: Ida67984ce45e60eb025fabfd997c7a9015496333

Fix ubo.random to only use scalar_block_layout in 'scalar' tests

Components: Vulkan
Affects: dEQP-VK.ubo.random*

VK-GL-CTS issue: 1493

Change-Id: I1f5c9b3d8269973c33de1908673d942e67ea3184

dEQP: change sdk version to test_current

The current dEQP tests running with cts-tradefed will pop up a window
saying 'was built for an older version of Android'. This change fixes
that issue.

Bug: 120244829
Test: CtsDeqpTestCases
Change-Id: I3ccc64f2784a071d49bfe2d7b60350d518980e68

Fix stray whitespace in external memory tests

Trivial, but causes check_build_sanity.py script to spuriously fail on
this branch.

Change-Id: I7eedbc2c4766ef4384a46c5c272ed4fcc693134c

Fix dEQP-GLES3.functional.fbo.multiview.*

When the width of render target is an odd value, these tests failed.

The reason is these tests set default sample parameters and sample from
a 512x512 surface, where the left/right views have different colors.
When the width of render target is an odd value, the color of central
line is undefined.

As these tests' purpose is mainly want to verify whether the left/right
views have the different color, but not exactly care what the central
line is, a simple solution is just skip the check of central line.

Components: OpenGL

VK-GL-CTS issue: 1256

Affects: dEQP-GLES3.functional.fbo.multiview.*

Change-Id: I62c86dc7269864a3919b11a331178bfc54d65ab0
(cherry picked from commit fb74c54eb8ab5776fc2cd2c6d0eff422a344ec78)
Bug: b/120491519