Fix a compilation error on GCC 5.4

Components: Vulkan
Change-Id: I6c6681d16613f18b2a3192368927f6d176f63a64

Fix bug in reallocation callback validation

vk::validateAllocationCallbacks() didn't correctly handle a case where
reallocate(NULL, size) returned previously seen pointer value.

VK-GL-CTS issue: 176
Components: Vulkan

Change-Id: I0822f8e4176d20d7bf296cd85f5c5223e189d250

Update zlib dependency to zlib 1.2.11

Fixes Vulkan CTS issue: #579

Change-Id: Ib0cd05616cbb4759d63bcf15ba8e8f6f374e291d

Update zlib and libpng dependencies

Fixes #568

Change-Id: I8ee890a2176bf6d42d3dd8b119b7263034127960

Add -Wno-delete-non-virtual-dtor to default cxxflags

That warning generates some false negatives that are hard to work
around.

Change-Id: Ib451eca3af7bf2a6bfd24aa966f89496257b9f0d

Get rid of invalid buffer create flags combinations

Removed tests:
- dEQP-VK.api.buffer.createBuffer_*_2
- dEQP-VK.api.buffer.createBuffer_*_6

Change-Id: I6cc6ed365de9d710f07fbfd2dde9db7edaecd007

Clarify --deqp-log-filename usage in README.md

Vulkan CTS issue: #525

Change-Id: Idb0f23cd13e548aaea21682ec02d89e52adee579

Framework: Add assert to position deviation image compare

When using position deviation tolerance it's easy to forget
about the depth and effectively skip the check for the whole
image.

Change-Id: Ide873be57ba19e7e9e45606d27aae403f9c131ec

Fix RelaxedPrecision mismatch in shader return statement tests

Affects:

dEQP-VK.glsl.return.single_return_vertex
dEQP-VK.glsl.return.double_return_vertex
dEQP-VK.glsl.return.last_statement_in_main_vertex
dEQP-VK.glsl.return.return_in_infinite_loop_vertex

See #518

Change-Id: I981b7623f7dc2eed2e438092226f3fa3bde44d58

Use highp by default in ShaderLibrary-based tests

This resolves numerous RelaxedPrecision mismatches

Affects:

dEQP-VK.glsl.arrays.*
dEQP-VK.glsl.conditionals.*
dEQP-VK.glsl.constants.*
dEQP-VK.glsl.conversions.*
dEQP-VK.glsl.functions.*
dEQP-VK.glsl.linkage.*
dEQP-VK.glsl.scoping.*
dEQP-VK.glsl.swizzles.*

See #518

Change-Id: I7b79f10c6dbdd5ae8da31359b680ac52ce2ad28c

Fix RelaxedPrecision mismatch in binding_model tests

Affects: dEQP-VK.binding_model.*

See #518

Change-Id: Iccda14ae17c7852c5283547e01573907f2eef18a

Fix race condition between query pool reset and accessing query results

Affects tests:
dEQP-VK.query_pool.occlusion_query.get_results*(wait_query|wait_none)*

These tests access query pool results on host, without any synchronization
with command buffer which performs the query - they either use
VK_QUERY_RESULT_WAIT_BIT or allow for incomplete results to be captured.

This may have a race condition, mentioned by spec:

"""
Applications must take care to ensure that use of the
VK_QUERY_RESULT_WAIT_BIT bit has the desired effect. For example, if
a query has been used previously and a command buffer records the
commands vkCmdResetQueryPool, vkCmdBeginQuery, and vkCmdEndQuery for
that query, then the query will remain in the available state until the
 vkCmdResetQueryPool command executes on a queue.
"""

In affected tests, this change moves vkCmdResetQueryPool to separate
command buffer that is sumitted and completed before query is used on
device & host.

Additionally code handling RESULTS_MODE_COPY mode is refactored to use
new haveSeparateCopyCmdBuf() function instead of tons of flags and conditions.

