build: fix ninja on windows

Use target_compile_options to avoid setting invalid compile options on
asm targets.

Change-Id: I5d95cfd77a2d515f41c7b8c2c9193f3aff906be7

Revert "build: Fix ninja on windows build"

This reverts commit 41e6a818ff70370cf4d190d9141ee3caecf01806.

Change-Id: I48f98533126354a8c310aca7dfb321c560873955

scripts: Extend scope of LaTeX handling

The 1.1.90 header introduced the use of "\textrm" in `vk.xml`, requiring
changes to `scripts/helper_file_generator.py`

Change-Id: I0ee5012611d95cfdd1715a4e870a3b2b4e7556ee

loader: Fix multiple SubmitDebugUtils callbacks

If an ICD also supported the VK_EXT_debug_utils extension, then
any call to vkSubmitDebutUtilsMessageEXT would get duplicated.

Change-Id: Ica3224ee598a99a925ec9343b6618d4d8ba190d0

loader: Fix vkSubmitDebugUtilsMessageEXT

If the object count was 0, then no message was reported by the loader.

Change-Id: Ia809d193fb117e85742af0db9856f896be0b2ea3

repo: Remove MIR specific code

Note that references in scripts/common_codegen.py and
scripts/loader_extension_generator.py will need to be
removed later

Change-Id: I7b17c80f7a06a339d7df0c199ff556212a7c6534

repo: Fix Travis CI job for MacOS

A change in the MacOS images caused a conflict when upgrading python3.
This change removes oclint first, since that package is the source of
the conflict

Change-Id: I990a7b0eb1f91f51fcc2ad2a0db120af49c7a754

loader: Handle vkGetDeviceQueue2 correctly

Change-Id: I15f7677d5c6d8b4f0ff62bb7d371d173d58ae5f4

repo: Update known good to 1.1.89

Change-Id: I31a45bd5e7ce076afab6e035a3fc2803be00b90f

repo: Disable gcc 8 stringop warnings

GCC 8 introduced some new "stringop" wanrings. The loader deliberately
does things that trigger these warnings. As such, they should be
disabled

Change-Id: I08f640fae2930a06cbdd48095f11febfc9e866ed

loader: Resolve incorrect error in create instance

Change-Id: Ib3f99bb150ddf9aaa36cbe08919d737f2636baf6

loader: Override layer support and settings

Add support for an override layer in the loader. This allows layers
to be set by an outside application.

Revamp the logic to detect manifest files in the various paths that
we allow them to be placed. This code significantly rewrites
loader_get_manifest_files.

Change-Id: I8abf558864b66eb71ee026ca559b0126cf2fa4e9

build: Fix ninja on windows build

Change-Id: Ie15d8a47636d228732950fb5b3462a2819d5b2af

repo: Bump known good to 1.1.87

Change-Id: If238e0fe6462cf90952d4c0132b49b46bbe96a4e

Create CODE_OF_CONDUCT.md

tests: Use specific version of gtest for CI

scripts: Add fuchsia to platform dictionary

Change-Id: I229825efa86109e3e24de722fac52179963d3e70

Revert LICENSE to Apache 2.0

Update LICENSE.txt

Change Apache 2.0 to Khronos Apache 2.0 CLA

build: Sync update_deps.py

Syncing changes made to update_deps.py
in Vulkan-ValidationLayers.

Change-Id: If617596654b982023ee883fe131390c760c53532

repo: Update known good to 1.1.85 headers

Change-Id: If1383facdfcd39b9ac07164574bb6264dcc82100

build: CMakeLists.txt cleanup, part 1

This is a first pass reorganization of CMake files in this repo.
It consists primarily of dead code/variable removal, simplification,
and reformatting by latest cmake-format.

bump to cmake_minimum_required(3.4) for ccache

simplify excessive use of generator expressions

clarify why CMAKE_OSX_DEPLOYMENT_TARGET is pre-project().

clarify setting Windows install prefix if needed

remove unused DisplayServer variable

remove obsolete argument to endif()

remove unneeded ccache property RULE_LAUNCH_LINK

ccache doesn't affect linking, so the property is meaningless.

remove unneeded CMAKE_VERBOSE_MAKEFILE hardcode.

remove extra CMAKE_MODULE_PATH

remove extra cmake_minimum_required

change PYTHON_CMD to PYTHON_EXECUTABLE

change CMAKE_SYSTEM_NAME to UNIX/APPLE/WIN32
    Use consistent platform identification.
    Note that UNIX evaluates true for OSX, so whenever the code intends
    "Linux only", we use (UNIX AND NOT APPLE).

remove unneeded TARGET_NAMES variable

remove unneeded VK_LAYER_RPATH variable

change 'ln -sf' to 'cmake -E create_symlink'

reorder dependencies for clarity

rearrange order of macro arguments

