Multiple fixes in dEQP-VK.pipeline.timestamp test

Merge branch 'jbolz_pipeline_barrier_1998' into 'master'

Put point vertex coordinates in [-.999, .999] rather than [-0.999, 1.0]

Fixes #236

See merge request !355

Put point vertex coordinates in [-.999, .999] rather than [-0.999, 1.0]

Merge branch '235-pipeline_multisample-alpha_translucent-higher-threshold' into 'master'

Use a higher threshold for translucency case to let the test pass:

dEQP-VK.pipeline.multisample.alpha_to_coverage.samples_2.alpha_translucent

Issue is described in #235

See merge request !352

Merge "Pass boolean values correctly to shaders." into vulkan

Fix int-size_t comparison warning

Merge in changes from Khronos repository

Pass boolean values correctly to shaders.

Map type to correct vkFormat for other than floating point types and
pass boolean values correctly as 0xFFFFFFFFu instead of 0x1u to shader
in vulkan ShaderLibrary tests.

Change-Id: Ib13b4a3c648b21a0b9020d00caeafd17a23b7402

Use a higher threshold for translucency case to let the test pass:

dEQP-VK.pipeline.multisample.alpha_to_coverage.samples_2.alpha_translucent

Issue is described in:

https://gitlab.khronos.org/vulkan/vulkancts/issues/235

Re-enable dEQP-GLES3.functional.shaders.builtin_variable.fragcoord_w .
am: 888059d575

* commit '888059d575bed0e8beedba85068f473a4f1fcdd8':
  Re-enable dEQP-GLES3.functional.shaders.builtin_variable.fragcoord_w .

Merge branch '218-builtin-precision-hw-issues' into 'master'

Remove builtin precision tests affected by HW limitations from mustpass

Fixes #218

See merge request !327

Merge branch '217-builtin-precision-test-issues' into 'master'

Remove precision tests with known issues from mustpass

Fixes #217

See merge request !328

Remove precision tests with known issues from mustpass

Fixes #217

Merge branch '231-fix-spirv-loop-exit' into 'master'

Fix #231 and #232.

Split apart the early-exit loop tests and ensure that %val defs dominate
uses and OpPhi instructions always mention parent blocks.

See merge request !351

Merge branch 'cube_and_shadow_face_coords-fix' into 'master'

cubeshadow sampler - valid coords setup

OpaqueTypeIndexingTests: fixed cubeshadow sampler - provided valid coords setup
I am closing #336 as it was incomplete.

See merge request !344

Merge branch 'issue_230_mustpass' into 'master'

Fix dEQP-VK.api.object_management.max_concurrent.device_memory_small

Fixes #230

Limit the number of allocations to 4K (which is the minimum required by the spec).

See merge request !348

Merge branch 'ppiotrowski_memory_pipeline_barrier_src_usage_flag_fix' into 'master'

memory.pipeline.barrier.* fix for copy*Image flags

This affects tests using cmdImageCopy and cmdImageBlitCopy.
change layout of destination image to VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL;
changed image layout of source image to VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL;

See merge request !335

Merge branch '139-fix-shaderexecutor-upload-image' into 'master'

ShaderExecutor: use buffer for copying images, fixes #139.

See merge request !350

Fix dEQP-VK.api.object_management.max_concurrent.device_memory_small

Issue 230

Clamp to 4K allocations (minimum required by the spec).

Fix #231 and #232.

Split apart the early-exit loop tests and ensure that %val defs dominate
uses and OpPhi instructions always mention parent blocks.

ShaderExecutor: use buffer for copying images, fixes #139.

Merge branch 'jbolz_nan_sign' into 'master'

Don't test sign of NaN value, as NaN values are not preserved

See https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15426

See merge request !349

Merge branch 'jbolz_timestamp_locations' into 'master'

Add location assignments to shader I/O variables in timestamp tests

See merge request !338

Merge branch '216-remove-lowp-shaderrendercase-and-shaderexecutor-cases' into 'master'

Remove ported lowp ShaderRenderCase and ShaderExecutor cases

fixes #216

See merge request !347

Don't test sign of NaN value, as NaN values are not preserved

Merge branch '221-fix-memory-leak-in-command-buffers' into 'master'

Fix memory leak in dEQP-VK.api.command_buffers tests

See merge request !337

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

Updated the SPIR-V in opphi.out_of_order to be the same as the comments

The previous test fixed the comments and expectations, but
not the actual SPIR-V.

Fixes #209

See merge request !334

Merge branch 'jbolz_image_sampled1' into 'master'

Change 'Sampled' operand to '1' for OpUndef image tests

Vulkan spec says:

"Variables of type code:OpTypeImage must: have a code:Sampled
operand of 1 (sampled image) or 2 (storage image)."

See merge request !317

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

OpaqueTypeIndexingTests: fixed cubeshadow sampler - valid coords setup

Remove ported lowp ShaderRenderCase and ShaderExecutor cases
fixes #216

Add location assignments to shader I/O variables in timestamp tests

Fix memory leak in dEQP-VK.api.command_buffers tests

changed source buffer usage flag to VK_BUFFER_USAGE_TRANSFER_SRC_BIT; changed image layout of destination image to VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL; changed image layout of source image to VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL

Updated the SPIR-V in opphi.out_of_order to be the same as the comments

The previous test fixed the comments and expectations, but
not the actual SPIR-V.

Fixes #209

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.

Remove builtin precision tests affected by HW limitations from mustpass

Fixes #218

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

Change 'Sampled' operand to '1' for OpUndef image tests

Re-enable dEQP-GLES3.functional.shaders.builtin_variable.fragcoord_w .

This test covers important functionality and driver bugs that affect it
should be fixed.

BUG: 26333630

Change-Id: I2342045da4860d76b4e8f48031b985cc884cd263

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.