cube: Workaround macOS cube issue

Latest version of MoltenVK behaves in an apparently non-conformant way
by not allowing some linear images to be mapped to memory with
VK_MEMORY_PROPERTY_HOST_COHERENT_BIT.  This change sidesteps the issue
by forcing the use of staging buffer for loading textures.

winrt: Correct `VulkanRT-License.txt` typo

Change-Id: I26c1fbdddc8d6f6fc344164de2cc7cfa74c08919

doc: Update docs with codegen instructions

CI: Add codegen consistency check

build: CMake/GN changes for pre-generated source

icd: Add generated source to repo

scripts: Port codegen script from VVL

scripts: Fix mock_icd_generator syntax warnings

build: Update glslang commit ID in known-good

Change-Id: I7d1031909f91dc14471d41e33bf50bafffaa7c58

scripts: Add required depth/stencil resolve props

Spec mandates that at least VK_RESOLVE_MODE_SAMPLE_ZERO_BIT_KHR be
supported for supportedDepthResolveModes and
supportedStencilResolveModes

build: Update known-good files for 1.1.114 header

Change-Id: I8dcbd49e7ccc07f0c891d9c303e1dc0d8445f9e8

build: Update cmake_minimum_required(VERSION 3.10.2)

Change-Id: I193d23da109a81e0b385a3d9fc54a6d266b6d026

vulkaninfo: Add descriptor indexing feats/props

Change-Id: I7863fb44fbc1e2c7f7833580799db6b8fb229afe

docs: Improve instructions for using update_deps.
[ci skip]

Same change as KhronosGroup/Vulkan-ValidationLayers#955

Change-Id: Ifdc30a41f4d7f1e4ad9bc887df6f43f5caaa69f9

build: Fix LunarXchange Issue 862

Modified MoltenVK build arguments for MacOS from
`MVK_LOGGING_ENABLED=0` to `MVK_CONFIG_LOG_LEVEL=1`.

This allows users to set the logging information they
want when an application is run using MoltenVK.

By default, only errors are logged.

Change-Id: I471fa28e68b5dc68b10ae17858475981158b2a86

build: Update known-good files for 1.1.113 header

Change-Id: I3151722ccb8f3c5d918db8ffd701661a33dcf3b6

cube: Fix DbgMsg() use of varargs

vprintf (not printf) must be used with va_list.

Also moved flush() outside the va_list's scope.

Change-Id: Ib82b9f7e7ec1f954619c3eadb8dcaeb08abf113a

vulkaninfo: Add more descriptive error message

Issue #207 mentions that one of the main problems users
encounter when troubleshooting is a failure to create a
Vulkan instance, and suggests that a statement be added
to this error message that describes the most common
causes of this problem.

This commit adds this additional statement to the current
error message.

Change-Id: I2fccf00a8d9895a602fad40913f2380fc1c0a862

build: Update known-good files for 1.1.112 header

Change-Id: Ibf447e4f562d49bda5ab60b1bdf4ebf71678a6ae

vulkaninfo: Fix issue 195

It was suggested that `maxComputeSharedMemorySize`
from VkPhysicalDeviceLimits be displayed as a decimal
instead of a hex number.

Change-Id: I4b5cb61d56adea4e4d8d33619fabc37e359b8e4a

vulkaninfo: Fix issue 202

Vulkaninfo was replacing function `snprintf`
with function `_snprintf` on windows.

This seems to be for backwards compatiblity
with VS2013.

`_snprintf` is considered deprecated in VS2015
as `snprintf` is properly implemented.

Since we are dropping compatibility with VS2013 it
seems like this replacement should be removed.

Due to AppVeyor failing to build when this pre-processor
command is removed entirely, the command has been modified
to replace `snprintf` with `_snprintf_s`.

This command should hopefully be removed entirely in a future
commit.

Change-Id: I85f726fcb0a1cfcc902487100d35dc63d2ba00d8

build: Update MoltenVK in known_good.json

Updated MoltenVK version in known_good.json
from 1.0.34v -> 1.0.35v

Change-Id: I3c574d22ab98fcf0e3bd0c718a1a3f5a69287b42

cube: Add subpass dependencies

Shared depth buffer and image layout transition both
need to be accounted for

Change-Id: Id26bc47798c58e4435a5585def3161105b7fffd1

cube: Fix compiler warning

Change-Id: I8a84556cf55b03cc39402067150fc86be3ed1a0a

vulkaninfo: Fix HTML format for multi-GPU computer

Fixes an issue found where the device properties
and extensions would only be printed for the first GPU
on a system when printing in HTML format.

Change-Id: Id6812d7c6b14915afb87fdb740e8918f4609d662

build: Update known-good files glslang commit ID

