loader: Rework include dependencies

This change removes the assumption that vk_layer.h will include
vk_layer_dispatch_table.h, since it will be removed from vk_layer.h
in the near future.

Change-Id: I3fed5efbc35781c96aa9d0977d046c8555e04a7c

tests: Remove validation error messages file

This is unneeded for Loader and causes dependency issues.

tests: Update JSON files for 1.1.79 header

build: Beautify cmake files

Add .cmake-format.py
Add cmake-format docs to CONTRIBUTING.md
Minor edits to existing files

Due to a bug in cmake-format, use this script to reformat:

for i in CMakeLists.txt external/CMakeLists.txt loader/CMakeLists.txt tests/CMakeLists.txt tests/layers/CMakeLists.txt
do
    sed --in-place='' 's/^  *#/#/' $i
    cmake-format --in-place $i
done

Change-Id: I8e59c77ae184c88485a97a015ac81d4f71ac4897

build: Insert copyright in cmake files

cmake: Update Kitware copyright

Get latest Kitware copyright notice from
https://gitlab.kitware.com/cmake/cmake/blob/master/Copyright.txt @ d45d5e2

repo: Update json files from 1.1.77 to 1.1.78

build: Improve googletest handling

Behavior is largely unchanged except that specifying BUILD_TESTS=ON
with googletest not present no longer quietly skips building the tests.

- Make inclusion of external directory unconditional.  We may someday
  put something there is not related to testing.
- Make default for BUILD_TEST dependent on googletest presence.
- Remove if() around entire contents of tests CMake file.
- Add CMake messaging to clarify googletest activity.
- Throw a CMake error if googletest not present and BUILD_TESTS=ON

header: Update MacOS json files for 1.1.77 header

Change-Id: I7f04938632f42727b8b1edc15be8981c1f239649

loader: Change a loader warning to an info message

The loader implemented a warning when it was unable to get layers from
the windows registry. The warning would be present even when there was
intentionally no layers available. This change makes it an info
message.

Change-Id: Ic2325daf3a402817b52fae3eb63dd854a57beb99

doc: Update BUILD.md [skip ci]

cmake: Fixup FOLDER properties

header: Update json files for 1.1.77 header

ci: Clean up AppVeyor config file

- Get rid of TOP_DIR - it wasn't set to anything
- Remove unmeeded submodule update
- Build 2 jobs instead of 4
- Use -A CMake argument instead of creating a generator string
  based on the worker image and architecture.  Instead, set the
  architecture (platform Win32 or x64) with -A and take whatever
  Visual Studio version is provided by the worker image.  The
  worker image is specified by the "os" config property.

cmake: Move loader copy to test dir

- Move the CMake code to copy the loader DLL from the loader target
  definition to the test target definition.  The test should be
  taking responsibility for copying this DLL and it avoids doing the
  copy if BUILD_TESTS is on and gtest is not present.
- Use better CMake code to perform the copy.

loader: Remove function pointer casts

cmake: Prevent redefinition of uninstall target

Allows for this repo to be included as a submodule of another
repo that defines its own uninstall target.  The uninstall target
is best defined at the top-level repo.

loader: Fix up missing 1.1 function exports

repo: Make tests not try to build without gtest

Change-Id: Idf3ed7bca713cb7ec1964513ac072344feaf17e6

repo: Modify gtest build logic

Change the logic regarding building the loader tests and Google Test
to match that of Glslang and Vulkan-ValidationLayers

Change-Id: I9f5a9b5385369514e3bec979c8de75ae432d00a6

loader: Add support for get display properties2

Change-Id: I14260b231dd097c85a99ab99c62876b868918869

docs: Remove references to old repo

Remove references to the old repo. Also, remove ninja build steps
since they are no longer valid.

Change-Id: I4a963ec7dc49af2d7d0d186079f0aee78d110d70

make vulkan headers searchable with CMAKE_PREFIX_PATH

docs: Document new google test mechanism

Change-Id: Ib51d38610ba554082391495878bf0113a2885bc0

repo: Make CI clone googletest

Change-Id: Ic942992d4fbc9701ba14e5973faed85b5b2e8c22

repo: Remove google test submodule

Change-Id: I25734926cdf267e565e0ab5878184a0955e435ee

docs: Update build instructions for Vulkan-Headers

Change-Id: I2897e25b3e6db340afa5a35d006ba88c0b3970b0

