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

tests: validation_cache review feedback

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

tests: Fix UncompressedToCompressedImageCopy

Created images were implicitly using TILING_LINEAR,
despite testing for TILING_OPTIMAL support.

Fixes failure on Vega mentioned in #2153

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

tests:Fix MSVS build warnings

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

tests: Fix indentation in ImageBarrierWithBadRange

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

layers: Remove redundant query_state_pair checks

Change-Id: I62b722124c58d98da642cc2c5c17670a7f290d64

tests: Add test case for vkCmdBlitImage() image layouts

layers: add image layout validation to vkCmdBlitImage()

tests:Fix DynViewportAndScissorMismatch crash

This test was crashing on Nvidia drivers.
The crash seems to be a driver bug as the workaround is to not destroy
a descriptor set layout prior to creating gfx pipelines whose
pipeline_layout object was created from the given descriptor set layout

tests:Fix signed/unsigned compare warning

tests:Fix testname typo

layers: Rewrite ViewportState checks and tests

- deal with #1685
- report correct object type
- improve? msg texts a bit
- add extension structs to pNext check
- use correct printf token where possible
- make tests to match  changes + improve coverage (nullptr +
multiViewport)
- remove viewport state (and other stuff) from tests that do not use it
- merge some tests together that are too related
- remove unnecessary dynamic state from PSOLineWidthInvalid)
- correct invalid VkViewport in some places
- correct missing/null pViewport and pScissors in some places
- update and correct the VUID database

tests:Add DevSim testing for 7 profiles

Update Travis-CI run LVL tests on seven different device profiles.

layers:Add pre-built DevSim layer

Checking-in DevSim binary for linux to use in Travis-CI testing along
with seven initial json profiles to be tested w/ DevSim on top of mock.

tests:Force 64bit format support for vtx buffer

For CreatePipeline64BitAttributesPositive test enable the DeviceProfile
layer in order to make sure that vtx buffer support is available for
VK_FORMAT_R64G64B64A64_SFLOAT.

tests:Fix ParameterLayerFeatures2Capture

Fix ParameterLayerFeatures2Capture so that it only requests to create
queues from families that have a queueCount greater than 0.

tests:Fix memory allocation sizes

Update QueryAndCopySecondaryCommandBuffers and
QueryAndCopyMultipleCommandBuffers tests to allocate buffer with size
based on the returned size requirements instead of hard-coded value.

tests:Add "--devsim" option

Add "--devsim" option to vk_layer_validation_tests. This will enable
the Device Simulation layer as the bottom layer on the stack.
It still requires the caller to set the VK_DEVSIM_FILENAME env var in
order to select the json profile that they want to test with.

When "--devsim" option is specified, the device_profile layer will not
be enabled by default when InitFramework() is called. Instead, any
tests that require the device_profile layer can call
EnableDeviceProfileLayer(). This function will cause the device profile
layer to replace the dev sim layer as the bottome layer on the stack.

icd:Increase WHOLE_SIZE mapped memory

Bump hard-coded mapped memory allocation size for WHOLE_SIZE from 4k to
64k. One validation test (InvalidMemoryMapping) was mapping 64k of
memory that was leading to segF as the 4k buffer was overwritten when
shadowed.

tests:Only init queue families that have queues

It's possible for a device to return queue families that don't have
any queues, but we should not be requesting creation of queues from
such a family when we create the device as it's against the spec and
leads to a parameter validation error..
This change only requests that a device initiate queue families that
have at least one queue.

tests:Get supported memory type when allocating

Before allocating memory for a buffer make sure that the requested
memory index type will support the required memory properties.

icd:Expand mock icd memory type bit support

Image/buffer memory requirement memoryTypeBits are still just hard-
coded in mock icd, but expand to support more bits for more simulated
memory types. This helps the mock work in conjunction with the devsim
layer so that if a simulated device has more than 5 memory types, the
mock icd won't cause an error.

tests:Detect buffer creation errors

Handle case when VkBufferTest fails to create the buffer. This means
setting the buffer as no longer valid, and checking for this case
in the test body so we don't try to use an invalid buffer.

tests: Make ExternalMemory test more robust

- Use dedicated allocation when required by the external memory implementation
- Add required pNext structs for all buffers and memory
- Relax the requirement that external memory be host visible
- Verify that the export/import memory is actually linked

Change-Id: Ia12f8cc9846c781efcbac6f3d448f4254ad2daa0

layers: Validate device dispatchable objects