Change-Id: I5ef3ac82301d42cbdb616da4c995cf6dc53c6708

cube: Add usage message on Windows

Change-Id: Id709544863b50def407d7e18c330914bebc9c03a

cmake: adjust cmake download process to match new design

Also took Karl off the list of e-mail addresses to be notified,
at his request.

cmake: Use CMake 3.10.2 for CI

These changes ensure that the Travis and AppVeyor
builds use a known version of CMake.

build: Update known-good files for 1.1.108 header

Change-Id: Ifc19c0e0fcb36f9a5009a73979eb20069e9db463

vulkaninfo: Run clang-format

Change-Id: I956d5777b657a25edb7f289047f320345af5cc44

vulkaninfo: VK_KHR_surface_protected_capabilities

Added output for the following structures from
extension VK_KHR_surface_protected_capabilities:

        VkSurfaceProtectedCapabilitiesKHR

Change-Id: I5fb89b8118abbdb3876463f2ff5f96c08221781b

vulkaninfo: Add VK_EXT_host_query_reset

Added output for the following structures from
extension VK_EXT_host_query_reset:

        VkPhysicalDeviceHostQueryResetFeaturesEXT

Change-Id: Ibc3dce6261ff5e46926d4f969562d377c74f23f5

vulkaninfo: Add VK_EXT_ycbcr_image_arrays

Added output for the following structures from
extension VK_EXT_ycbcr_image_arrays:

        VkPhysicalDeviceYcbcrImageArraysFeaturesEXT

Change-Id: I0f10ae006e99fff5192729ac32bafae2d9170d0c

update_deps: make --clean-repo switch work in all cases

--clean-repo right now will raise an exception if the repository
directory was not present.  This makes it work in all cases.

scripts: Temporary workaround for python2

Temporarily patch abc module with ABC helper class until upstream
conventions.py script stops using it.

build: Add GN build

Add support for GN, Google's meta-build system for Ninja:
https://gn.googlesource.com/gn/

Pull BUILD.gn and script dependencies from
https://chromium.googlesource.com/angle/angle

Add build_overrides for this project to enable both standalone and
component builds using GN.

Add GN configuration to Travis CI

scripts: Update known good for 107 header update

Updated:
- `build-android/vulkan-headers_revision_android`
- `scripts/known_good.json`

Change-Id: Ibf9d23645407e78805b8b8192a35338452baaf18

Fix memory leak in `vkcube`.

vulkaninfo: Fix crash at Presentable Surface

Checks to make sure surface extensions were enabled
neglected to leave important struct variables outside
the if statements.

This resulted in these variables becoming uninitialized
and causing undefined behaviour.

Change-Id: Ice27314aa1d6520fc83c14726e2075a8574ba883

vulkaninfo: Check support for surface extensions before using them

Not all ICDs support all the surface extensions that vulkaninfo
enumerates.

For example, on Linux with an ICD that only support VK_KHR_xcb_surface,
vulkaninfo crashes:

Presentable Surfaces:
=====================
ERROR: VK_KHR_xlib_surface extension not enabled.  vkCreateXlibSurfaceKHR not executed!

vulkaninfo/vulkaninfo.c:1427: failed with VK_ERROR_EXTENSION_NOT_PRESENT

Update CMakeLists.txt

Adds Control Flow Guard compiler flag to vulkaninfo

Control Flow Guard (CG) protects against some forms of memory corruption.
See https://docs.microsoft.com/en-us/windows/desktop/secbp/control-flow-guard for details.

This compiler flag enables CFG on vulkaninfo. (CFG is already enabled on the Cube project.)

Enabling CFG is a low-risk, "set and forget" low hanging fruit for security hardening.

known_good: Update MoltenVK known_good version ID

Updated the MoltenVK version in known_good.json to
"v1.0.34"

Change-Id: I81bfec2e80f724815c4e9576386d00a96b784a48

test: Update cloud CI to use known_good.json

Updates to other repos can cause errors to propagate
from changes that have nothing to do with PRs on
Vulkan-Tools.

This commit changes cloud CI systems to build with
known_good.json in Vulkan-Tools so that issues
affecting other repos do not prevent PRs in Vulkan-Tools
from getting merged.

Change-Id: Iad4907c0bf07322c9b02477e09fed9fd5b72b631

cube: fix scoping of some variables.

use_invalid: Using "present.pNext", which points to an out-of-scope variable "regions".

In theory these local vars are out of scope when they are used later.

Pointed out by coverity

cube: remove unused validation_layer_count.

This is assigned but never used.

Pointed out by coverity.

vulkaninfo: use correct type in malloc sizing.

warning: Result of 'malloc' is converted to a pointer of type 'VkPresentModeKHR', which is incompatible with sizeof operand type 'VkPresentInfoKHR'