repo: Make CI clone and install Vulkan-Headers

Change-Id: Ib1a3c82f829377814105d54f552ac3ef00f00681

repo: Remove Vulkan-Headers submodule

Change-Id: I84ec2f2c07482685a275ca2d1804359f09321bad

build: Add FindVulkanHeaders.cmake module

This is useful for locating the Vulkan headers and registry
in a Vulkan-Headers repository.

[skip ci]

loader: Use new VkBase*Structure objects

The 1.1.75 Vulkan spec added two new structures, which accomplish the
same thing is the loader's VkStructureHeader. This removes the loader
struct and uses the ones from the spec.

Change-Id: I0b924b7bb45476645712f600bb6966448f67d8c7

build: Remove Vulkan headers from install target

These are now installed by the Vulkan-Header repo.

docs: Add notes for Win install target

[skip ci]

build: Activate install target for Windows.

repo: Update headers to 1.1.75

Change-Id: I6ab1ec0bcbd3904b834d18637a437c700cf9e912

scripts: Remove scripts that are in Vulkan-Headers

Change-Id: I741c441b5a7b88c496882906e59f5f50c8f289a0

docs: Add a short readme to tests directory

tests: Add user env file for Visual Studio

Add a .user file so that Visual Studio will properly set VK_LAYER_PATH
when running through the IDE.

tests: Make layer tests use layers from this repo

There are a couple of tests in the loader tests that test behavior
when enabling layers. This changes them to use layers that are present
in this repo, instead of validation layers.

docs: Document CMake build options

Change-Id: Ic74794d7de1b7f652aa75a262ebf808cb8b6a4db

repo: Allow BUILD_LOADER=OFF on Windows

If the repo is built without the loader, cmake will now look for an
SDK installation on Windows. On Linux, it will look in the standard
system paths for a loader.

repo: Remove API_NAME from cmake files

There used to be functionality to specify a name other than "Vulkan"
for the files, which would allow side by side installations. This
has proven to cause more trouble than it solves, so I'm removing it.

repo: Stop building dynamic and static loaders

Add a cmake variable to select weather you want a dynamic or a static
loader, rather than building both every time. This avoids the
redundancy of building everything twice.

scripts: Use vk.xml in Vulkan-Headers for codegen

Change-Id: I071f5125dab74b6f646d7ddd1c996ae4c1edb672

scripts: Remove unused scripts

There were numerous scripts in this repo that weren't used, but were
inherited from the old Vulkan-LoaderAndValidationLayers repo. Most
of them were validation layer codegen, but there were also some
utilities for dealing with glslang and spriv-tools.

repo: Add MacOS config to Travis CI

docs: Update MacOS build section

Updated the MacOS section of
BUILD.md to reflect the repo split changes.

Change-Id: I27a2dc2e36866cdf58ed111758432bff6ecad2ad

cmake: Update header lookup location

Updated the CMakeList.txt file to look for
header files in the Vulkan-Headers submodule.

Change-Id: I633e224d7b4c94e5db36e3ec67d8aeb078670d8a

cmake: Add warning if submodules not initialized

docs: Fix broken link in readme

docs: Update readme for recent repo split

repo: Cleanup .gitignore

repo: Change project name to Vulkan-Loader

repo: Update AppVeyor config

repo: Update Travis-CI config

repo: Update documentation files

repo: Reenable Linux install target

repo: Minor updates for repo split

Add cmake code to copy gtest libs to test binary dir for Windows.

repo: Fixup test code and scripts

repo: Remove dependencies on layers dir

repo: Make loader tests work on Linux

repo: Add googletest submodule

repo: Make loader build work on Linux

repo: Update Vulkan-Headers submodule pointer

repo: Update Vulkan-Headers submodule pointer

repo: Update submodules

Add Vulkan-Headers
Remove googletest

icd:Add GPDP2 support for descriptor indexing

Set some default mock icd values for vkGetPhysicalDeviceProperties2
query of VkPhysicalDeviceDescriptorIndexingPropertiesEXT props.

icd:Add support for extended features queries

Update mock ICD generator so that if we have an extension that's been
promoted to core, the core version will call any existing custom
implementation of the extension function.

Update Features query to detect the two existing extension structs
(VkPhysicalDeviceDescriptorIndexingFeaturesEXT and
VkPhysicalDeviceBlendOperationAdvancedFeaturesEXT) and enable all of
those features when those queries are present.

