docs: Add governance doc, Update readme, contributing

Change-Id: I92bebd8a33df564daff1b44633e79b56ffa6ae86

layers: Add validation caching for draw/dispatch

Validation of descriptors at draw or dispatch time is now cached, s.t.
without changes that affect validity, validation checks are not
repeated.

Change-Id: I713662d00813989bf4441921456afca431d730d7

layers: Replace iterative DS Get.* with maps/sets

Profiling indicated hot spots in DescriptorSet Get functions which
iterated in binding space to lookup values.  Replaced iterative searches
with maps/sets.

Additionally simplified construction, optimized map/set creation and
Get.* for DescriptorSet and DescriptorSetLayout.

Change-Id: Ia2948e56333d3643d4377b39e75acf4c951d558b

layers: Add cache of memory binding set for perf

Added a cache of the GetBoundMemory set to avoid its continual
regeneration at validation time.  Call appeared in profile for high
descriptor count sets.  Also cleaned-up memory binding state setting
to avoid code duplication.

Change-Id: I84beb411a83ab311d371be940e629fdb308f5458

layers: Combine global binding start/end lookup

Binding start/end lookup where using two separate maps indexed on a
common key.  Combined to avoid repeated lookups.

Change-Id: I57ed71736537de5eb83fe41599a8acec9bf23d13

icd: GetImageSubresourceLayout zeros passed layout

Calling tests use return values from GetImageSubresourceLayout to
compute offsets in texures.  Zero'd values are safe for this usage and
add minimal overhead to icd.

Change-Id: If52cf240925d2f7a41bb22d87519a918be5eb60a

build: Fix application of /bigobj compile flag

Fixes #2316
Affects only Windows builds.
Need to compile code that generates large object files
with the /bigobj option in order to avoid exceeding the
default number of sections in an object file.
This change applies the /bigobj compiler option to all
configurations, rather than just Debug.
It also applies it only to selected files, instead of
all the files in the project.
Although /bigobj is now applied to Release builds,
this does not increase build time.

Update android-build/cmake scripts to build with Android Studio 3.0.0+

icd: Move include to header to fix clang build

Move include <string> to header file to fix
clang build when using libstdc++.

Fixes #2303

Change-Id: Id2fc97be7cb83fa12ee1495ef57177c0a7dad893

build: Retarget AppVeyor build to VS2013

Change-Id: I88665dadbac75695034a731412984edbecbf988f

loader: Change overzealous warning to debug msg

Change-Id: Idde5f8c8f009e365bdcf1fffaffa669d0557d66b

build: Update BUILD.md to specify VS2013

Change-Id: I7533459cd338e233aad4d81ffc3c22aa87db1937

tests: Add a guard around noexcept in vktestbinding.h

VS2013 does not provide 'noexcept'. Layers had multiple occurences
of a NOEXCEPT macro to protect the VS2013 build. This commit
consolidates that macro to a single location in vk_sdk_platform.h,
which is also visible to the layer tests via test_common.h.

Updates the vktestbinding.h occurences of 'noexcept' to use the
NOEXCEPT macro.

Change-Id: I38bf8c08cda6fe75de78dff69050a913b47f91b1

build: Remove APP_MODULES from android Application.mk

This option is optional if all components are to be built. Removing
this will make VulkanTools trunk merges safer and easier.

Change-Id: I4c0bb13bf23924c49bb1b3bde1eaa26de1b180df

layers: Accept non-unique shader caps

Change-Id: If14c4c58a6b6df04e97f0f83d8f8eb9b26e866ad

Proper decoding for msbuild version number readout

layers: Change target name for devsim json

Fixes duplicate targets in layers / layersvt for VulkanTools

Change-Id: Ieb2577454743ac085153b04b2cbb3fcb25918578

layers: Fix git.bat fallback on Windows.

This fixes the external revision generator script to first try to
use 'git', then fall back to 'git.bat' if the first invocation throws
a WindowsError.