Pointed out by coverity.

vulkaninfo: handle NULL layer_name.

Earlier we check for a NULL layer name but not here.

Vulkan-Tools-sdk-1.1.101.0/vulkaninfo/vulkaninfo.c:4403: var_deref_model: Passing null pointer "layer_name" to "strcmp", which dereferences it.

Pointed out by coverity.

vulkaninfo: handle html output file open failure.

Vulkan-Tools-sdk-1.1.101.0/vulkaninfo/vulkaninfo.c:5008: dereference: Dereferencing a pointer that might be "NULL" "out" when calling "PrintHtmlHeader".

Pointed out by coverity.

cube.cpp: fix uninit non-static member.

This was pointed out by coverity.

vulkaninfo: Fix HTML output

Fixed the HTML output for the
VkPhysicalDeviceDepthStencilResolvePropertiesKHR
structure.

HTML output was being sent to the terminal instead of
the HTML file.

Change-Id: I26daa670aa732c0c758dbbc48c1fcdc02e2da38b

scripts: Update known good for 106 header update

Changes:
- Integrate upstream script changes: We have to plumb-through the new
conventions object to continue using the makeCParamDecl utility function
- Add GGP to available platforms
- Define `VULKAN_HPP_TYPESAFE_CONVERSION` in `cube.cpp` to avoid build
errors concerning explicit/implicit casting in `vulkan.hpp`
- Update known-good files

Updated:
- `build-android/vulkan-headers_revision_android`
- `cube/cube.cpp`
- `scripts/common_codegen.py`
- `scripts/known_good.json`
- `scripts/kvt_genvk.py`
- `scripts/mock_icd_generator.py`
- `scripts/vulkan_tools_helper_file_generator.py`

Change-Id: Ie9deb9f088d666195bcb987e30974f274d33fc85

cube: Update cube apps to use Khronos val layer

format: Run clang-format

Run clang-format to reformat appropriate changes.

Change-Id: I0e8bd083806f5da81d889d306e1802621d4bf7b3

vulkaninfo: Add VK_KHR_depth_stencil_resolve

Added output for the following structure from
extension VK_KHR_depth_stencil_resolve:

        VkPhysicalDeviceDepthStencilResolvePropertiesKHR

Change-Id: Ib997c0c8d8f35be14cd611e6b6473accc28a2985

vulkaninfo: Add VK_EXT_buffer_device_address

Added output for the following structure in the
VK_EXT_buffer_device_address extension:

        VkPhysicalDeviceBufferAddressFeaturesEXT

Change-Id: I04f18b62ccc485ea98522bafc01d9d3f15d19ac2

vulkaninfo: Add VK_EXT_memory_budget output

Added output for the following structures provided
by VK_EXT_memory_budget:

        VkPhysicalDeviceMemoryBudgetPropertiesEXT

Change-Id: I442675019ca7b3fbdbed7cd52ac1843f3de341f2

vulkaninfo: Add VK_EXT_memory_priority Output

Added out put for the following structure from extension
VK_EXT_memory_priority:

        VkPhysicalDeviceMemoryPriorityFeaturesEXT

Change-Id: I5c32039e878dff7b4323d143830ae7515733f74b

vulkaninfo: Run clang-format

Change-Id: I9fc6732e87c71061c0cabb6b46022b525cf5c84b

vulkaninfo: Fix HTML output warnings and errors

Updated vulkaninfo HTML output to fix errors and
warnings output by https://validator.w3.org.

Change-Id: Ic84cc71467e69a33d2c816e8e79b918006c3194e

docs: Update GOVERNANCE

Updated GOVERNANCE.md to better reflect the
KhronosGroup/Vulkan-Tools repo.

Change-Id: I74a609324e2e2e24c303d180dde4ba165ed2575a

deps: allow a --generator switch to update_deps.py

This allows you to use any CMake generator available
on your system to configure the dependent repositories,
allowing you to match the same generator being used by
the project build.

update_deps.py:
- Take a parameter to specify a CMake generator, and pass it to CMake
  if present
- Allow the user to reduce the count of parallel make jobs that can
  happen at one time
- remove an unnecessary semicolon

docs: Update CONTRIBUTING.md for CI process

Change-Id: Iaca18022e840df01fd73654630e3f6377b4af2a7

docs: Update BUILD.md for CMake versions

build: Update CMake version

Updated suggested CMake version for platforms in
BUILD.md.

Added CMake version output on Travis and AppVeyor.

Change-Id: I128b17c85a8e4c8a5c1f334b818ea8171d9a7143

vulkaninfo: Fix compiler warnings

vulkaninfo: Report images compatible with memory type

vulkaninfo: Remove invalid json for present_support