Change-Id: Id4a538b912030f2a86b09593fecaae37f1819a71

Update to latest libpng release

Previously used release 1.6.17 has been deprecated due to security bugs.

Change-Id: Ie5b749eea96e34f88a6833fbfab40c59cd3d162c

Fix alignment issues in uploading image data in ShaderExecutor

Fixes #485

Change-Id: If04aebab19fb25ef26b51b4af86e718004438d82
(cherry picked from commit 9141d25d08d53c203f8ac5a9c5937a41e1d83d02)

dEQP-VK.compute: Fix layout of storage image descriptor

Affects: dEQP-VK.compute.basic.copy_image_to_ssbo*

IMAGE_STORAGE type descriptor was updated with *_OPTIMAL layout.
This is invalid according to spec:

"Load and store operations on storage images can only
be done on images in VK_IMAGE_LAYOUT_GENERAL layout."

Submitted Vulkan-LoaderAndValidationLayers issue !1034 to
ensure this is caught in val layers.

Additionally one validation warn on 0 src stage in barrier is fixed.

Change-Id: Ie6d6875465f382c62dd9802f286de77c4c808fc2

Bugfix: use normalized coords in glsl sampler tests

Modified tests:
- dEQP-VK.glsl.opaque_type_indexing.sampler.*

Fixes #505

Change-Id: Ie7c7f48276b52bcce3cb8ac41914398b669fe5ae

Fix incorrect limits in device properties test

