add spec file

loader: Fix unwrapped loader instance being used

Change-Id: I64cbfa42e2e2ac83a6977acd59e6fff6bc6f73ef

loader: Downgrade layer json warning to info

The loader used to warn when using the vkGet*ProcAddr fields
in a layer json file from 1.1.0 on. However, it was desirable for
applications to still use this field to allow support from older
loaders. As such, this is no longer a warning.

Change-Id: I401febffe0a15081772dd4a791f5fffbd98d6ac9

repo: Bump known good to 1.1.92

Change-Id: I850fe7799cd13405c3505c164758292a07e2c6c0

loader: Fix override expiration functionality

Change-Id: Ib2415cb1a9218930dc68827abdd1587bd0672342

scripts: Fix windows paths in helper.cmake

Normalize (all slashes going the same direction) and escape ("\" to "\\")
paths when writing helper.cmake.  CMake was previously interpreting the
single '\' characters and whatever followed as invalid escape sequences.
Linux and macOS are unaffected.

repo: Fix MacOS travis build

Travis changed its images to no longer have oclint installed by
default. This means we need to stop manually removing it.

Change-Id: I75790e7fb25815f509cbeb5ee6d5e1902cbc8a98

loader: Fix double free when enabling override

Change-Id: I92f0012528bf1fb4cd9acf3053a4850665b225e8

loader: Fix mac bundle logic

Change-Id: Id8eade275293d4812ecf3fb6ee8e6e1f4ed51885

scripts: Remove Mir exceptions from codegen

Change-Id: Ief43a6294c4cd1a9397d6c6630bb0319c62f387c

repo: Bump known good to 1.1.91

Change-Id: Icae25bc3fbd2408c19efa0a647154380ffc4b17b

loader: Fix VK_EXT_debug_utils termination

Some of the commands aren't needed in the loader, but this would
cause command chains to crash when they hit the NULL.
Also, the ICDs expect the commands to come through vkGetInstantProcAddr
not vkGetDeviceProcAddr.  So, we need to redirect them when generating
the ICD dispatch table because the device commands in the instance
extension exist in the device dispatch table but need to call into
the ICD using the instance queries.

Change-Id: I24cd0ea4f3e5058b1137881452df75bf1ff7487a

loader: Fix python3 warning

Python 3.8 will remove time.clock() and the warnings suggest that
time.process_time() or time.perf_counter() should be used instead.
So, move the script to using time.process_time().

Change-Id: I982da1560adec4aab49f42936334e520456647c1

scripts: Compare to `None` using `is` operator

This is a trivial change that replaces `==` operator with `is` operator, following PEP 8 guideline:

> Comparisons to singletons like None should always be done with is or is not, never the equality operators.

https://legacy.python.org/dev/peps/pep-0008/#programming-recommendations

Change-Id: I4f9f6c921e4158365d4e41965bfcd43b7a3c07e0

repo: Bump known good to 1.1.90

Change-Id: If494bd4abe7817d358eaa3966dd82e1c582cbd85

loader: Fix unknown type

Set var to VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT instead of
VK_OBJECT_TYPE_UNKNOWN.

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