rename run_vk_xml_generate to GenerateFromVkXml
rename add_vk_layer to AddVkLayer
    Borrowing from the Google style guide, use camel-case
    to more easily identify locally-defined macros/functions.

remove unneeded WRAP_SRCS variable

remove unneeded TEST_SRCS variable

move check of VulkanRegistry_FOUND

move SCRIPTS_DIR closer to point of use

move include(FindPkgConfig) earlier

move CMAKE_MACOSX_RPATH earlier

move option() eariler

move GenerateFromVkXml invocations earlier

Move invocations to be closer to the definition of the macro.
Minor edits to comments.

move API_NAME earlier

remove unused BUILDTGT_DIR variable

move add_executable() earlier

combine if(WIN32) trees

.cmake-format.py 0.4.1

reformat using cmake-format 0.4.1

Change-Id: Ia251e932dbc9a8f2655201bb12248dea8c940c30

repo: Update known-good to 1.1.84

Change-Id: I73b1337dad35d7ecf28c6acfc61c37832cc4497b

loader: Fix unused variable compiler warning

Change-Id: I331c532e55034f004b55e2738214c0301a08375e

loader: Fix overzealous duplicate manifest warning

This downgrades the warning when a manifest file is given multiple
times to a debug message (previously it was a warning). This is
because any Windows system that has multiple GPUs that use the same
driver will encounter duplicates as correct behavior.

Change-Id: Id64f4c34d58c4b806164b3e7cd7a07dab36ab88f

repo: Bump known good to 1.1.83

Change-Id: Ie85da5d3a20b8bad4f4c4a5f7429c0fb7463a7b2

loader: Add missing entry point to gpa helper

Resolves an issue where vkGetBufferMemoryRequirements2 was not present
in gpa helper, which caused problems when calling GetProcAddr

Change-Id: I40933ec5adba54b09b88200d46b307150bb52d9d

build: Correct Windows architecture options

build: Unify `update_deps.py` between repos

docs: Fix path typo in interface doc

loader: Remove layer negotiation from device chain

Previously, when the loader created a device chain it would attempt
to negotiate the layer interface version. Now, it just saves the
version when creating the instance chain.

Change-Id: I8a48914474e5600e023d837b01198c23da9c8464

loader: Save gipa and gpda ptrs in instance chain

Change-Id: Iaf7d0084cbeb41e7542787a5b013a918a2431b82

loader: Fix VK_EXT_debug_utils crash

Fix a crash where the debug utils extension would crash if no layers
were present that implemented a given entry point.

Change-Id: Ied1da23dfdc952893a5aaad4ae01f69bd6010523

repo: Bump known good to 1.1.82 headers

Change-Id: I7ac0d6f31bd1c09d37a6436c022769bd134fea58

Fix build with CFLAGS="-std=c11 -D_GNU_SOURCE"

repo: Bump known good to 1.1.81 headers

Change-Id: I1da4208c4a57914e3795eef5907128d350029b1a

docs: Remove vk_layer_dispatch_table.h from BUILD.md

Change-Id: I34fded0bcc970fd09640993ed4dfd67fbc29dcbe

docs: Update build doc with known-good info

Change-Id: I89fbc453e0ca2b83a3ad8ea3d0756655fe1e651f

repo: Add handling for cron CI jobs

Change-Id: I4376f1f92863115ca2a1ec818f888c61b1c8e8e5

repo: Use update_deps.py in ci jobs

Change-Id: Id7d72950c3b1459815fd5798daf59302463e1e64

repo: Add update_deps.py script

This script is copied from the Vulkan-ValidationLayers repo and allows
us to track commits from external repos that are known to be good.

Change-Id: Ibd00e9824e326b7af00eb75b6c3e3029542b0464

repo: Use version from FindVulkanHeaders in build

Change-Id: I565d2e49d965588404ad9ad4ce21f59b3a5abc00

loader: Fix detection of patch

Karl pointed out that the patch detection wasn't that future proof.
Added his coding suggestions.

Change-Id: Ieedd53776da1835a38ed1815a20775988df94900

loader: Update FindVulkanHeaders CMake

Update the FindVulkaHeaders.cmake file to determine the Vulkan
header latest Major/Minor/Patch version.

Change-Id: Ia2a2df56d6ca6687ccb904f6c286c82553586695

repo: Rework test layers to use cmake generators

Change-Id: I6a09049619b7155e9f5331f0267d39f1edd42d38

loader: GH54: Fix setfault with VK_EXT_debug_utils

The loader was not setting the device flag indicating that the
VK_EXT_debug_utils extension had been set in the instance unless
one or more device extensions had been added to the list.

This fixes the bug caught by nikitablack on StackOverflow.

Change-Id: I23c24ee059ab5a66b9dbfdb9fa9ddab009fb2118

cmake: Clean up macOS install target

Stop hardcoding CMAKE_INSTALL_PREFIX to CMAKE_BINARY_DIR

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