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

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

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.

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

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

Merge branch 'dynamic-state-fixes' into 'master'

Various fixes for dynamic state tests.

* Added location qualifier to vertex output variables.
* Added missing initialization of flags in some CreateInfo structures.
* Check if depth bounds test is supported.
* Check if multi-viewport is supported.
* Check if geometry shaders are supported.
* Fixed reference image in lineWidth test.

See merge request !215

Merge branch '122-fix-timestamp-tests-typo' into 'master'

Fix timestamp tests typo (issue #122)

Fix a typo and some assertions to make timestamp basic compute tests work.

See merge request !214

Merge branch 'jbolz_fix_cmdbuf' into 'master'

Fix command buffer tests.

Fix number of command buffers for secondary_execute_twice test. Fix result buffer memory binding for order_bind_pipeline test. Fix comments for CommandBufferAllocateInfo structs.

See merge request !213

Merge branch 'jbolz_push_constants' into 'master'

Push constants fixes

Push Constant fixes: add location=0 on color I/O. Replace 'size 40' test with 'size 48', replacing two push constant blocks with one and using proper std140 alignment.

See merge request !206

Merge branch 'jbolz_ubo_align' into 'master'

Fix 'ubo' tests to lay out data satisfying the min ubo alignment

Fixes #114.

I didn't see any way to get the alignment during the init() function, so I just made a copy of it before download.

See merge request !205

Merge branch 'jbolz_glsl_matrix2' into 'master'

Fix glsl.matrix.outerproduct tests

Using 'getPtr' selected a template instantiation that downloaded the pointer value rather than the vector.

See merge request !204

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

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

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

pipeline_barrier fixes

Fix bugs for vertex attribute count in push constant tests.

Change-Id: Iccacad6b3a787feedbb0b4df7f8689226fa64a7a

Remove few tests affected by a HW limitation am: 3f71117a2f am: 58213c37ff am: 8f5b937334  -s ours
am: e2eb2f509b

* commit 'e2eb2f509bd1a6a041d6e39fcbe4844708e83336':
  Remove few tests affected by a HW limitation

Remove few tests affected by a HW limitation am: 3f71117a2f am: 58213c37ff
am: 8f5b937334  -s ours

* commit '8f5b937334b80c4961fe4119984e23279d911c0a':
  Remove few tests affected by a HW limitation

Remove few tests affected by a HW limitation am: 3f71117a2f
am: 58213c37ff

* commit '58213c37ff819fce0572fb0e65b3b9fe3e19b398':
  Remove few tests affected by a HW limitation

Remove few tests affected by a HW limitation
am: 3f71117a2f

* commit '3f71117a2f57d08ef9a5efe2f211ba515268af60':
  Remove few tests affected by a HW limitation

fix case 'buffer_to_image' - changed src buffer usage flags

Various fixes for dynamic state tests.

* Added location qualifier to vertex output variables.
* Added missing initialization of flags in some CreateInfo structures.
* Check if depth bounds test is supported.
* Check if multi-viewport is supported.
* Check if geometry shaders are supported.
* Fixed reference image in lineWidth test.

Added test case for conditional assignments to matrix elements.
am: cbf84e3898

* commit 'cbf84e38987721d18994470b5ecabd0c2f812590':
  Added test case for conditional assignments to matrix elements.

Change gl_InstanceID to gl_InstanceIndex. Increase width/height to avoid false positives in fuzzy compare.

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

VkImageViewCreateInfo.mipLevels should equal to 1 to render anything

Fix descriptors for SSBO tests.

Fix floatbitstouint shaderexecutor test copy/paste bug

Fix issues after code review.

Merge branch 'master' into jbolz_push_constants

Fix warnings in copies and blit tests

Merge 70-copies-and-blitting

Merge branch 'dkoch_mv_draw_shaders' into 'master'

Move draw shaders to correct location

The draw/ tests are expecting the path vulkan/draw/ instead of just draw/.

See merge request !212

Merge branch 'imageLoadStore-too-small-descriptorPool' into 'master'

vktImageLoadStoreTests: fix *__layer test cases - fix too small descriptor pool

In dEQP-VK.image.store.1d_array.r32g32b32a32_sfloat_single_layer test case, there are 8 descriptor sets allocated from descriptor pool that has space only for one.

See merge request !210

Merge branch 'renderpass-proper-aspect-for-cmdClearAttachments' into 'master'

Renderpass test: use proper aspect for cmdClearAttachments

dEQP-VK.renderpass.attachment.3.206 tests uses this chunk of code to clear D32 image, so VK_IMAGE_ASPECT_STENCIL_BIT is invalid.

See merge request !209

Merge branch '80-ApiObjectManagement-desriptorPool-size' into 'master'

Fix descriptor pool size in api.object_management.multiple_shared_resources.descriptor_set

Fixes issue #80

- there are multiple sets allocated, so desciptor VkDescriptorPoolSize.descriptorCount must be multipled by number
of sets

See merge request !208

Merge branch 'api-bufferView-image-layout' into 'master'

vktApiBufferViewAccessTests: image cannot be created with COLOR_ATTACHMENT_OPTIMAL layout.

- According to spec, the only valid layouts in vkCreateImage are UNDEFINED and PREINITIALIZED
- This changes the test so image is cratead with valid layout, than transitioned to COLOR_ATTACHMENT_OPTIMAL using additional barrier

See merge request !207

Merge branch '117-push-constant-wrong-glsl' into 'master'

Fix bugs of the shaders in push constant tests

This fix issue #117 and #118

-fix the offset of the uniform block member to align with it's size

-modify the shader to fix wrong expected value for dEQP-VK.pipeline.push_constant.graphics_pipeline.data_update_partial_1

See merge request !199

Merge branch 'jekstrand-fix-timestamp-c++11' into 'master'

PipelineTimestampTests: Stop relying on C++11 features

The cbegin and cend functions weren't added to std::vector until c++11.
They also aren't really serving a purpose here given that there's no
automatic type inference going on, just a regular for loop.  This fixes the
build on Fedora 23 when using clang and default dEQP build flags.

See merge request !198

Merge branch '76-fix-commandbuffer-params' into 'master'

fix VkCommandBufferBeginInfo struct values, fixes #76

just a short fix to change those two mixed struct values

See merge request !190

Merge branch 'jbolz_fix_ubo' into 'master'

Fix ubo.per_block_buffer tests

See merge request !187

Merge branch 'jekstrand-binding-model-hang-fix' into 'master'

api.binding_model: Wait for idle before throwing NotSupportedError

If we don't, then the GPU could be busy and some of the objects destroyed
by the exception propagation could be in-use.  This is something of a
stop-gap measure; the problem of exceptions and the GPU being busy is
something that needs to be solved in general.  However, it does fix GPU
hangs on some drivers.

This is related to #20.

See merge request !185

api.binding_model: Wait for idle before throwing NotSupportedError

If we don't, then the GPU could be busy and some of the objects destroyed
by the exception propagation could be in-use.  This is something of a
stop-gap measure; the problem of exceptions and the GPU being busy is
something that needs to be solved in general.  However, it does fix GPU
hangs on some drivers.

This is related to #20.

Fix typo in compute shader and assertion when creating buffers.

Fix number of command buffers for secondary_execut_twice test. Fix result buffer memory binding for order_bind_pipeline test. Fix comments for CommandBufferAllocateInfo structs.

Move draw shaders to correct location

The draw/ tests are expecting the path vulkan/draw/ instead of just draw/.

Style fixes for the Copy and blitting

Add copy BufferToBuffer tests.

Add Image to image copies

Fix the wrong bufferSize in CopiesAndBlitting

Set queueFamilyIndices in CopiesAndBlitting

Rename ImageToImageCopies to CopyImageToImage

Fix warnings in the CopiesAndBlitting

Remove leftover comment

Add CopyBufferToImage test

Add image to image test

Finish the buffer to buffer copies base class

Rename testgroup of the copies and blitting

Implement copyRegions for the buffer to buffer copies

Remove m_regions from the CopiesAndBlitting class

Fix the CopyImageToBuffer test

Add WIP version of BufferToBufferTest.

Improve TestParams in CopiesAndBlittingTests

Several fixes around the readImage and copyImage

Improve the calculateSize utility function

Refactor uploadBuffer and uploadImage.

Fix the upload image and the ImageToBuffer test

Remove BufferType enum, add uploadBuffer.

Add readImage

Implement uploadImage utility function in CopiesAndBlittingTests

Refactor checkResult function

Add generateExpectedResult function

Implement generateBuffer function

Add base of the CopyImageToBuffer test

Add base of the copy image to image

Add base of Copies And Blitting tests

Copies and blitting: update the test specification

vktImageLoadStoreTests: fix *__layer test cases - fix too small descriptor pool

In dEQP-VK.image.store.1d_array.r32g32b32a32_sfloat_single_layer test case, there are 8 descriptor sets allocated from descriptor pool that has space only for one.

Renderpass test: use proper aspect for cmdClearAttachments

dEQP-VK.renderpass.attachment.3.206 tests uses this chunk of code to clear D32 image, so VK_IMAGE_ASPECT_STENCIL_BIT is invalid.

Fix descriptor pool size in api.object_management.multiple_shared_resources.descriptor_set

- there are multiple sets allocated, so desciptor VkDescriptorPoolSize.descriptorCount must be multipled by number of sets

vktApiBufferViewAccessTests: image cannot be created with COLOR_ATTACHMENT_OPTIMAL layout.

- According to spec, the only valid layouts in vkCreateImage are UNDEFINED and PREINITIALIZED
- This changes the test so image is cratead with valid layout, than transitioned to COLOR_ATTACHMENT_OPTIMAL using additional barrier

Fix couple of invalid VkBufferCreateInfos and ws violations in command buffer tests