Merge branch 'jekstrand_copy_and_bit_usage_flags' into 'master'

copy_and_blit: Fix image/buffer create-time usage flags

For buffers, it had both source and destination marked as DST.  For images,
it both uploads and downloads the images, so it needs both set.

Fixes #199

See merge request !309

Merge branch '194-fix-builtin-precision-testcase-logs' into 'master'

move statement and definition logs to test instance, fixes #194

See merge request !308

Merge branch 'copy_image_usage_flag_dst_issue' into 'master'

Fix image usage flags used in copy and blit test, fix regression from MR 180

I didn't pay attention with merge request 180 and the second getPhysicalDeviceImageFormatProperties had the wrong image usage flags.

This should fix the issue.

See merge request !299

Merge branch '158-fix-cmdbuf-query-index' into 'master'

Change the incorrect entry index in cmdBeginQuery

According to the Vulkan 1.0 Specification the value of entry has to be
less than the number of queries. Three test cases used an incorrect
value of entry (equal to the query count).

Fixes #158

See merge request !288

Merge branch '162-163-fix-timestamp-issues' into 'master'

Fix 2 issues in pipeline.timestamp tests (#162,#163)

for issue #162, set aspect flag when clearing images.
for issue #163, add optimal tiling feature check when creating images.
Also fix some typo.

See merge request !272

Merge branch 'marka_arm_calling_convention' into 'master'

Use aapcs-vfp calling convention for ARMv7A

This is required to match the calling convention specified in
vk_platform.h. Tests using functions with float parameters were failing
when built for 32-bit ARM since the values were in different registers
than expected.

In particular, most of the dEQP-VK.dynamic_state.* tests were failing since the driver was seeing incorrect parameter values.

See merge request !331

Merge branch 'marka_pipeline_float' into 'master'

Use memcpy to copy floats for vertex input

The destination buffer may not be aligned as required to
cast as a float* or deFloat16*. This was causing crashes
due to bus errors on 32-bit ARM systems in several of the
dEQP-VK.pipeline.vertex_input.multiple_attributes.* tests.

See merge request !333

Coding style fixes

Merge branch '197-206-vertex-input-fixes' into 'master'

dEQP-VK.pipeline.vertex_input.*: check 64-bit float support, use correct alignment in vertex buffers

This should fix issues #197 and #206.

See merge request !332

Merge branch 'jbolz_fp16denorm' into 'master'

Flush fp16 denorms to zero for format_reinterpret tests.

Similar to what is already done for fp32 values, when int16 values are reinterpreted as fp16 they can include denorms which may be flushed to zero.

See merge request !307

Use memcpy to copy floats for vertex input

The destination buffer may not be aligned as required to
cast as a float* or deFloat16*. This was causing crashes
due to bus errors on 32-bit ARM systems in several of the
dEQP-VK.pipeline.vertex_input.multiple_attributes.* tests.

dEQP-VK.pipeline.vertex_input.* fixes:
- Check if 64-bit float types are supported.
- Use correct alignment of vertex attributes.

Use aapcs-vfp calling convention for ARMv7A

This is required to match the calling convention specified in
vk_platform.h. Tests using functions with float parameters were failing
when built for 32-bit ARM since the values were in different registers
than expected.

Merge branch 'marka_flush_offset' into 'master'

Fix offset parameter for flushMappedMemoryRange

In ShaderRenderCaseInstance::uploadImage2D(), the allocation object was
being passed in instead of the offset. This resulted in attempting to
flush an invalid memory range.

This was resulting in intermittent crashes in the dEQP-VK.glsl.discard.* tests.

See merge request !329

Merge "Fix missing/error entries in unsupported table" into vulkan

Fix missing/error entries in unsupported table

Change-Id: Ibf0d21a7573dd4cd1ce1460df67a112071ebd2c9

Update Vulkan CTS README

 * Build instructions for Windows, Linux, and Android
 * Command line options for official CTS runs
 * Other misc improvements

Fix offset parameter for flushMappedMemoryRange

In ShaderRenderCaseInstance::uploadImage2D(), the allocation object was
being passed in instead of the offset. This resulted in attempting to
flush an invalid memory range.

Merge branch 'shaderExecutor-descriptorSet' into 'master'

ShaderExecutor: fixed FragmentOutExecutor that tried to create empty DescriptorPool

Spec says (on VkDescriptorPoolCreateInfo): The value of poolSizeCount must be greater than 0

See merge request !221

Merge branch 'awoloszyn-variable-and-modules-fixes' into 'master'

Fixes for module color inversion and placement of OpVariable Instructions.

Contains two fixes:

 * Follow the SPIR-V universal validation rule: OpVariable instructions must precede (most other kinds of) instructions. So don't interleave the OpVariable with OpStore. It's likely implementations are forgiving in this respect, but better to do it correctly. Discovered during code review of something else.

 * Always write 1.0 into the alpha channel. This fixes #211

See merge request !325

Moved all OpVariables with Function scope to the top of functions.

Some OpVariables were not at the top of Functions which is invalid.

Fixed color inversion in multi-module tests.

Now force the alpha value to 1 in the test so that output colors
are kept.

Fixes #211.

Merge "Add ULP part of limit for PointSize/LineWidth/Interpolation offset" into vulkan

Add ULP part of limit for PointSize/LineWidth/Interpolation offset

Fixes #172

Change-Id: Ic87495c9b8806993f77412b4cc3414fa284e2d29

Merge branch 'jbolz_compareFRem' into 'master'

Add 'compareFRem' function verifyIO function

Use an approximate comparison for FRem.

See merge request !316

Merge branch 'awoloszyn-spirv-fixes' into 'master'

Fix issues in SPIRV graphics.decoration_groups and graphics.opphi tests.

See merge request !324

Fixed constants spirv_assembly.instruction.graphics.opphi.out_of_order

Fixes #209

Fixed spirv_assembly.instruction.graphics.decoration_group tests.

The tests were incorrectly calculating input + 2 - 1, and expecting
no change. Now they calculate intput + 2 - 2.

Fixes #213

Add 'compareFRem' verifyIO function

Merge branch 'jbolz_opquantize' into 'master'

Fix various OpQuantize test bugs

See merge request !315

Merge branch 'alloc-callback-erase-ptrToSlotIndex' into 'master'

Fix allocation callback validation

Allocation callback validation assumed incorrectly that alloc or realloc would never return same pointer twice. This fixes that assumption.

See merge request !283

Merge branch 'mchomicz_spirv_assembly_opcopyobject_spotcheck' into 'master'

Fix dEQP-VK.spirv_assembly.instruction.compute.opcopyobject.spotcheck

Shader in the test extracts an column from a matrix (%c_fmat_copy) , but the result type of the operation is said to be a float scalar.

See merge request !323

fix typo update realloc path

Merge branch 'scygan_pipelineVertexInput_64bit_locations' into 'master'

pipeline/vktPipelineVertexInputTests: fixed attribute location for double formats

The tests uses 3- and 4-element double (fp64) vertex attributes, but does not provide enough location space for these.

According to spec:
```
 Input attributes which have three- or four-component 64-bit formats will consume two consecutive locations.
```

See merge request !275

Merge branch '198-input-assembly-check-topology-support' into 'master'

dEQP-VK.pipeline.input_assembly: check if the primitive topology under test is available.

This should resolve issue #198.

See merge request !322

Merge branch '143-remove-gl-arb-extensions' into 'master'

Remove GL_ARB_ extensions from ShaderRenderCase- and ShaderExecutor-based tests.

Fixes #143

See merge request !320

Merge branch '207-208-basic-compute-bugfix' into 'master'

Basic compute: fix wrong enum and uninitialized atomic variable

Fixes #207 and #208

#207 was partially fixed by !281, but the same initialization logic was also needed in SSBO barrier test, so I'm adding it here.

See merge request !318

Merge branch 'draw-tests-missing-flags-fix' into 'master'

Draw tests: added missing flags in create info structures.

See merge request !313

Merge branch 'jbolz_createBuffer' into 'master'

Skip createBuffer tests that exercise an invalid combination of create flags

See merge request !311

Merge branch 'jekstrand_renderpass_transfer_bit_fix' into 'master'

dEQP-VK.renderpass: Set IMAGE_USAGE_TRANSFER_SRC_BIT when needed

Later on in the test, if "lazy" is not set, we try and download the image.

See merge request !266

Merge branch 'awoloszyn-fix-opname' into 'master'

Fixed typo in OpName instruction.

Fixes #137

See merge request !310

Merge branch '129-remove-shader-executor-builtin-constants-tests' into 'master'

Remove dEQP-VK.glsl.ShaderExecutor.builtin_constants tests.

Fixes #129

See merge request !321

pipeline/vktPipelineVertexInputTests: fixed attribute location for double formats

fix dEQP-VK.spirv_assembly.instruction.compute.opcopyobject.spotcheck

Shader in the test extracts an column from a matrix (%c_fmat_copy) , but the result type of the operation is said to be a float scalar.

dEQP-VK.pipeline.input_assembly: check if the primitive topology under test is available.

Remove dEQP-VK.glsl.ShaderExecutor.builtin_constants tests.
Fixes #129

Remove GL_ARB_ extensions from ShaderRenderCase- and ShaderExecutor-based tests.
Fixes #143

Basic compute: fix wrong enum and uninitialized atomic variable

Issue #207 and #208

Fix various OpQuantize test bugs

Merge in changes from Khronos repository

Merge "Add initial Vulkan CTS 1.0.0 mustpass" into vulkan

Merge "Refactor mustpass script" into vulkan

Merge branch 'awoloszyn-fix-double-definition' into 'master'

Removed double-defined %fvec3

dEQP-VK.spirv_assembly.instruction.compute.opcopyobject.spotcheck
and
dEQP-VK.spirv_assembly.instruction.compute.opconstantnullcomposite.spotcheck

Contained a double-defined %fvec3 one in s_CommonTypes and one in the test.

See merge request !314

Removed double-defined %fvec3

Add initial Vulkan CTS 1.0.0 mustpass

Change-Id: Ic91aa817944c856ce18af46909cfee3c04fd7e20

Refactor mustpass script

Move actual configuration-independent implementation to
scripts/mustpass.py

Change-Id: I706b08379db2d4b17caecac192cbace4aa6e718a

Draw tests: added missing flags in create info structures.

Update glslang

Change-Id: I4056cc5632cf68ba57aea4928ca3e24a790d13e0

Skip createBuffer tests that exercise an invalid combination of create flags

Fixed typo in OpName instruction.

Fixes #137

copy_and_blit: Fix image/buffer create-time usage flags

For buffers, it had both source and destination marked as DST.  For images,
it both uploads and downloads the images, so it needs both set.

Fixes #199

make new allocation live and update the record

Remove unused variable.

Merge in changes from Khronos repository

Merge branch 'scygan_astc_fix' into 'master'

pipeline.image.view_type.2d.format.astc: fix compression parameters for  ASTC formats

Looks like there is an error in tcuCompressedTexture.hpp file. Here:
TexDecompressionParams (AstcMode astcMode_ = ASTCMODE_LAST) : astcMode(astcMode_) {}
ASTCMODE_LAST is set instead of ASTCMODE_LDR.

This causes an assert in tcuCompressedTexture.cpp file:
DE_ASSERT(params.astcMode == TexDecompressionParams::ASTCMODE_LDR || params.astcMode == TexDecompressionParams::ASTCMODE_HDR);

See merge request !276

Merge branch 'scygan_compute_to_small_dset' into 'master'

dEQP-VK.compute.indirect_dispatch.*  - to small descriptor pool

Descriptor pool is created with not enough storage for a used descriptor type increased proper descriptorCount to match the number of allocations in test.

See merge request !306

Merge branch 'jbolz_vertexIndex_instanceIndex' into 'master'

Search/place VertexID->VertexIndex and InstanceID->InstanceIndex

Fixes #181.

This is all just simple search and replace. None of these files used non-zero firstVertex/firstInstance.

I did some testing of the affected tests, but I can't run all the spirv assembly tests.

See merge request !298

Merge branch 'jbolz_compute' into 'master'

Fix compute tests: UBO/SSBO layout mismatch, and uninitialized atomic counter value

ubo_to_ssbo tests had mismatching layout (UBO defaults to std140, SSBO defaults to std430). Since there are tests for UBO->SSBO and SSBO->SSBO, easiest thing to do is to consistently use std140 layout.

ImageBarrier tests did not initialize the output buffer, which is written via atomic adds.

See merge request !281

Merge branch 'fix-unsupported-result-in-api-buffer-tests' into 'master'

Fixing buffer test results if sparse memory is not supported.

Changed the results of the buffer tests from 'pass' to 'not supported' if the sparse memory is required but the hardware does not support it.

See merge request !302

Merge branch 'awoloszyn-fix-opquantize-tests' into 'master'

Fix OpQuantizeF16 Spirv-assembly tests.

Fixes indexing error in constructNormalizedFloat.
Fixes incorrect constants being used in some tests.

Fixes #193

See merge request !290

move statement and definition logs to test instance, fixes #194

merge master

Merge "Bug fixes to render pass tests." into vulkan

Flush fp16 denorms to zero for format_reinterpret tests.

fix compression parameters for ASTC formats

Merge branch 'scygan_spirv_assembly-opunreachanble_bad_ptr_type-pr617' into 'master'

Fix dEQP-VK.spirv_assembly.instruction.compute.opunreachable.all

Fix hand-written SPIR-V: const int was passed to a function taking a pointer as a parameter

See merge request !305

Merge branch 'scygan_spirv_assembly-wrong_type_of_struct_member_pr620' into 'master'

Fix dEQP-VK.spirv_assembly.instruction.graphics.opconstantcomposite.struct* - wrong type of struct member

The %fp_v4f32 structure has two members: [0] of vec4 type, and [1] of float  type. However OpAccessChain for [1] member, which is a float, so as subsequent OpLoad use vec4 as type.

See merge request !304

Merge branch 'jekstrand_spirv_assembly_fixes' into 'master'

Various fixes for the spirv_assembly tests

This fixes all of the problems I've found so far with the SPIR-V assembly tests.

I think this should be enough to fix #184.

See merge request !301

Descriptor pool is created with not enough storage for a used descriptor type - increased proper descriptorCount to match the number of allocations in test.

Fix dEQP-VK.spirv_assembly.instruction.compute.opunreachable.all

Fix hand-written SPIR-V: const int was passed to a function taking a pointer as a parameter

Fix dEQP-VK.spirv_assembly.instruction.graphics.opconstantcomposite.struct* - wrong type of struct member

The %fp_v4f32 structure has two members: [0]  of vec4 type, and [1] of float  type. However OpAccessChain for [1] member,
 which is a float, so as subsequent load use vec4 as type

Fixing buffer test results if spares memory is not suppoerted.

Switch to libpng_ndk to remove platform library dependency am: cf4407563b am: aa4099c48f
am: fdd37578e7

* commit 'fdd37578e709ea65535aab80038a6ea4704384fa':
  Switch to libpng_ndk to remove platform library dependency

Switch to libpng_ndk to remove platform library dependency am: cf4407563b
am: aa4099c48f

* commit 'aa4099c48f58c93d951b64451ef87adb31fce406':
  Switch to libpng_ndk to remove platform library dependency

Switch to libpng_ndk to remove platform library dependency
am: cf4407563b

* commit 'cf4407563b4783b69214f3542728806468b200a6':
  Switch to libpng_ndk to remove platform library dependency

vktSpvAsmInstructionTests: Add array strides to OpSpecConstant tests

Merge "Do not normalize binary path in xs::PosixTestProcess"
am: 5231f5a8b9

* commit '5231f5a8b9a6a3f177033157b83bf92353d7864d':
  Do not normalize binary path in xs::PosixTestProcess

Use device limits in api object management tests.

Change-Id: Ibad7fa9df8fb5ad8eec7298ac360585f2f3e0c2f

Merge "Do not normalize binary path in xs::PosixTestProcess"

Merge branch 'jbolz_mapBorderColor' into 'master'

Fix mapping of float border colors

See merge request !300

Fix mapping of float border colors

Switch to libpng_ndk to remove platform library dependency
am: f6346adae0

* commit 'f6346adae0294a30fd71bd38ac4e11a68732af6b':
  Switch to libpng_ndk to remove platform library dependency

replace VK_FORMAT_FEATURE_BLIT_DST_BIT by VK_IMAGE_USAGE_TRANSFER_DST_BIT in vkGetPhysicalDeviceImageFormatProperties

Switch to libpng_ndk to remove platform library dependency

Bug: 26513978

(cherry picked from commit f6346adae0294a30fd71bd38ac4e11a68732af6b)

Change-Id: If2e08e80be97c3c1b2dbabf9fd482f2bf5179540

Bug fixes to render pass tests.

- Check that subpasses never use more than maxColorAttachments color
  attachments. Fixes #196.
- Handle rounding and quantization in checkColorRenderQuad().
  Fixes #64.
- Add attachment count group for 4.

Change-Id: Ib218ddef344b8b626f822db1cde7e20295298e03

Do not normalize binary path in xs::PosixTestProcess

Normalizing path will prevent deProcess correctly stripping leading
working directory name from binary path.

Change-Id: If9e22b683f0f929219147e19cc31b42cf6e6bdeb

Fix bugs in pipeline barrier tests.

- Set m_currentImageLayout correctly. Fixes 201.
- Make sure GPU has finished before host memory accesses. Fixes #192.
- Handle _BOTTOM_OF_PIPE stage. Fixes #187.

Change-Id: Idd288583486dbd4239515587147301320b065eef

merge master

Search/place VertexID->VertexIndex and InstanceID->InstanceIndex

Merge branch 'awoloszyn-fix-IDs' into 'master'

Replaced uses of VertexID and InstanceID.

Switched to using InstanceIndex and VertexIndex.

See merge request !297