tests: Update tests to remove duplicate slashes

Change-Id: Ie09bfcd61c57024f4c7a0ff884001989fec2a565

loader: Fix gcc 8.1 cast warning

Change-Id: Id8f0301c4f208d185abe2cc3df71021cb4aa29b1

loader: Fix unhandled out of memory errors

Change-Id: I3b672a9d54e1509b550407f1b4ffae58b48beb85

loader: Remove duplicated delimiting chars in paths

Before loading a manifest file, its full path is strcmp() with already
loaded ones in order not to load the same file twice. While being simple
and efficent, this mechanism is not able to detect subtle duplicates
like these:

/usr/share/vulkan/icd.d/intel_icd.x86_64.json
/usr/share//vulkan/icd.d/intel_icd.x86_64.json

This patch ensure that searched paths do not contains such duplicated
directory delimiting characters in order to avoid this kind of problem.

Fixes #2331
Fixes #2629

layers: Track display and display mode objects

Add VkDisplayKHR and VkDisplayModeKHR tracking to
VK_LAYER_LUNARG_object_tracker.

Change-Id: I63004cfe734793593bea12700ea9cd1bdefeab0d

layers: Fix down-chain null funcptr crashes

Modified dispatch table helper to replace null device extension
function pointers with an appropriate stub function.

Change-Id: If5e22333f084d3f41c0eefdf3e82c9060b872e8c

layers: Add missing promoted device extensions

Add three missing device extensions to promoted device extensions list
in vk_extension_helper codegen.

    VK_KHR_relaxed_block_layout
    VK_KHR_shader_draw_parameters
    VK_KHR_storage_buffer_storage_class

Change-Id: I392155d318ca580e576436ef46695218e5eb446e

layers: Sort promoted extensions list

For ease of maintenance, sort the promoted extension lists.

Change-Id: Ib1d690d55c6b9c3a156c9682fbe96bbebed17c14

layers: Add dependency info to vk_extension_helper

Add additional metadata to *Extensions structs to enable extension
dependency validation at CreateInstance CreateDevice time.

Refactor the code generator for the extension helper prior to
refactoring the generated code to address an incomplete.

Change-Id: I79ae680d9c41f1f153ee8108d4deac8ec8c5a886

layers: Refactor codegen for vk_extension_helper

Refactor vk_extension helper prior to adding extension dependency
information to code gen. Refactor to simplify feature addition.

Change-Id: I7b2902dd1a4070c83810f57c0836b50cb6ecfb86

loader: Eliminate warning for uninitialized value.

Added default value to severity to kill a Linux warning.

Change-Id: Iaf3e62a0fa01d74874aaec4951f9a2aa38479ca0

layers: Sort extension order for extension helper

Sorted the extensions by name s.t. the output of the vk_extension_helper
generator would be consistent run-to-run.

Change-Id: Ief3d822b91260e469f7f2d563076d8a79f1a1bf0

header: Update to version 1.1.74 of Vulkan hdr

- update layer json files
- update vulkan_core.h
- update vk.xml
- update vulkan.hpp
- update validusage.json
- update vk_validation_error_messages.h
- update vk_validation_error_database.txt
- maintain local changes to reg.py

Change-Id: If0c52d13f78ddfcc7c9d316386c0d20399e97668

loader: Fix a 1.0/1.1 versioning bug

Fix a bug where when mixing a 1.0 and a 1.1 driver, if the loader
stores the 1.0 driver first, the apiVersion field in vkCreateInstance
will be rewritten to 1.0 for both drivers. Now it will only be
changed for the 1.0 driver, as intended.

Change-Id: I9e7c5e0b0b7c0eb660082319dcddd9e8ef8adc61

scripts: Add spec.py check for ext changes

When the surrounding asciidoctor conditionals of a VUID change it may
impact validation even if the VUID text does not change.  Detect this in
spec.py and flag for manual intervention.

Change-Id: Idf802955ba9bbfe911edd4114fb28bdddf5fcc9b

loader: Clarify loader warning

Clarify several loader warnings about layers so that they include
the name of the layer that is causing an issue.

Change-Id: Ie48d73f6cc8466c3114597831fef451b3ab32bf7

macos: Specify deployment target version

This allows using the build on macOS 10.12 even if built on 10.13.

Change-Id: I21939163551bb9a0d0088513a23dc5d2cf7f5206

