Extra space is removed and Context::getDeviceFeatures() is used instead of the InstanceInterface::getPhysicalDeviceFeatures().

Spares feature is checked and sparse bindig condition bug is fixed.

Sparse memory bindig is fixed.

Update to latest glslang revision

Change-Id: I4327ca3828c69b494a020bb62440946cd264ab35

Merge changes from Khronos repository

Merge branch 'imageLoadStore-reinterpet-test-must-use-mutable-flag' into 'master'

dEQP-VK.image.format_reinterpret.* tests must use VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT image FLAG

These test create image views with format different from underlying image. According to spec, such images must be created with VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT flag

See merge request !211

Merge "Fix image layout for storage images in binding model tests" into vulkan

Merge branch '161-push-constant-uniform-offset' into 'master'

Fix pipeline.push_constant.* incorrect uniform buffer layout declaration in shader.

Fix issue #161

See merge request !259

Merge branch 'query-pool-missing-flags' into 'master'

dEQP-VK.query_pool.*: added missing flags.

See merge request !260

Merge branch 'scygan_dynamic_states_add_feature_support_checks' into 'master'

dEQP_VK.dynamic_state: added feature support checks

Added feature support checks in dynamic state tests.

This fixes issue #164
This fixes issue #169
This fixes issue #170

See merge request !261

dEQP-VK.image.format_reinterpret.* tests must use VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT image flag

These test create image views with format different from underlying image. According to spec, such images must be created with VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT flag

DynamicStateTests: added feature support checks
ApiCommandBuffersTests: fixed one case

dEQP-VK.query_pool.*: added missing flags.

Fix pipeline.push_constant.* incorrect uniform buffer layout declaration in shader.

Change-Id: Ie4d63baedc5b6aca87c6760b63bdbda17cc78a15

Add missing layout(location = N) qualifiers to glsl.linkage tests

Change-Id: Ib17b83da25d637adc08f10a761f9deadd3ded77a

Fix image layout for storage images in binding model tests

Image load & store is only supported in VK_IMAGE_LAYOUT_GENERAL.

Change-Id: I1837394b81ba3c8c7cab8059b63345fbb7e4258d

Merge in changes from Khronos repository

Merge branch '168-bitwisexor-valid-for-opspecconstantop' into 'master'

Update to newer SPIRV-Tools

This version fixes the list of allowed opcodes for OpSpecConstantOp
Fixes Khronos bug 15379.
Fixes #168

Updates to SPIRV-Tools tip of master.
Tested in dEQP by rebuilding all 2373 "*spirv_as*" tests.

See merge request !257

Merge branch '92-binding_model-ranges_buffers' into 'master'

This fixes issue #92

- The parameters of VkCmdPoolCreateInfo are given in wrong order.

- typo fix in case name: seconday -> secondary

Merge branch 'multisample-mask-count-fix' into 'master'

Corrected formula to calculate number of pSamplerMasks.

This should fix issue #160

See merge request !256

Merge branch '166-remove-undef-function-type' into 'master'

Remove OpUndef of function type, fixing #166.

See merge request !253

Merge branch 'scygan_api_commandbuffers_fence_overuse' into 'master'

dEQP-VK.api.command_buffers.submit_count_equal_zero: Do not pass fence used by queue to vkQueueSubmit()

The test calls two vkQueueSubmits() on the same fence. This is illegal according to spec, especially it is in conflict with following wording:
```
Valid Usage for vkQueueSubmit:
"fence must not be associated with any other queue command that has not yet completed execution on that queue"
```

See merge request !255

Merge branch 'timestamp-check-support' into 'master'

Check support for timestamp queries.

See merge request !252

Merge branch '165-fix-loop-control' into 'master'

165: Fix spirv_assembly.instruction.loop_control.

No longer store an i32 to a u32.
Fixes #165

See merge request !251

Merge branch 'query-pool-reset-fix' into 'master'

Do not call vkCmdResetQueryPool() inside render pass.

See merge request !250

Merge branch '156-fix-quantized-nans' into 'master'

Fix spirv_assembly*opquantize.nan to not violate Vulkan spec.