Device object validation required dereferencing possibly bad device
objects. Moved ownership of device objects to instance objects, and
to search each of the layer_data object lists for a created device
handle matching the our candidate device object.

Change-Id: If5615294c397ff6991d4d21ac75ab58d1b0b0841

scripts: Fix incorrect object type mappings

'Unknown' type was at the end of a couple of the lists, causing
incorrect IDs to be emitted.

Change-Id: I55cae6ecaff47977dca78e2f73468980f9450bda

layers: Move BuildTemplate call inside global lock

Helper routine was being called outside of the lock causing
freezes in Dota2

Change-Id: I45ab0b23c6ff3ed4ec16dca15107ce16f205599f

tests:Add doc validation to travis CI

loader: Fix pre-instance proc addresses

Fix a bug where vkGetInstanceProcAddr would return a null pointer for
any command that does not require an instance if an instance was
given

Change-Id: Iec5766f9c37104c36852f758d391a9b0b54ec0ec

docs: Update some Android sections

android: Only compile toolchain once build_all.sh

layers: Don't confuse VkResult/VkBool32 in generators.

In some cases, the generator would return the special error for
validation failed for functions that return a VkBool32. Fix this, and
also some cases of initializing a VkBool32 with VkResult.

Change-Id: Icc071e647f293848d62a922d492ed41dfc5a4be1

layers: Make generator helpers Python 2 compatible.

This is an easy compatiblity change, which uses a workaround to import
a version of 'open' that can support unicode across Python versions.

Affects the object tracker and parameter validation generators.

Change-Id: Ie958dc64168db9e06b36321b4b6ad2336f995ea3

layers: Return an error when generator scripts fail.

Currently the scripts would abort with an error message, but not an OS
error code. Adding an error code allows build systems to correctly
detect the script failure as a failed build step.

Change-Id: I142a6215a3c02e6a880bfce0c3aba2c3c8fc4ff1

layers: Adds createSampler VUID/test for IMG_CUBIC

Added VUID check to parameter_validation_layer and matching test for
VUID 12600872 (must disable anisotropy for CUBIC_IMG filters)

Change-Id: I3d0f9449413637292c2726480ca7ce0e6b793e21

layers: VUIDs/test for vkCreateSampler anisotropy

Added VUID checks to parameter_validation_layer and matching tests for
three VUID's:

  * VALIDATION_ERROR_12600868:
    Must disable anisotropy when unnormalized coordinates are enabled.
  * VALIDATION_ERROR_1260085e:
    maxAnisotropy must be in range [1.0, maxSamplerAnisotropy]
  * VALIDATION_ERROR_1260085c
    anisotropyEnable must be VK_FALSE if samplerAnisotropy feature is
    not enabled.

Removed test for: "samplerAnisotropy feature was not enabled ...
maxAnisotropy member ... must be 1.0" to conform with the resolution of
https://github.com/KhronosGroup/Vulkan-Docs/issues/503.

Change-Id: Ia1c561997adb9391ef463f6df90b424730366a7f

layers: Rename resetCB to ResetCommandBufferNode

Small cleanup change to clarify command buffer node code.

Change-Id: Ib98010e10161ef5b15aaa736d994ec48b4e659fb

layers: Free commandbuffer data when deleting pool

Fixes for https://vulkan.lunarg.com/issue/view/59eeea8166315153977b0fe6
Issue #720 - Possible crash in layers when deleting command pools.
Ensured pool commandbuffers fully freed in the layer before deleting the
pool using refactored common code from FreeCommandBuffers.

Added postive test which exercises the implicit free path.

Change-Id: I71e1fcb6ffcd9a2977a89c0a8b388639e9a743cb

build: Update BUILD.md to specify VS2015

Change-Id: I9180304a8089db057af68cbf86b73118e1d29c16

demos: Remove HTML in plain-text vulkaninfo output

Change-Id: I6f1192b71aa6f33664a6782dc81e1f8cb7cc5ba1

tests: Fix comment

Change-Id: If13c8393680baa96df4d24bf8d247ab9607f1b94

tests:Add comment

Fix formatting of one line and add a comment related to an offset
calculation.

icd:Update list of tests with unexpected errors

Only 2 tests remain with unexpected errors when running with default
mock ICD configuration. These same two tests also have unexpected
errors when running on actual HW.

icd:Reduce offset alignment limits

Buffer min offset alignments were set to the max value. Reduce them to
16 to simplify testing of offsets that are a multiple of the min
alignment.