scripts:Account for extnumber key to fix warnings

With the addition of the 'extnumber' key update comparison to skip over
the extnumber as well as extname in order to avoid a bunch of
irrelevant warnings.

loader:Fix build warnings

Add "fall through" comments on intentional fall through switch cases to
avoid compiler warnings.

loader: Bump loader copyright to 2018

Change-Id: I59b80b7f4bff37ac15fd30acc3f4600164afee90

header: Update to version 1.1.73 of the Vulkan hdr

- update json files
- update vk.xml
- update vulkan_core.h
- update validusage.json
- update vk_validation_error_messages.h
- update vk_validation_error_database.txt
- update vulkan.hpp
- removed checks for the following deprecated VUIDs related to
  ImageCreateInfo and FormatProperties:
  - VALIDATION_ERROR_09e007ae
  - VALIDATION_ERROR_09e007b4
  - VALIDATION_ERROR_09e007ac
  - VALIDATION_ERROR_09e007a6
  - VALIDATION_ERROR_09e007b2
  - VALIDATION_ERROR_09e007a4
  - VALIDATION_ERROR_09e007aa
  - VALIDATION_ERROR_09e007a8
  - VALIDATION_ERROR_09e007a2
  - VALIDATION_ERROR_09e007b0
- modified CreateImageFormatSupportErrors test to remove killed checks
- modified ImageLayerUnsupportedFormat test to remove killed check

Change-Id: I8f1577747d87fdeb6f68b9db0340443a83c44630

icd:Add new physical device query funcs to ICD

Add mock ICD support for vkGetPhysicalDeviceSurfacePresentModesKHR,
vkGetPhysicalDeviceSurfaceFormats[2]KHR,
vkGetPhysicalDeviceSurfaceSupportKHR,
vkGetPhysicalDeviceSurfaceCapabilities[2]KHR,
vkGetSwapchainImagesKHR and
vkAcquireNextImagesKHR.

travis: Avoid empty Travis CI job

Change-Id: Ifb917ab3cf72f0c89f3c7bc9b647156bf1259a50

travis: Set up to test commit messages

Change-Id: I64adcdb59329eed77cd7c0dd4ff89362f73f72f4

scripts: Add commit message format checker

Change-Id: I94efe0326e3f68e19af605a1ff381da59d2d7d1d

scripts: exclude vulkan headers from clang-format

Travis CI shows linux build failure on every header update because
upstream Vulkan-Docs files are not formatted with the clang-format style
of this repo.  This change excludes the vulkan headers directory from
the CI clang-format check.

Change-Id: I6d1bee486d06b690f65aa2c3098de14f3c5931c9

header: Update to version 1.1.72 of the Vulkan hdr

- updated layer json files
- updated vulkan_core.h
- updated validusage.json
- updated vk_validation_error_database.txt
- updated vk_validation_error_messages.h
- updated vuid_mapping.py
- updated vk.xml
- updated vulkan.hpp
- moved vuid 1ae00652 to 1ae00f06

Change-Id: Id6ba342841cca04c0669b7feb39d82e2b8052443

scripts: Output vuid-string->message map

Change-Id: Id4123053da62fb8d2b4d32393d11e4aa6c030f92

layers: Add checks for CreateDescriptorUpdateTempl

Added four validation checks for vkCreateDescriptorUpdateTemplate(KHR)
and removed a false positive.

VALIDATION_ERROR_052002bc
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00350
VALIDATION_ERROR_052002be
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00351
VALIDATION_ERROR_052002c0
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00352
VALIDATION_ERROR_052002c2
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00353

Updated autogen of object tracker s.t. that logic for noautovalidation
handles matches parameter validation, and spec generation, eliminating a
source of false positives.

Change-Id: Ib831eabd45a5083168355bca6b0e2beda3a76222

docs:Fix my name spelling in README

macos: Update vulkan framework version to 1.1

layers: Avoid reading pointer to array when count is zero

Take for example VkSubmitInfo. It has, amongst others, two fields:
waitSemaphoreCount and pWaitSemaphores. The specification states that
if waitSemaphoreCount is zero, the application may leave pWaitSemaphores
uninitialized (in essence, that field is ignored).

The layers read that value anyway, triggering uninitialized read errors
by memory sanitization tools.

Fixes uninitialized read in QueueSubmitSemaphoresAndLayoutTracking.