build: Address clang compilation warnings

android: Roll back to NDK r15c

Until we can sort out issues with NDK r16 preventing us from
completing 32-bit test runs.

android: Add missing shaderc file

scripts: Scan CWD in object tracker generator.

This makes it more easy to locate a header in configs which have the
CWD set to something less obvious. In Chrome's case the build directory
is completely separated from the script directory.

Also reset the CWD after the header is found to keep the build intact.

This is similar to the change to the parameter validation generator.

demos: Remove vulkaninfo dependency on DISPLAY

layers: Remove unused variable from core validation.

This commit was accidentally reverted in dec37fb9172:
layers: Add descriptor limit checks to vkCreatePipelineLayout()

build: Add clang-format checking of pull request changes to CI

Check modified lines in a pull request for correct clang-format.
Note that checking is not performed unless there is a pull request.

Heavily leveraged from https://github.com/KhronosGroup/SPIRV-Tools.

layers: Ignore pImmutableSamplers on non-samplers

layers:Kill PV return code check

Most of these return code warnings were already removed from validation
so killing this one as well. They can potentially be re-added in an
assistant layer.

layers: Work around bugs in old STL versions.

Some old versions of the C++ STL have a few bugs with advanced features.
This works around two such issues:

- std::unordered_map::const_reference having the wrong type
- an error with an explicit std::vector constructor

android: Update to NDK r16b

This allows us to switch to libc++ runtime as it is no longer beta:
https://android.googlesource.com/platform/ndk/+/ndk-release-r16/CHANGELOG.md

scripts: Scan CWD in parameter validation script.

This makes it more easy to locate the header in configs which have the
CWD set to something less obvious. In Chrome's case the build directory
is completely separated from the script directory.

Also reset the CWD after the header is found to keep the build intact.

scripts: Prefer git.bat on Windows.

Calling git.bat seems to work in every instance, whereas some
installations don't have access to "git" directly.

demos: GH1054, make cube shaders inline

icd: Fix json for windows build

layers: Fix Graphics Pipeline pointers not ignored

Some VkGraphicsPipelineCreateInfo pointers must be ignored under some
conditions, but were not in the layers.

Add relevant tests.

Fix tests found broken (using depth or color without attachment in
subpass)

layers: improve const-ness in buffer_validation.cpp

Change-Id: Ic7a2ef6e26b76c1b5f3a4c279e0d15228513af9e

layers: Repro and fix GH 2239

Modified positive test UncompressedToCompressedImageCopy to provoke
the issue reported in GH 2239

Fixed destination extents calculations in buffer_validation.cpp to
avoid the erroneous errors

Change-Id: I268f14a6be678d767df14f0a6694aca479854c3

layers: Fix PV overflow

Buffer pCreateInfo->size was being truncated to uint32_t before
comparison with 0

Change-Id: Iba6b94f2a0df6514d1ffb79d38e6c0738ea44d71

layers:Add VUID to existing check

Add VUID to existing check for VALIDATION_ERROR_0f6004ac and update
the DB file to reflect that this check is in place.

layers: Add some missings locks of global_lock

Add locks of global_lock in BindBufferMemory, BindImageMemory
and GetImageSubresourceLayout to have safe access to imageMap
and bufferMap.

icd:Update README with current status

build: Clear some compiler warnings

- remove unnecessary ";" after function definitions and namespaces
- fix
wrong type match in printf for handle types
- convert some left-overs to `HandleToUint64`

tests: Add tests for VUs 0fe0023c-246

Add CreatePipelineLayoutExceedsSetLimit() which tests 0fe0023c.
Add CreatePipelineLayoutExcessPerStageDescriptors() which tests
VUs 0fe0023e, 0fe00240, 0fe00242, 0fe00244 & 0fe00246.

Change-Id: I3795e6a726c10626b33bac254c396db5fb8a8e6c