This family of tests depended on NaN+0 returning NaN, which isn't
guaranteed in Vulkan.

See merge request !246

Merge branch '149-eliminate-SSA-violation' into 'master'

Eliminate SSA violation in spirv_assembly.instruction.graphics.opquantize

Due to a bad merge in 0644efb693c3612681bda085bcbbdb31a115568b, there
was an extra assignment to %cond.

Also fix two accidentally swapped comments.

See merge request !244

Merge branch 'ssbo-buffer-create-fix' into 'master'

Corrected order of members of VkBufferCreateInfo in SSBO layout case.

See merge request !249

Merge branch 'pipeline-barrier-frag-source-fix' into 'master'

Fix fragment shader source being registered as vertex source in dEQP-VK.memory.pipeline_barrier.*.

See merge request !248

Merge "Fix issues in memory tests." into vulkan

Update to newer SPIRV-Tools

This version fixes the list of allowed opcodes for OpSpecConstantOp
Fixes Khronos bug 15379.
Fixes #168

Corrected formula to calculate number of pSamplerMasks.

Binding model test: correct test case name, correct VkCommandPoolCreateInfo parameters

Check support for timestamp queries.

Do not pass signaled fence to vkQueueSubmit()

Remove OpUndef of function type, fixing #166.

Fix spirv_assembly.instruction.loop_control.

No longer store an i32 to a u32.
Fixes #165

Do not call vkCmdResetQueryPool() inside render pass.

Corrected order of members of VkBufferCreateInfo in SSBO layout case.

Fix fragment shader source being registered as vertex source in dEQP-VK.memory.pipeline_barrier.*.

Fix spirv_assembly*opquantize.nan to not violate Vulkan spec.

This family of tests depended on NaN+0 returning NaN, which isn't
guaranteed in Vulkan.

Merge "Change formats used in render pass tests." into vulkan

Fix pipeline barrier test issues

- Fix unmapping wrong memory object.
- Fixes to findMaxBufferSize.
- Round max buffer size to W*H*4 to make sure copies to and from
  images will cover whole buffer.
- Make sure that buffer copies to and from image always use the whole buffer.
- Fix life time issues of the buffer and image objects.
- Fix creating buffers / images with 0 usage.

Change-Id: I03fcc1b493dab36f4a55241a15a9f0bf804b27c8

Fix issues in memory tests.

Change memory mapping tests to use 1/16th of memory at most.
Change memory allocation tests to use 1/8th of memory at most.
Fix number of ranges in random flush and random finish calls.

Change-Id: Ie8c5ad688c6243c69ea5de8681cf3b4e4a7c5c93

Merge in changes from Khronos repository

Eliminate SSA violation in spirv_assembly.instruction.graphics.opquantize

Due to a bad merge in 0644efb693c3612681bda085bcbbdb31a115568b, there
was an extra assignment to %cond.

Also fix two accidentally swapped comments.

Merge branch 'mem-random-div-zero' into 'master'

Fix division by zero in random memory allocation test

I got a division by 0 with some configuration when running some random memory allocation tests.