tests:Update DSBufferInfoErrors

Allocate buffer memory based on requirements returned from device.

tests:Cleanup DSUsageBitsErrors on early return

If we can't find a good format, make sure to cleanup all of the objects
allocated thus far in the DSUsageBitsErrors test.

icd:Don't return support for Undefined Format

tests:Update InvalidDynamicOffsetCases

Allocate buffer memory based on size requirements.

tests:Fix unexpected error text

Update InvalidCmdBufferDescriptorSetImageSamplerDestroyed test to have
new unexpected error text based on "valid" being added to spec VU.

tests:Fix InvalidCmdBufferDescriptorSetBufferDestroyed

Fix buffer allocation for InvalidCmdBufferDescriptorSetBufferDestroyed
to be based on memory requirements.

tests:Fix InvalidCmdBufferBufferDestroyed

Size allocation for InvalidCmdBufferBufferDestroyed based on memory
requirements.

android: Remove mips targets

android: Update to shaderc known-good revisions

build: Add optional ccache support to cmake

travis: Use multiple cores for build

travis: Switch to ccache, skip Android toolchain build

scripts: Add --no-build for Android toolchain

With switch to importing shaderc, we don't need to
create prebuilts anymore.

Replaced fixed path python 2.7 executable calls

layers: Refactored pNext chain walks to template

The while loops for the walking the pNext chains were implemented by
repeated code. These were refactored into a common template.  Added
autogenerated 'traits' objects for the pNext linked structs.

Delete cut and paste duplicate pNext chain walk.

Change-Id: I46457bb5432219c74f9356e5230c70e4a9ef16df

layers: Protect GetImage/BufferMemRqmts map access

Lock 'global_lock' while looking for IMAGE_STATE/BUFFER_STATE in
GetBufferMemoryRequirements/GetImageMemoryRequirements. This is needed
because other threads could add/remove other VkImage handles to/from
the maps concurrently. The specs say the VkImage must be externally
synchronized, but this doesn't cover the internals of the layers about
VkImages in general. vkCreateImage, vkDestroyImage and
vkBindImageMemory/vkBindBufferMemory correctly lock 'global_lock'.

Change-Id: I93cb00c9a7f99efc927da52e42103ab72516397e

demos: Clarify cube --present_mode option

Change-Id: Ie53724c3612eb5bb9adfbb826149ef87916f0d84

icd: thread protect maps

Lock to protect threaded access to queue_map and mapped_memory_map.

demos: Add html backend and option to vulkaninfo

 - Run clang-format on vulkaninfo.c
 - Format Instance Extensions section
 - Complete Layers info tree
 - Convert spaces to tabs for html output
 - Fix indentation of html output file
 - Add Presentable Surfaces to html option
 - Ammend Presentable Surfaces html output
 - Extend platform compatibility
 - Display VkPhysicalDeviceProps in html
 - Convert DeviceLims and SparseProps to html
 - Convert memoryType/flags to support html
 - Complete AppGpuDumpMemProps html upgrade
 - Complete format properties html upgrade
 - Improve boolean logic for html_output var
 - Change output filename to vulkaninfo.html
 - Create auto-open format property flag info
 - Update page title, further .html fixes
 - WIN32 formatting and bug fixes
 - Clang-format changes
 - Close clang-format comments
 - Update file format to code style standard

Change-Id: I49c7c6c62a54493fe622bd457f7c61e81806487e

scripts:Cleanup comments and output for mock icd

Commented out a couple of debug output statements.
Killed a bunch of comments and code that were carried over from
original script but that are no longer relevant.

quickly wire mock icd into travis

tests: Refactor duplicated SamplerCreateInfo code

There were numerous instance of nearly identical SamplerCreateInfo
initializations. Refactoring improves readability and code size.

Change-Id: I1c8d085e1a92b088da11b7088878ed2ab7d20428

tests: Test layer Properties2 feature support

Added test which creates a device using features from
vkGetPhysicalDeviceProperties2KHR and the pNext pointer to set enabled
features. Test verifies parameter checking vs. features is correct for
two cases -- one case in each parameter validation layer and core
validation layer.

Change-Id: Iba5c0f161f088438bfdd7c37a01015c2fe1dbff1

tests: Remove duplicate queue creation code

Refactored identical code to create the pQueueCreateInfos from
queue_properties present in four functions.

Change-Id: I5e61549c17661611662ca6b39fd0163facd37526