layers: Add descriptor limit checks to vkCreatePipelineLayout()

Change-Id: I570dc49652997cfaed7dcd6dfaf2e6f125f861cc

Revert "layers: Add descriptor limit checks to vkCreatePipelineLayout()"

Remove checks to avoid database complaints

This reverts commit af313b4683b42a9e4d15aa5819feddc1b9eee6d8.

Revert "tests: Add tests for VUs 0fe0023c-246"

This test fails on google's Vega system

This reverts commit 29cedfebcc891d85e938893166a3d919fc2845d2.

winsdk: Move file copying into NSIS installer

Change-Id: If01c5024af802be784d5ea0f897cebdcc604934d

tests: Add tests for VUs 0fe0023c-246

Add CreatePipelineLayoutExceedsSetLimit() which tests 0fe0023c.
Add CreatePipelineLayoutExcessPerStageDescriptors() which tests
VUs 0fe0023e, 0fe00240, 0fe00242, 0fe00244 & 0fe00246.

Change-Id: I3795e6a726c10626b33bac254c396db5fb8a8e6c

layers: Add descriptor limit checks to vkCreatePipelineLayout()

Change-Id: I570dc49652997cfaed7dcd6dfaf2e6f125f861cc

layers: VS2013 compatibility

Remove all usages of constexpr to restore compatibility with Visual Studio
2013.  Also make a handful of other minor edits where the VS2013
compiler was reporting errors.

Change-Id: I3c426bea460a4a902ab306b01deb1d76d60f5fa2

build: Remove clumsy syntax in Travis-CI config

We were using a clumsy workaround in order to
get good failure information when a Travis-CI build
failed and also to make it stop at the point of
failure.

This commit uses some techniques to workaround
travis-ci/travis-ci#1066, which asserts that
a Travis-CI job should stop when a before_install
or script command fails.  Today, Travis-CI
continues with the next commands.

This was tested with both an intentional build
script error and a compilation error.  In both
cases the Travis-CI job ended at the point of
failure and the logs were clear enough to
diagnose the problem.

tests: Separate LogicOp test

- separate LogicOp test so it does not choose what it is testing at
runtime
- update to `CreatePipelineHelper`
- use VUIDs and update database
- should fix one of the "Unexpected" in CI

tests: Update MismatchedQueueFamiliesOnSubmit

- report clearly if it is skipped
- don't rely on test state for the pool
- use VUID and update database
- should fix one of the "Unexpected" in CI tests

tests:Add new case to ClearDepthStencilImageErrors

Add test case where VK_IMAGE_USAGE_TRANSFER_DST_BIT is not correctly
set for an image that is passed to vkCmdClearDepthStencilImage().

tests:Fix unexpected error

Update ClearDepthStencilImageErrors test to remove unexpected error.
Updated the error check to use VUID instead of text.
Also, this is the new name of the test in anticipation of testing at
least one more error condition in this same test.

layers:Add image checks for DS image being cleared

Adding checks for two new VUIDs related to vkCmdClearDepthStencilImage
function. Making sure that the image being cleared supports use as a
transfer destination and that the image was created with the usage bit
VK_IMAGE_USAGE_TRANSFER_DST_BIT.

cmake: Fix layers/icd cmake files for in-tree builds

See Github #2243.

Change-Id: I40119c73a1518357df4360654c888b8306aa94d8

layers: Add warning for LOAD_OP_LOAD+LAYOUT_UNDEFINED

While not technically an error (VK_IMAGE_LAYOUT_UNDEFINED is always
allowed as an initial layout), this combination is almost never what
you actually want.  If nothing else, it's a potential performance
problem because VK_ATTACHMENT_LOAD_OP_LOAD is liable to be more
expensive than VK_ATTACHMENT_LOAD_OP_DONT_CARE.  Give developers a
helpful warning in this case.

This would have caught an actual bug in some of the Sascha Willems
demos as fixed in this PR:

https://github.com/SaschaWillems/Vulkan/pull/400

Change-Id: I73c64ae60a65cfb200db00707d650ec8da7ba130

Revert "tests: Update MismatchedQueueFamiliesOnSubmit"

This reverts commit 21420916e4dee67474655fe08b3d1125598b407d.

tests: Update MismatchedQueueFamiliesOnSubmit

- report clearly if it is skipped
- don't rely on test state for the pool
- use VUID and update database
- should fix one of the "Unexpected" in CI tests

layers:Limit warning on secondary command buffer

Only warn if a secondary command buffer has a renderPass that is valid
and has not set VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT. This
avoid warning for the case where uninitialized data is present in the
renderPass handle.

layers: Remove unused variable from core validation.

loader: Fix Clang errors/warnings.

Fixes:

format specifies type 'int' but the argument has type 'DWORD' (aka 'unsigned long') [-Werror,-Wformat]
error : expected ')' (issue with __FUNCTION__ and format string)

build: Change AppVeyor config to use multiple jobs.

This is a complete rewrite of the AppVeyor config file.

A separate job is used for each config in the Platform-Configuration
matrix.  This helps get around the problem of the 60-minute limit per job.

The invocation of the update_external_sources script has been
replaced with individual commands to build glslang and spirv-tools
only for the platform configuration selected for each job.
This avoids having to build glslang and spirv-tools 4 times for each job.

layers:Add secondary CB warning

Fixes #2237

If VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT is not set by a
secondary command buffer, then any renderPass specified in the
inheritance info will be ignored. Adding a warning if a non-NULL
renderPass is included in the inheritance info but the
VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT is not set. This is
likely an oversight by the user and the current validation error that
this case leads to (no active renderPass) doesn't directly point to the
root cause.

demos: Fix barrier settings for texture in cube demos

Change-Id: Ibaa3901daa7a632022b67e53dd34f79be594ded1

tests: Tests for pipeline multisample pnext check

Added test for validation checks of
    VUID-VkPipelineMultisampleStateCreateInfo-pNext-pNext
    (VALIDATION_ERROR_1001c40d)

Change-Id: I41fb10698d88db1e4b143007c36b5a798e1adc6b

layers: Update pipeline multisample pnext check

Updated validation check for
VUID-VkPipelineMultisampleStateCreateInfo-pNext-pNext
(VALIDATION_ERROR_1001c40d) to include the valid extension pNext sType
values valid for the current spec:
    VkPipelineCoverageModulationStateCreateInfoNV,
    VkPipelineCoverageToColorStateCreateInfoNV, or
    VkPipelineSampleLocationsStateCreateInfoEXT

Change-Id: Ic3e054110c2bad4b44121a67e7e61e627fa9c559

tests: Added PipelineMultiSample VUID tests

Added tests for validation checks:

VUID-VkPipelineMultisampleStateCreateInfo-sampleShadingEnable-00784
(VALIDATION_ERROR_10000620)

VUID-VkPipelineMultisampleStateCreateInfo-minSampleShading-00786
(VALIDATION_ERROR_10000624)

Change-Id: I49fcfc8f55275f27a976ea88619674c3b87285ae

layers: Added PipelineMultiSample VUID checks

Added validation checks for:

VUID-VkPipelineMultisampleStateCreateInfo-sampleShadingEnable-00784
(VALIDATION_ERROR_10000620)
    If the sample rate shading feature is not enabled,
    sampleShadingEnable must be VK_FALSE

VUID-VkPipelineMultisampleStateCreateInfo-minSampleShading-00786
(VALIDATION_ERROR_10000624)
    minSampleShading must be in the range [0,1]

Change-Id: I21cdaf99f8970df4f2e03026a4dae00746ad8386

layers: validation_cache review feedback

tests: Add test for vkMergeValidationCachesEXT()

layers: Check EXT_validation_cache's sole non-trivial VU