The related code is:

    const VkDeviceSize size = 1 + (rng.getUint64() % ((m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage - 1));

It's possible for "(m_heap.size / MAX_MEMORY_USAGE_DIV) - m_usage - 1" to be equal to zero since we are doing a -1 and the method "full" just compare "m_usage * MAX_MEMORY_USAGE_DIV >= m_heap.size".

With this change I do a max(1) which should resolve the issue

See merge request !228

Change maxAnisotropy to 1 in object mgmt and smoke tests

Anisotropic filtering is optional and value 1 disables it.

Change-Id: Ib1d5ff3e72d8271eb24321c04782dbe9cd9c833f

Merge branch 'jbolz_copy_and_blit_fixes' into 'master'

Copy_and_blit fixes: replace format during CopyImage expected result calculation…

…, to make it use memcpy. Use Any memory for image allocation for buffer-to-image copy.

See merge request !234

Merge changes from Khronos repository

Merge branch 'jbolz_shaderrender_128' into 'master'

Change 'shaderrender' (glsl) tests to use a size of 128x128 rather than 100x100,…

… to avoid interpolation noise.

This is a slightly unusual bugfix, but I was seeing failures in the dEQP-VK.glsl.operator.binary_operator.bitwise_xor and dEQP-VK.glsl.operator.float_compare.lessThan/lessThanEqual (and GT/GEQ) tests, only in the fragment shader variants. These are both tests where a small interpolation difference can get magnified into a very large diff due to the nature of the operation, and this was causing these tests to fail. Using power-of-two dimensions reduces the frequency of GPU vs reference interpolation differences.

See merge request !225

Merge branch 'awoloszyn-fix-undefined-ids' into 'master'

Fixes a handful of tests that used undefined IDs.

Fixes #137
Fixes #149

See merge request !243

Merge branch 'jbolz_bitwisexor' into 'master'

Fix BitwiseXor specconstant test. Use only positive values for SRem/SMod

See bugs https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15380 and https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15379. Note that we will need to pick up an updated SPIRV-tools with the fix for 15379.

See merge request !235

Merge branch '116-bufferview-swizzle-scale-and-bias' into 'master'

Enhance verification of swizzling in pipeline.image_view tests

Fixes issue #116

- apply swizzle also to scale and bias parameters in shaders, so valid channels are not zeroed

Test pipeline.image_view verify the functionality of sampling a view of image by performing following operation:

```
texture(texSampler, vtxTexCoords) * lookupScale + lookupBias
```
and comparing results from Vulkan and ref renderer. The problem here is lookupScale and lookupBias ignore the swizzle set in image view object, thus for non 4-channel formats scale and bias zero the result on wrong channel, causing loss on information and comparing 0-es to 0-es in final validation of rendered image.

As a fix, I propose to apply component mapping from image view to both values (both in Vulkan and ref renderer shader).

See merge request !192

Merge branch '98-remove-3d-astc' into 'master'

Removed test cases for 3D ASTC textures.

See merge request !242

Merge branch 'jbolz_pipeline_barrier_fixes2' into 'master'

pipeline_barrier: Keep the VkBuffer in the PrepareContext alive until the work is executed...

…, by putting it in the same scope as the execution. Also wait for idle before the destructors are called.

This may not be the best fix, but it works. Buffers can be created and destroyed by various OPs in the test, but I think all the Execute work happens with the last buffer. The test may need some restructuring to using the different buffers.

See merge request !230

Fixes a handful of tests that used undefined IDs.

Fixes #137
Fixes #149

Removed test cases for 3D ASTC textures.

Merge "Use -Wno-date-time to allow __DATE__/__TIME__" am: bd1826b4a3 am: f4e3b26175
am: c3d988aab7

* commit 'c3d988aab70b74c98ce51d7345eac519ad5c9222':
  Use -Wno-date-time to allow __DATE__/__TIME__

Merge "Use -Wno-date-time to allow __DATE__/__TIME__" am: bd1826b4a3
am: f4e3b26175

* commit 'f4e3b26175490978db76e76576b488cbe2cf6bb6':
  Use -Wno-date-time to allow __DATE__/__TIME__

Merge "Use -Wno-date-time to allow __DATE__/__TIME__"
am: bd1826b4a3

* commit 'bd1826b4a347e28e7aab1479d53b1c6d8ce4c640':
  Use -Wno-date-time to allow __DATE__/__TIME__

Merge "Use -Wno-date-time to allow __DATE__/__TIME__"

Merge changes from Khronos repository

Merge 138-dont-require-tess-geom

Merge "Update MNC GLES2 mustpass list"
am: 661fef704e

* commit '661fef704eeecc3d83997bffc2615b19e68efa81':
  Update MNC GLES2 mustpass list

Merge "Update MNC GLES2 mustpass list"

Merge "Add check for unsupported limits in api.info.device" into vulkan

Remove few tests affected by a HW limitation from master and MNC lists
am: 2c2db488c8

* commit '2c2db488c8596e3c8fa9791d019aee22cd654821':
  Remove few tests affected by a HW limitation from master and MNC lists

Update MNC GLES2 mustpass list

Change-Id: I9a123692804d61fd716e39b60a87276baeb3fcc9

Remove few tests affected by a HW limitation from master and MNC lists

Some primitive_bbox tests were earlier removed due to a HW defect, but
the list was incomplete.

Bug: 22713865
Change-Id: I989e5c65de75f2b67b3e19f17a531b37831f0fe9

Clean up test hierarchy and case naming

Affects following test groups:

 * dEQP-VK.glsl.ShaderExecutor
 * dEQP-VK.spirv_assembly.instruction.graphics

Change-Id: Ia14ee2deb112670d0a222380b5e0ddf6e03daffd

Merge branch 'jbolz_pipeline_barrier_fixes' into 'master'

pipeline_barrier fixes

- Try to use the full (power of two) buffer size. Not doing so tickles several issues to do with not writing all bytes of the buffer or image.
- Fix FillBuffer::verify byte selection.
- Use R8G8 rather than R8G8B8 for the vertex format. The third component is out of bounds for the last vertex, which makes the entire attribute undefined (i.e. substituted with zero on some implementations).
- Subtract out epsilon in vertex shader math to avoid points on the frustum boundary which may be culled.

See merge request !227

Use context.getDeviceFeatures instead of instanceInterface.

Merge branch '144-145-fix-assembly' into 'master'

Fixes for assembly in opMemoryAccess and opQuantize tests.

Fixes #144
Fixes #145

See merge request !241

Merge branch '141-fix-android-build-win-64' into 'master'

Fix Android build script on Windows 64 bit (NDK r10e)

- Correct Host OS name on Windows 64 bit in common.py
- Add the missing Host OS in toolchain file for NDK r10e
- Downgrade to LLVM 3.5 in NDK r10e because the prebuilt version 3.6 is
  missing the llvm-ar utility which is needed by the build script.

See merge request !236

Updated to only include necessary stages in tests.

This simplifies the framework, and flow of the tests.
Any stage present in a test will be required. Tests will
only require tess/geom if we are specifically testing those stages.

Fixes for assembly in opMemoryAccess and opQuantize tests.

Fixes #144
Fixes #145

Fix Android build script on Windows 64 bit (NDK r10e)

- Correct Host OS name on Windows 64 bit in common.py
- Add the missing Host OS in toolchain file for NDK r10e
- Downgrade to LLVM 3.5 when using Windows NDK r10e,
  because the prebuilt version 3.6 is missing the llvm-ar
  utility which is needed by the build script.

Add check for unsupported limits in api.info.device

Fixes https://gitlab.khronos.org/vulkan/vulkancts/issues/120

Change-Id: If43e22c7007325eb00d87bbe59a2dca65aae584e

Fix BitwiseXor specconstant test. Use only positive values for SRem/SMod

Change formats used in render pass tests.

Limit formats to formats that are required by the core specification
to avoid not supported errors.

Add separate format tests for depth and stencil formats.

Change-Id: I962d0dfede03748de7af34759a6e3c0126d626fc

Merge branch 'bug137-fix-spirv_assembly.instruction.graphics.module-geom-var-decls' into 'master'

Fix 137: bad geom entry point I/O var declarations

Fix declarations of Input and Output declarations in test
dEQP-VK.spirv_assembly.instruction.graphics.module.*
They were missing "geom_" prefices.

Changed:

    OpEntryPoint Geometry %geom_main "main" %out_gl_position %gl_in
      %out_color %in_color

To the following, to match usage in the function bodies:

    OpEntryPoint Geometry %geom_main "main" %geom_out_gl_position
       %geom_gl_in %geom_out_color %geom_in_color

Also fix associated OpDecorate instructions.

Fixes #137

See merge request !238

Fix implicit VkDeviceSize -> size_t conversion warnings

Change-Id: Ibb4b63cf0651ddec66a62e39cc1ab1b155cedd13

Fix a few warnings

Change-Id: Icc60dcf8f72c1b9140c76bb1568af37b0b3b1843

Merge in changes from Khronos repository

spirv_assembly.instruction.graphics.* tests no long require tess/geom.

Tessellation/Geoemtry shaders will be used if present, but disabled
otherwise. NotSupported is set for any tests that requires an optional
feature.

Merge branch 'shaderExecutor-imageLayout' into 'master'

ShaderExecutor: fixed FragmentOutExecutor that used Images in UNDEFINED layout

Added proper transitions and fixed layout used for cmdCopyImageToBuffer

See merge request !224

Merge branch 'scygan_image_reinterpret_snorm_magic_0x80' into 'master'

dEQP-VK.image.reinterpret: bad handling of -128 SNORM value

Some tests in reintepret group create images of R32I type, than issue an imageLoad()-ed in shader, reinterpreting the image as RGBA8_SNORM. Than the value is imageStore()-ed again as RGBA8_SNORM.

Effectively each channel undergoes SNORM8 -> float -> SNORM8 conversion.

The problem is test requires the stored image to match exactly the loaded one, while the loaded image contains bytes of -128 value.

-128 (same as 127) converts to -1.0 float, however according to following equation in spec, the -1.0 float value can convert only to 0x81, what makes test fail.

```
f′=convertFloatToInt(f*(2^(b−1)−1),b)
```

The proposed fix here is to replace all 0x80 values in loaded image - same it is already done for NaNs and Infs for float format.

See merge request !233

Fix 137: bad geom entry point I/O var declarations

Fix declarations of Input and Output variable declarations and
the gl_PerVertex type declaration in test
dEQP-VK.spirv_assembly.instruction.graphics.module.*
They were missing "geom_" prefices.

Changed:
OpEntryPoint Geometry %geom_main "main" %out_gl_position %gl_in
   %out_color %in_color

To the following, to match usage in the function bodies:

    OpEntryPoint Geometry %geom_main "main" %geom_out_gl_position
       %geom_gl_in %geom_out_color %geom_in_color

Also fix associated OpDecorate instructions.

Should fix https://gitlab.khronos.org/vulkan/vulkancts/issues/137

ShaderExecutor: fixed FragmentOutExecutor that used Images in UNDEFINED layout

Added proper transitions and fixed layout used for cmdCopyImageToBuffer

replace  problematic  values 0x80 (-128) for snorm formats

Merge branch '134-fix-spirv-graphics-loop' into 'master'

Fixed invalid spirv in dEQP-VK.spirv_assembly.instruction.graphics.loop.*

Fix for #134

See merge request !232

Merge branch '136-fix-opMemoryAccess' into 'master'

Fixed invalid SPIR-V in spirv_assembly.instruction.graphics.opMemoryAccess.*

For issue #136

See merge request !231

Merge branch '133-fix-opfrem' into 'master'

Fix invalid SPIR-V in dEQP-VK.spirv_assembly.instruction.graphics.frem.*

See merge request !229

Merge branch '132-push-constant-fix' into 'master'

Fix bugs for vertex attribute count in push constant tests.

Fix issue #132 : set wrong vertex attribute count in pipeline

See merge request !226

Merge branch 'shaderExecutor-imageView' into 'master'

ShaderExecutor: fixed FragmentOutExecutor that used imageView with mipLevels == 0

VkImageViewCreateInfo.mipLevels should equal to 1 to render anything

See merge request !222

Merge branch 'jbolz_ssbo_fix' into 'master'

Fix descriptors for SSBO tests.

This is the same fix as in !187, just for SSBO tests.

See merge request !220

Merge branch 'api-copiesAndBlitting-bufferUsage' into 'master'

copies_and_blitting test: fix case 'buffer_to_image' - changed src buffer usage flags

There is a buffer in 'buffer_to_image' test, that is used as a source of a copy command, however it's usage is set to DST.

See merge request !219

Merge branch 'jbolz_fix_floatbitstouint' into 'master'

Fix floatbitstouint shaderexecutor test copy/paste bug

See merge request !217

Merge branch 'jbolz_draw_instance_index' into 'master'

Change instanceID shader to use InstanceIndex

Change gl_InstanceID to gl_InstanceIndex. Add location qualifier on fragment input. Increase size of rendertarget to reduce likelihood of false success due to fuzzy compare.

Fixes #123.

See merge request !216