docs: Update CONTRIBUTING.md

Updated CONTRIBUTING.md to link to instructions
about validating vulkaninfo JSON output.

vulkaninfo: Make present support 1\0 in json

vulkaninfo: Fix missing struct declaration (warning)

vulkaninfo: Restructure present support output of multi-platform case

vulkaninfo: Convert surface dynamic array to a linked list

vulkaninfo: Prevent surface creation silent failure

vulkaninfo: Fix uninitialized VkDevice pQueuePriorities

vulkaninfo: Fix window\surface lifetime

vulkaninfo: Revert "Fix Issue 136"

This reverts commit 607c301efbb2ee5228f8ddc8eb7500f11dedecbd.

vulkaninfo: create JSON validation process doc

build: Correct Metal platform name

In `common_codegen.py` the Metal platform was defined without the "_EXT"
suffix

Change-Id: Ib36778735c09ae54d0e4a9bb0d7aa2fcec26b985

build: Update known-good for 1.1.102 header

Additionally, updated copyright dates and added Metal to the platform
list

Updated:
- `build-android/vulkan-headers_revision_android`
- `scripts/common_codegen.py`
- `scripts/known_good.json`

Change-Id: I161f2dfaf8fafd25be50e09b8c6e7da13d9ad9b3

vulkaninfo: Fix Issue 136

Added listing for all present extensions and
listing for present support for each extension on
GPU queue families.

Change-Id: I3ad18bb346ff50b6dfa141cebb58155f59fd754c

macos: Change LunarG Icons to Vulkan Logo

Replaced LunarGIcon.icns with VulkanIcon.icns as part
of rebranding initiative.

Change-Id: I582b5d057f9177940c3f959a87a304c74993dec2

vulkaninfo: fix nesting of HTML output

Four extensions, if supported on a device, didn't emit the proper "</details>"
closing tag.  This caused the nesting to be incorrect while viewing the HTML output.

vulkaninfo: Update `vulkaninfo.rc` copyright dates

Change-Id: I9eadccd98083d7e77b74a5bcfe832745cb13701a

Add librt when necessary

Change-Id: Ie4b3fdec4d61a52e82880308bb567f2f22c6c273

build: Update glslang known-good for 101 SDK

Change-Id: Ib28abfe172079ad60ea5bfcbbb922990e302b135

Make copyright internal to the file

build: Update known-good to 1.1.101 header

Change-Id: Ifedc7998fbf520f48748b646307071c07c58e627

docs: Add vulkaninfo.md

Added vulkaninfo.md

Change-Id: Ie190cf31f34bff649b36be977a1334e22a3e89e7

Fix the build error on Android

Currently, the vulkaninfo is not supported on Android.
But someone might use the vulkaninfo with their own makefiles
crossing all the platforms including Android.

This change fixed the compiling error on Android platform.

known-good: Update Vulkan-Tools to 1.1.100 Header

Updated the the Vulkan-Headers tag in known-good.json
to point to the Vulkan 1.1.100 headers.

Change-Id: Ib641abeb75da36504537bfc29d0cb05dc1109612

vkcube: Implement key events in Windows

vulkaninfo: print VK_MEMORY_PROPERTY_PROTECTED_BIT

Adding the VK_MEMORY_PROPERTY_PROTECTED_BIT in the printing.

build: Revert known-good to 1.1.97 header

Reverting known-good files to 1.1.97 due to a spec bug introduced in the
1.1.99 header

Change-Id: Ie9f91658cc9a22021a7c85a0734f325ec1c63c72

vulkaninfo: Reintroduce vkCreateInstance pNext debug report callback

vulkaninfo: Rewrite instance creation

1) localize `VkPhysicalDeviceSurfaceInfo2KHR (and remove from instance creation)
2) Remove debug callback. We are not even using layers...
3) Enable all supported instance extensions. Close #100.
4) Prettify instance command loader
5) other esthetics

build: Update known-good for 1.1.99 header

Updated:
- `build-android/vulkan-headers_revision_android`
- `scripts/known_good.json`
- `scripts/kvt_genvk.py`

Change-Id: Ibd1a61c359a417a90bdde6327536fc8d890f3734

macos: Update known-good.json

Updated the commit ID for MoltenVK in
known-good.json.

Change-Id: I0658c7499021836a4baa7c78f4f16fb3e525629f

vulkaninfo: Remove clang-format off

It doesn't make sense to enforce clang-format rules
and then turn it off for more than half of the file.

Removed clang-format off from vulkaninfo.c and ran
clang-format to clean up the code.

Change-Id: Id9358d8272568314c57dd861a954a2247320b36f

vulkaninfo: Check VkResult of vkGetPhysicalDeviceSurfaceSupportKHR

patch #125