A cache can't be merged with itself.

layers: Tidy up shader cache, reserve space before merging

inlined commitIdLen into assert

Convert SPIRV-Tools commit ID to binary before use as a UUID

This doubles the number of effective hash bits from 64 to 128.

Fix build errors:

- No more SPIRV-Tools revision file. Instead, the
  revision-parsing script just runs "git rev-parse HEAD"
  in the spirv-tools repository to extract the commit ID.
- Invoke the revision-parsing script from the .sh Android scripts,
  in addition to the .bat scripts.

Fix for Android build error with Linux host

Use SPIRV-Tools commit ID as validation cache version

shader module codeSize is already in bytes

layers: Add layer metadata for EXT_validation_cache extension

layers: Add VK_EXT_validation_cache implementation for shaders

layers: Add xxhash

Add missing include in android_util.cpp

loader: fix memory leak for layers with >1 dev_exts

layers: Fix typos in viewport state validation msgs

tests: Implement lineWIdth state checks

layers: Fix lineWidth checks

- migrate to parameter_validatio
- do not check against limit (outside limits is valid; impl clamps
value)

layers: Quick fix odd create PSO param checks

- param check did only check the first create info
- `cullMode` check should be redundant to the automated part

header: Update to version 1.0.66 of the Vulkan hdr

- update vulkan.h
- update Lin/Win json file versions
- update vk.xml
- update vuid_mapping.py
- update vk_validation_error_messages.h
- update vk_validation_error_database.txt
- update vulkan.hpp
- fixed error in vk.xml, unofficially

Change-Id: Ia719bc069cb776c5b174f41667d591b43aaaf4d7

layers: Fix MSVS conversion warning.

This fixes the following:

warning C4245: 'argument': conversion from 'int' to 'unsigned int', signed/unsigned mismatch

It also modifies the build to enable this warning by default.

Change-Id: If2e6c8d43811162a9a382883b3d55a148975fc37

loader: Fix improper cast warning.

warning C4057: 'function': 'LPBYTE' differs in indirection to slightly different base types

This was popping up in some Windows builds. Also add it to the list
of warnings enabled by default with MSVS.

Change-Id: I2703f2e57ba3fb33ec03fbc7b8324c75187abec6

build: Fix gslang->glslang errors in BUILD.md

Change-Id: I221fde5b016cf36ec1047287dbfb82a1a81e0b09

build: Updated BUILD.md for spirv-tools changes

Change-Id: Ic2f62a226474dbe5705be2706efc093b3dde7b9e

build: Removed unused spirv-tools/headers rev files

Change-Id: Ie199e1b2b94a7bc8c2ea998a7725d307e9a369f8

build: Avoid redundant spirv-tools/hdrs builds

Changed update_external_sources scripts to use spirv-tools and
spirv-headers already present in the glslang repo instead of
fetching and building separate versions.

Change-Id: Iff9666f1939d814d0888b2da9b9e7450af697722

scripts: Update toolchain revision check script

No longer need to check spirv-tools or spirv-headers as they are now
part of glslang.

Change-Id: Ib3594d064d33a94d6443426e02a617c96b243893

tests: Added GetQueryPoolResults test case

And updated the error database.

Change-Id: Id293ee836866ce9bbfc3a4eda4ee0e8efd0e09ab

layers: Add GetQueryPoolResults validation check

Added check for a pool of queryType TIMESTAMP having the RESULT PARTIAL
bit set and updated the error database.

Change-Id: I139733202c085a7d64d7359f659714b75a46ebf1

layers: Remove invalid query avaibility error check

This case in GetQueryPoolResults flagged errors for cases that are
valid.

Change-Id: I00a823992d200cd30988fde39ef679ea8ad8bce3

layers: Refactor PreCallValidateGetQueryPoolResults

Cleared up logic flow.

Change-Id: I32157a408a7ff7e413ab82b05bf86de48f98495b