Coding style fixes

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.

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

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

merge master

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

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

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

Replaced uses of VertexID and InstanceID.

Switched to using InstanceIndex and VertexIndex.

Switch to libpng_ndk to remove platform library dependency

Bug: 26513978
Change-Id: I4cb929149264adccdaf3de0f0f57a1e6b64921b5

use dummy descriptor in descriptor pool instead of empty one when there are no uniforms declared

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

Added missing assembly to spirv_assembly.instruction.compute.opquantize

Fixes #190

See merge request !292

Merge branch '51_renederpass_grow_fix_image-layouts' into 'master'

51 dEQP-VK.renderpass.grow and others: do not use DEPTH layouts for color images, and vice versa.

This is the continuation of issue #51, that was not completely fixed in merge request !106. This change just replicates what !106 did for other test cases.

This fixes issue #51

See merge request !294

Merge branch 'jbolz_astc_srgb_5x4_8x5' into 'master'

Fix mapping of ASTC/sRGB formats for 5x4 and 8x5 block sizes

See merge request !284

Merge branch 'scygan_pipeline_memoryBarrier-usage_flags' into 'master'

memory.pipeline_barrier test: wrong buffer usage flags

The `srcBuffer` buffer is used as a source in vkCmdCopyBufferToImage - however it lacks VK_BUFFER_USAGE_TRANSFER_SRC_BIT usage flag.

See merge request !293

Merge branch '175-fix-opaque-type-indexing-atomic-counter-tests' into 'master'

ShaderExecutor: read counter values from the buffer

Problem: At vktOpaqueTypeIndexingTests.cpp:1077, the tests creates a std::vector called counterValues and default-initializies. Then, in the loop below, it checks counterValues as if it contains the actual counter values even though they are never read from the buffer.

Solve: Read back the counter values from the buffer.

Fixes #175

See merge request !291

Merge branch 'jbolz_descriptor_set_regression' into 'master'

Fix recent regression binding unallocated descriptor set

See merge request !285

Merge branch '86-fix-api-buffer' into 'master'

Fix sparse memory issues in buffer test

Fixes #86
Fixes #171

See merge request !268

Merge branch '56-fix-image-copy-inside-render-pass' into 'master'

Move cmdCopyImageToBuffer outside from render pass

Fixes #56

See merge request !287

Merge branch '185-fix-timestamp-test-issue' into 'master'

Reset the query pool entries before write timestamp to fix issue #185

In Test case dEQP-VK.pipeline.timestamp.transfer_tests.transfer_stage_with_copy_query_pool_results_method,
the timestamp will be written twice and the query pool entries need be reset between these write timestamp operations.

See merge request !286

Merge branch 'copy-blit-size-format-issue2' into 'master'

Fix format and texture level width issue

Fixes #180

See merge request !282

Merge branch '143-fix-version-in-shader-render-case-based-tests' into 'master'

Use version 310 es in ShaderRenderCase based tests.

fixes #143

See merge request !280

dEQP-VK.renderpass.grow and others: do not use DEPTH layouts for color images, and vice versa.

This is the continuation of issue #51, that was not completely fixed in merge request !106. This change just replicates what !106 did for other test cases.