- alignment/offset type limits must be greater than 0
- nonCoherentAtomSize must be in range [1, 256] (the upper limit was
  relaxed with spec update 1.0.27
- point sizes and line widths must be positive

Modified test:
- dEQP-VK.api.info.device.properties

Fixes #502

Change-Id: I98a956d1cea741dd6d787415a0ad91178ffa0abf

dEQP-VK.dynamic_state tests: fix sizeof(size) bug

Change-Id: I7e90b30a88bb59bbe73fec00ecb8ce0fd92ac1f0

Merge "Revert "Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests""" into vulkan-cts-1.0.0

Revert "Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests""

This reverts commit 1106b0194f7571aebfdf76766760d91be772d8e7.

Fixes #440

Change-Id: Icdb5f71cd87b970edd1bf1770b4a0c15418f600b

Fix for #474. Add Vertex ordering and Spacing to tessellation shaders.

Change-Id: I33760e0f23e6b8489e38cb93ad00e3483dc73d16

Fix image layout in pipeline barrier tests

(cherry picked from commit a80919a57ef8137712259273c18de0663cc724a0)

Change-Id: I59a6a1facd9348c949f78f808075a0377c1bfe9a

Revert CUBE_COMPATIBLE flag check from "Fix required sample count tests"

Spec does not say CUBE_COMPATIBLE flag forces
vkGetPhysicalDeviceImageFormatProperties to return sample count equal
to 1, so the flag should be irrelevant for sample count check.

Fixes #440
This partially reverts 8753a9e7528ca67b25bfabb4f03915a25721a690.

Change-Id: Ic4005cb0a651f433ff421eab671fb013561c78c4

Fix required sample count tests

According to,

   Table 11.1. Image and image view parameter compatibility requirements

Only non-cube images with type VK_IMAGE_TYPE_2D can have sample counts
greater than one.

In addition, the following properties should hold for multisampled
images: the formats must support being used in a color or depth/stencil
attachment, or in a storage image when shaderStorageImageMultisample is
supported. Images not fitting the above descriptions may have a sample
count of one or zero (in error conditions). Align tests for sample
counts to this behavior.

(cherry picked from commit dc4227d5ec87080f5dad810b5e33ae3b3fa014a4)

Update glslang to (f5dcdf01) "Front-end: Don't lower precision of bit operations taking highp inputs."

Fixes #382

Explicitly declare gl_PerVertex block

This fixes SPIR-V validation error in shaders generated from GLSL (not ES)
due to missing ClipDistance capability.

(cherry picked from commit 0a6fe1448b303a6a4e4ab48712eaf1c0b16d75b1)

Merge branch 'git_log_process_update' into 'vulkan-cts-1.0'

See merge request !577

Merge branch 'scygan-fix_image_properties_tests' into 'vulkan-cts-1.0'

Fix dEQP-VK.api.info.image_format_properties - TRANSIENT usage flag used with DMA flags

Spec:
```
VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT indicates that the ....
If this is set, then bits other than VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT,
VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT, and VK_IMAGE_USAGE_INPUT_ATTACHMENT_BIT must not be set.
```

The test tries the forbidden combination of usage flags, for example VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT + VK_IMAGE_USAGE_TRANSFER_SRC

See merge request !570

Update git log command

Fix dEQP-VK.api.info.image_format_properties - TRANSIENT usage flag used with DMA flags

Conflicts:
intel/negativeList.txt

Merge branch 'memory-release-fix' into 'vulkan-cts-1.0'

Use TCU_CHECK instead of DE_ASSERT

DE_ASSERT is optimized away in release build. In this case it resulted
in the 'modifyXor' function not being called, causing the reference
memory to go out of sync with the actual memory. Follow up to the fix
for #353.

See merge request !566

Use TCU_CHECK instead of DE_ASSERT

DE_ASSERT is optimized away in release build. In this case it resulted
in the 'modifyXor' function not being called, causing the reference
memory to go out of sync with the actual memory.

Fix several issues in dEQP-VK.memory.mapping tests

This is a very manual cherry-pick of memory mapping test fixes from
development branch into the release branch.

Fixes #353

Merge branch 'cherry-pick-validation-fixes' into 'vulkan-cts-1.0'

Cherry-pick validation fixes to release branch

See merge request !563

Merge pull request #16 from jchen10/image_layout

Fix Pipeline barrier test

Fix test copying zero sized regions

Bug: 28677103
Change-Id: I4c23277f51ab4ea3e28a76ab757d0c2c8fe007d6
(cherry picked from commit 5dcb4c684341f681eea37ffc4d1727b2239516ef)

Update sample conformance statement with latest release tag

Remove use of non-standard strnlen()

(cherry picked from commit febb0bbd97bc3004df7e81f90e9a5ddb49504aac)

Validation fix dEQP-VK.binding_model

- Add srcStageMask to barriers

(cherry picked from commit f2e4f11a0803db89f7c7d8f9a9038a79e27a65ea)

Validation fix for dEQP-VK.ssbo

Fixed an issue where descriptor set bindings were updated incorrectly
(array of descriptors vs a few bindings of a single descriptor).

(cherry picked from commit 517e989710e57ac0008c0ea26bafb67418e52610)

Remove dEQP-VK.glsl.linkage.varying.rules.fragment_declare

Test removed because it used the API incorrectly.

(cherry picked from commit 077398babe16b80a3934ec970932a3c80ae02bc1)

Validation fixes for dEQP-VK.renderpass

- Correct access flags in layout barriers
- Remove duplicate barrier
- Fix minDepthBounds value

(cherry picked from commit ba01feddc2c4b2cafba566462732ac2f3e813c83)

Validation fixes for dEQP-VK.glsl

- Use uint type for booleans to match attribute format
- Don't bind descriptor set when it's not updated
- Use correct number of vertex input bindings
- Use correct color attachment format and layout
- Use correct access flags and layout in image barriers

(cherry picked from commit 5533f025bb144cdeee813ebabba2885e14b33b2e)

dEQP-VK.binding_model validation fixes

- Add image layout transition barrier flags
- Fix shader interface mismatch
- Add fence after command submission
- Remove buffer barrier from render pass

(cherry picked from commit ad5e2b6869cdbe14bd7f9847b2eef498f7a41a8c)

Removed not needed reading of image

(cherry picked from commit 908d6bebb1e8149a8ec877853fc4a84641923425)

Bug Fix: Corrected barriers in the tests dEQP-VK.query_pool

(cherry picked from commit 0319b008a20f6c61504d39c118c0dc271e015c25)

Validation fixes for dEQP-VK.api

- Correct minDepthBounds value
- api.copy_and_blit: corrected image layout barriers usage

(cherry picked from commit ed9d438ecd49d41e86247421c253692b92bcef63)

dEQP-VK.api.info.device.properties struct offsets

Use correct offsets and comparison algorithm when checking
VkPhysicalDeviceProperties structure.

(cherry picked from commit 33878d45346dddf85669f610783939cad0d32817)

Bug Fix: Corrected barriers in the tests dEQP-VK.dynamic_state.

(cherry picked from commit 188341def51b4a425163576ce1cd1c8931c45df8)

Turn on well defined signed integer overflow

clang and gcc assume that signed integers never overflow.
This leads to unexpected bugs when optimizer removes bound checks etc.

Bug: 27703278
Bug: 22715185
Change-Id: Iab4c884fa190d05c0ac9236d01eeeb190e765e69
(cherry picked from commit ae6c306e39080de59a8535cf1797b27951dcaa6c)

Add Android NDK r11 toolchain as new default

New NDK r11 toolchain fixes libc++ on x86 enabling glslang support in
Android builds.

Change-Id: I5141b97f3152cfb1420a116cae40d671477ed254
(cherry picked from commit de7fb14fc30a8dd60e0215c333950b6a996c825a)

Update glslang to (7b3096a) to include bitfield functions precision fix

This fixes issue #385

Fix Pipeline barrier test

Set the old layout of color attachment image to VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL

Update glslang to (af4592) - include proper RelaxedPrecision handling

Without this all build-in functions get RelaxedPrecision decoration in fragment shader, making _many_ tests fail.

Remove instances of pDynamicState->dynamicStateCount = 0

Specification explicitly disallows using
VkPipelineDynamicStateCreateInfo with dynamicStateCount = 0 when
creating pipelines. Instead pDynamicState must be set to NULL.

Change-Id: Iff9e2f10e0080ffae0f6698aff342132f148b41d
(cherry picked from commit 86a8ae9e0d61bbfe1f1e273fdbb026369bdb5f29)

ShaderRender tests: do not use images in UNDEFINED layout

Just transition them at the beginning of renderpass

(cherry picked from commit 9f60392d15906ca26e0ae2ec97fe3e2c40f69a1f)

Merge branch '357-ports-to-release-1.0' into 'vulkan-cts-1.0'

Check existing tests with validation layers - partial port to 1.0 branch

This is a port of subset of MRs listed in #357 - the ones that fix usage of images in UNDEFINED layout.

We need this one because of implementation change on our side.

The contents of this MR equals !526 + !516 + !510, all of these are in MERGED state.

See merge request !534

Revert to original boilerplate in Vulkan CTS license appendix

Validation fixes in dEQP-VK.pipeline

- Corrected out of range minDepthBounds value
- Added missing image layout transitions
- timestamp, cache tests: pass data through geometry shader
- cache test: use correct initialLayout in attachment when starting
  a second render pass

(cherry picked from commit 1d3585c5ec71cf90d6519913db6a01ef2c0f840c)

Conflicts:
external/vulkancts/modules/vulkan/pipeline/vktPipelineCacheTests.cpp

Bug Fix: Corrected barriers in the tests dEQP-VK.compute

(cherry picked from commit 9b3e984f996ac8d7cc18c16e466a149eef2397e3)

Bug Fix: Corrected barrier in the tests dEQP-VK.ubo

(cherry picked from commit ee29316060b9858e85c5aabb2a6c563c6d34826a)

Fix unused parameter warning in vktDrawIndirectTest.cpp

Merge branch '355-fix-tesc-tese-executors' into 'vulkan-cts-1.0'

Fix a few issues in tessellation shader executors

 * Off-by-one buffer overflow in TessEvaluationExecutor

 * Set patchControlPoints to sensible values

Fixes #355

See merge request !519

Merge branch 'scygan-draw-tests-split-firstInstance-1.0' into 'vulkan-cts-1.0'

deqp-vk-draw: split indirect_draw tests into groups depending on `drawIndirectFirstInstance` optional feature

Fixes #337 on 1.0 branch, as an alternative to !507

See merge request !512

deqp-vk-draw: split indirect_draw tests into groups depending on `drawIndirectFirstInstance` optional feature

Remove initialization check for deviceName

Initialization checking code doesn't currently handle null-terminated
strings correctly. Implementation is not required to initialize bytes
following the terminator 0 byte in string fields.

Fix VkPhysicalDeviceProperties init validation

dEQP-VK.api.info.device.properties was using incorrect member offset
table to verify initialization of VkPhysicalDeviceProperties.

Fixes #367

Improve VK_MAKE_VERSION and VK_BIT macros

VK_MAKE_VERSION was causing int->uint conversion warnings.

Put arguments inside parenthesis to make evaluation unambiguous.

Change-Id: Ia47efdf77789905f4c45a551c08eb0ff3496c556
(cherry picked from commit dd99172aee72493b3e508dc8990e7eac95a40f8f)

Merge branch '361-memory-properties' into 'vulkan-cts-1.0'

Fix dEQP-VK.api.info.device.memory_properties

Includes both fix for missing propertyFlags 0 as well as improved logging.

Fixes #361

See merge request !518

Fix a few issues in tessellation shader executors

 * Off-by-one buffer overflow in TessEvaluationExecutor

 * Set patchControlPoints to sensible values

Change-Id: Ie94cf02c2fccd5e018a0e4de02c956f0d2abc6b0

Add 0 to list of valid memory property flags

Fixes #361

Improve logging in dEQP-VK.api.info.device.memory_properties

Change-Id: I70927510e3dce6dbc05cc0f49f24f5d331f49412
(cherry picked from commit cdf80ba37ef186bb2d8fc5a3a97dd633e6a67d85)

Merge branch 'extend-query-validation' into 'vulkan-cts-1.0'

Extend feature and limit query validation

Manual cherry-pick of extended validation code in `vktApiFeatureInfo.cpp` from the development to the release branch.

Includes framework API update to 1.0.5 as the new code requires `VK_FORMAT_FEATURE_SAMPLED_IMAGE_FILTER_LINEAR_BIT`

See merge request !499

Extend feature and limit query validation

This change manually cherry-picks extended feature and limit query
validation code from development branch to the release branch.

Merge branch '340-cube-corner-mustpass' into 'vulkan-cts-1.0'

Remove test affected by cube corner texel selection issue

See #340

See merge request !505

Merge branch '352-missed-pipeline-barrier' into 'vulkan-cts-1.0'

Add missing memory barrier to draw tests

Fixes #352

See merge request !493

Merge branch '350-LOD-precision-for-snorm-cts-1.0' into 'vulkan-cts-1.0'

Use larger threshold for EAC formats in sampler tests

Fixes #350

See merge request !490

Add missing memory barrier to draw tests.

Fixes #352

Use larger threshold for EAC formats in sampler tests

Fixes #350

Remove test affected by cube corner texel selection issue

 Fix int->float conversion warning

Change-Id: I3a8c01a9223b1b261b2cf162ab0ef9babb8bbb45
(cherry picked from commit de443fdd0e0f38f5fd8bd02aa27e6931063a39e3)

Update Vulkan framework to API 1.0.5

Change-Id: I4e469b9384701303f96826c25a51e623316719dc
(cherry picked from commit 931d1df9bc5ac16200e2e63dfce17dcbbbb35d36)

Merge branch 'change-license-rel' into 'vulkan-cts-1.0'

Change license in release branch

See merge request !481

Change Vulkan CTS license to Apache 2

Fix malformed copyright boxes

Merge branch '341-fix-invocations' into 'vulkan-cts-1.0'

Remove invalid Invocations=0 from geometry shaders.

All of these geometry shaders are default, so they do not need any
Invocations decoration.

Fixeds #341

See merge request !477

Remove invalid Invocations=0 from geometry shaders.

All of these geometry shaders are default, so they do not need any
Invocations decoration.

Merge branch 'side-effect-of-322_api-patch-version-test' into 'vulkan-cts-1.0'

Add a test for checking random values of PATCH component of VKApplicationInfo.apiVersion

This is a continuation of #322, where negative test for checking random patch versions is removed.

Here, a positive test is added instead to ensure that vkCreateInstande does NOT fail if PATCH version is unrecognized, weird or just 0.

See merge request !462

Merge branch '329-fix-undef-tests' into 'vulkan-cts-1.0'

Fix createOpUndef tests.

If we get a Nan from the OpFMul %f32 %undef %c_f32_0 then zero that
out as well.

Fixes #329

See merge request !473

vktApiDeviceInitializationTests patch iteration added

Merge branch 'jbreton-buffer-test-heap-limit' into 'vulkan-cts-1.0'

Fix createBuffer tests to clamp the buffer size based on the size of the selected heap

Right now, those tests clamp the allocation size based on the size of heap 0.
This is incorrect if the allocation is not done on heap 0.
In order to get the first compatible heap, we need to create the buffer and then call getBufferMemoryRequirements to query the requirement.

Then if we detect if the size need to be clamped we will recreate another buffer with the clamped size.

Fixes #326

See merge request !464

clamp allocation size based on heap size

Fix createOpUndef tests.

If we get a Nan from the OpFMul %f32 %undef %c_f32_0 then zero that
out as well.

Merge branch '325-spirv-assembly-avoid-rounding' into 'vulkan-cts-1.0'

dEQP-VK.spirv_assembly.instruction.compute.*: avoid platform-dependent rounding

Avoid platform-dependent rounding in tests for OpCopyObjectGroup, OpPhiGroup, SelectionControlGroup and FunctionControlGroup.

See merge request !461

Merge branch 'issue-307' into 'vulkan-cts-1.0'

Issue 307: Add new --deqp-log-flush command line option

Fixes issue #307

It defaults to enabled. When disabled it skips calling fflush() for every XML token and line of log.

See merge request !441

dEQP-VK.spirv_assembly.instruction.compute.*: avoid platform-dependent rounding in tests for OpCopyObjectGroup, OpPhiGroup, SelectionControlGroup and FunctionControlGroup.

Merge branch '323_timestamp_buffer_overflow_fix' into 'vulkan-cts-1.0'

Fixes buffer overflow in pipeline timestamp tests

Fixes #323

See merge request !459

Merge branch '310-update-glslang' into 'vulkan-cts-1.0'

Update glslang to include "SPV: Use heuristic to avoid geometry multi-streams"

Fixes issue #310

See merge request !463

Merge branch '308-fix-opaque-type-indexing-array-descriptors' into 'vulkan-cts-1.0'

ShaderExecutor: use descriptor arrays when uniform/buffer arrays are used in the…

… shader code. Another partial solution to #308.

See merge request !455

Merge branch 'spec-176-api-remove-patch-version-check' into 'vulkan-cts-1.0'

Issue 322: dEQP-VK.api.device_init.create_instance_invalid_api_version - remove patch version check

In the spirit of spec bug #176, the implementation should not emit error on unrecognized PATCH component of VkApplicationInfo.apiVersion.

This makes implementation compatible with all header patch versions that may be used by the application.

This fixes #322

See merge request !454