vulkaninfo: Fix padding for integer dot product

The 1.3 update forgot to rename
VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_SHADER_INTEGER_DOT_PRODUCT_FEATURES_KHR to
VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_SHADER_INTEGER_DOT_PRODUCT_FEATURES as this
is the non-aliased name. Without it, the fix wouldn't be applied.

build: Add build target for vkcube-wayland

mockicd: Fix queue destruction for multiple devices

Fix DestroyCommandPool command buffer leak

Fix FreeCommandBuffers leak

vulkaninfo: Update to 1.3

Only changes are making the autogen print all required structs, and
moving from using the EXT version of tool properties to the core version.

build: Update to header 1.3.204

- Update known-good
- Generate source

Add support for building on ChromeOS

cmake/FindWaylandProtocols: fix wayland protocol path generation for cross compile

build: Add dependency on wayland-protocols

build: Update to header 1.2.203

- Update known-good
- Generate source

Fix error messages upon failure to connect to window systems.

Change-Id: I650958a8b2607eb3402eb6531909881d4a2ccc0f

scripts: Fix codegen to support VK_KHR_format_feature_flags2

If the implementation reports that it supports this feature, it should
fill out VkFormatProperties3KHR properly.

ci: Use windows-latest in github actions

build: Update to header 1.2.201

- Update known-good
- Generate source

vkcube/vkcubepp: Add option to force validation errors

During tests, it's hard to determine if the --validation option is
actually working properly.  So add an option (--force_errors) to
intentionally force a few validation error messages to make sure
that validation is working properly.

vulkaninfo: Rename trivially_optional to vulkaninfo_optional

Expand the scope of the type to allow non trivial types. This helped make the implementation of
parsing command line arguments much cleaner.

vulkaninfo: Fixup argument parsing for --output

There were a few edge cases which weren't well handled. Examples include
providing a filename and using the vkconfig_output but would ignore the file
name; the filename not being picked up for none file outputs like json and
text. Ammended the wait_for_console_destroy() to only run when not printing
to a file.

vulkaninfo: Error check --output

Make sure the following argument is a filename and not another argument.

Also:
Add `--text` as it makes sense to have it as an explicit flag even if it
is the default.
Move --summary to the top, since its the most useful thing to know at a
glance.

vulkaninfo: Allow specifying file output

Use `-o <filename>` or `--output <filename>` to specify the filename of which
vulkaninfo will write to in the current directory.

vulkaninfo: Refactor main function

The main funciton was a chonky 300+ lines of code, this commit pulls out some
of the sections into their own functions to make it easier to read. It also
simplifies the logic in many places by doing so.

vulkaninfo: Remove unecessary surface macro guards

Many of the macro guards in vulkaninfo are unecessary due to not directly
touching the contents of the surfaces outside of already guarded functions.
Thus, there is little point in having guards for stuff that is always able
to compile.

vulkaninfo: Fix formatting

vulkaninfo: Refactor Win32 function loading

The only reason there were global booleans to decide which output to use was due to the macro
WAIT_FOR_CONSOLE_DESTROY. This commit restructures how the win32 function loading operates so
that such globals are no longer necessary. It introduces an OutputCategory enum to declare which
output format to use.

build: Update to header 1.2.200

- Update known-good
- Generate source

scripts: initialize properties for VK_EXT_fragment_density_map2

Update known_good.json

Updated MoltenVK to 1.1.6 release

vulkaninfo: Make printing ToolProperties optional

This field has caused numerous crashes and has generally been more trouble than
its worth to have as a default output. Now, a new command line argument is
needed to get vulkaninfo to print tooling info.

vulkaninfo: Use DebugReport callback

Add a debug callback to vkCreateInstance to debug instance failures,
since there are occassionally errors introduced
which yield validation messages that are currently being missed.

build: Update to header 1.2.199

- Update known-good
- Generate source

vulkaninfo: Prevent drivers writing out of bounds

The structure VkPhysicalDeviceShaderIntegerDotProdcutFeaturesKHR contains only a single
feature boolean flag. However, before the final version was published to the public, it
had many boolean flag values. Pre-release drivers often contain support for unpublished
extensions, such as VK_KHR_shader_integer_dot_product. Because the final version was
much smaller than the pre-release version, several drivers try to write to members that
do not exist in the final version, which cause out of bounds writes to the features struct
that vulkaninfo passes into the driver.

By increasing the size of the features struct allocation manually, we can prevent potential
crashes from OOBs writes from drivers.

build: Update to header 1.2.198

- Update known-good
- Generate source (no change)

build: Update to header 1.2.197

- Update known-good
- Generate source

build: Update to header 1.2.196

- Update known-good
- Generate source

vkcube: Reorder setting of pipeline stage flags

Pipeline stage flags should be set before assigned to pWaitDstStageMask
in VkSubmitInfo.

build: Update to header 1.2.195

- Update known-good
- Generate source

vulkaninfo: Generate vulkaninfo.rc file

Previously whenever a SDK was being built, it would have to manually find and
replace the vulkaninfo.rc file with the appropriate values. This commit makes
it simpler by providing a CMake build argument which allows setting the version
to use from the command line.

build: Update to header 1.2.194

- Update known-good
- Generate source

scripts: Fix codegen with hard-coded enum indices

Currently some of the generators use a fixed child index to locate
extension enums, which will break if comment blocks are added near the
top of an extension definition. This change makes the enum lookup more
robust by searching for the expected enum name instead of using a
hard-coded offset.

build: Update update_deps

Bring forward the update_deps script in this repo to match VVL

ci: Add android build to github actions

Change-Id: I1ef9815548f7483ddc712c76210cf938e70c3eae

build: Remove unused components from android build

When Vulkan-Tools split off from Vulkan-LoaderAndValidationLayers it
inherited the android build scripts, most of which are not relevant to
the projects left in this repo. Also update BUILD.md with latest android
toolchain and remove references to the validation layers.

Change-Id: If982829fc20e14156c52a8f364ef7294733f62cd

scripts: Use f-strings in vulkaninfo generator

f-strings are a python feature which allow more natural formatting of text, making
the code easier to read and work with.

vulkaninf: Support 64 bit flags

64 bit flags use the bitvalues field instead of the requires field in the XML
to specify the flagbits name. Thus the autogen needed to account for that.
In addition, if a flag was promoted to core, the autogen didn't use the core
version, this commit also fixes that.

build: Update to header 1.2.193

- Update known-good
- Generate source

build: Update to header 1.2.192

- Update known-good
- Generate source

scripts: Ignore compatibility constants in wrapper

When the working group changes the name of a constant:

  VK_KHR_MAINTENANCE1_SPEC_VERSION ->
  VK_KHR_MAINTENANCE_1_SPEC_VERSION

they will still #define the original name to maintain backward
compatibility with existing code that uses it:

  #define VK_KHR_MAINTENANCE_1_SPEC_VERSION 2
  #define VK_KHR_MAINTENANCE_1_EXTENSION_NAME "VK_KHR_maintenance1"
  #define VK_KHR_MAINTENANCE1_SPEC_VERSION  VK_KHR_MAINTENANCE_1_SPEC_VERSION
  #define VK_KHR_MAINTENANCE1_EXTENSION_NAME VK_KHR_MAINTENANCE_1_EXTENSION_NAME

In this case, we need to ignore the non-numeric _SPEC_VERSION and
unquoted _EXTENSION_NAME constants because they don't define a separate
extension.

build: Update to header 1.2.191

- Update known-good
- Generate source

vulkaninfo: Fix error message when gpus.size()==0

Avoid an amusing and slightly misleading error message:

"The selected gpu (0) is not a valid GPU index. The available GPUs are
in the range of 0 to 18446744073709551615."

Update known good to the latest MoltenVK tag

cubepp: Fix gitlab 828.

Expect and deny negative window dimensions.

build: Update to header 1.2.190

- Update known-good
- Generate source

build: Update to header 1.2.189

- Update known-good
- Generate source (no change)

build: Update to header 1.2.188

- Update known-good
- Generate source

scripts: Use altlen instead of parsing latexmath

KhronosGroup/Vulkan-ValidationLayers switched to this method a while ago
and this syncs the KhronosGroup/Vulkan-Tools version of
helper_file_generator.py to match.

build: Update to header 1.2.187

- Update known-good
- Generate source

build: Update to header 1.2.186

- Update known-good
- Generate source (no change)

Change-Id: I529e0a7175f18f50ae717c4caae27ef066b80839

ci: Add verification step for generated source

This check ensures that the committed generated source files match the
output of the committed generators. We used to have this check on
TravisCI, but it must have gotten lost in the transition to GitHub
Actions.

Change-Id: If6a27994b3bfe8dbd09a962d90cc0c528489bd93

mockicd: Commit missing generated files

These changes should have gone in with the 1.2.185 header update

Change-Id: Ifeded577343a3ee6cf0ca5d14c420bc5d97b28fc

build: Update to header 1.2.185

- Update known-good
- Re-enable code generation for VK_HUAWEI_subpass_shading
- Generate source

Change-Id: Ibb8d8898c9cdbb655c8162e76b68fa465fe8f912

vulkaninfo: correct copyright date

The copyright date declared in vulkaninfo/vulkaninfo.rc
ends in 2020.

vulkaninfo/cube: added support for FreeBSD

Made some changes for Vulkan-Tools to compile on FreeBSD.

build: Update to header 1.2.184

- Update known-good
- Generate source

Change-Id: I6f8219e31038b0efe1e141147292d041655dd640

docs: Fix cube wsi selection variable

Change-Id: I0b4d5502d98b5602013c6d56cf9cd4ee4d55db9e

known_good.json updated for latest MoltenVK tag

build: Update to header 1.2.183

- Update known-good
- Generate source

Change-Id: Idd87f69c520cd4ef34a1b9d205f79f7c1c7d9e05

windows: updated copyright notices

Updated the master copyright (for Khronos, Valve, and LunarG) to 2021.

Also identified that the four files containing the MIT copyrights were
last updated by LunarG in 2017, so updated those copyrights (again for
Khronos, Valve, and LunarG) accordingly.

I also corrected the spelling of "open-source".

vulkaninfo: Fix device version check

am 72c2d5f0b0e59bf429144d2e60d8fd6ce1710887

vulkaninfo: Fix image format query arguments

The VK_IMAGE_USAGE_TRANSIENT_ATTACHMENT_BIT and
VK_IMAGE_CREATE_SPARSE_BINDING_BIT were swapped, causing erroneous output
and validation messages.

Change-Id: Iff61c379f66aedb14e27e7f422516e1469274248

vkcube: Remove unused variable from Wayland build

Change-Id: Iedc4929547cd58063481e0cf19dda6ba38846a16

build: Update to header 1.2.182

- Update known-good
- Add support for printing `int64_t` in `scripts/vulkaninfo_generator.py`
- Disable codegen for VK_HUAWEI_subpass_shading
  KhronosGroup/Vulkan-Docs#1564
- Generate source

Change-Id: I4417e99ed37da510bfaf94d676e22ac3fb088d5d

cube: Handle occlusion where currentExtent == 0xFFFFFFFF

The Vulkan spec states that: "On some platforms, it is normal that maxImageExtent may become (0, 0), for example when the window is minimized. In such a case, it is not possible to create a swapchain due to the Valid Usage requirements."

Yet vkcube previously only handled this path for platforms where currentExtent != 0xFFFFFFFF.

cube: Don't constantly call vkAllocateCommandBuffers and vkBeginCommandBuffer when minimised

Before this, vkcube would leak these crash when minimised for long periods of time.

vkcube: Remove unused variable

Change-Id: I8064d5e97700cb1a233b61d37aba92ca5f752560

vulkaninfo: Dont print None in vkconfig output

vkconfig prints arrays as it finds them, even if there is only a "None"
element in it. This commit prevents this element from being printed,
removing confusing results where there is a `count = 1` but contains "None".

Change-Id: I1ef8c63458728a9fd3dc531dc9983f309c851034

vkcube: Fix alpha enum type

Change-Id: Id85e0228aeb834de37586fa5d8d334ddc1c7371e

vkcube: fix crash on exit from double delete

build: Update to header 1.2.181

- Update known-good
- Generate source (no change)

Change-Id: Ibfdf958ca32779f7635c324af3ed4146186b27c9

build: Add Werror and warning flags to builds

Added the Werror, Wall, and Wextra flags to builds in CMake.

Change-Id: I58ff7c7d175b512906141b0e691f42b8eb4bf68f

cube: Port Wayland impl from wl-shell to xdg-shell

This change ports vkcube in both C and C++ versions to use the stable
xdg-shell protocol for Wayland compositors. The original wl-shell
protocol has been deprecated and is being removed from major compositor
libraries; wlroots has already dropped support.

New cmake modules have been added to look for Wayland-Protocols
(containing XML descriptions of all common protocols) and
`wayland-scanner`, the tool used to convert the XML files to a usable C
interface.

The change also adds support for the xdg-decoration protocol, which for
some compositors is provided and needed to let them know that they
should draw a titlebar, borders with resize controls, and other standard
features.

Change-Id: I39bedda93a7c5d0aeeb59c68023552723b413567

vulkaninfo: Improve formatting of surface caps

VkSurfaceCapabilities has the width & height way out to the right
while the rest of the structure doesn't. This makes it look inconsistent.

This PR also changes the layout of the flag and flagbit printers to return
earlier if possible.

Change-Id: I1b478838b3c95e33354d3daf13040e891454324e

vulkaninfo: Use autogen supportedSurfaceCounters

Change-Id: I65056198dfe1c9a10c9f04f09f9bdb11d4ff903e

vulkaninfo: Fix compiler warnings

Fixes the implicit conversion of int to size_t and hidden declarations.

Change-Id: I98a3eeb76acfc4dffa0c6a06bb6e72931e4e16d6

vkcube: Fix compilier warnings

Change-Id: I199d4064f0e034cb747a87844c250203fcf22157

build: Update to header 1.2.180

- Update known-good
- Generate source

Change-Id: I41d51ecab3257530ad733fe977f92d9c482543e6

vkcube: Resize on SUBOPTIMAL if necessary

Change-Id: I6421c661880695ba359969fb5e8752e2140167f5

vkcube: Fix disturbed rotation after many iterations

vkcube computes the model matrix for the cube by repeatedly multiplying
a rotation matrix with it. After a sufficient number of iterations the
matrix becomes disturbed (likely due to numerical errors) and stops
being orthogonal and the cube vertices start scaling increasingly on the
XY plane.

* Orthonormalize the model matrix after each multiplication to prevent
the scaling.
* Use mat4x4_rotate_Y() instead of mat4x4_rotate() as it is less
verbose.
* Remove unused duplicate vkcube_vs_uniform struct.

Trigger demo quit from applicationWillTerminate

Allows for a clean shutdown of the rendering thread (CVDisplayLink) whilst the main thread is exiting

tests: Run update on Linux before tests

Change-Id: I523972dd44236659764e12d00ebf2ccb2568d5a6

build: Update to header 1.2.179

- Update known-good
- Generate source

Change-Id: Ib8666fe063852297f746dbea1b8b34346000b13d

vulkaninfo: Update License and Refactor AppGpu

* Simplify AppQueuefamilyProperties
* Create and store AppQueueFamilyProperties in AppGpu instead of creating them
on the fly
* Make all uses of 2 struct's use the original struct where appropriate
  * This means the printers will use props.limits instead of props2.limits
* Remove unused functions
* Remove unused variables
* Add sparseBinding to enabled features (previously there were none)
* make_vulkan_version function introduced

Change-Id: I872dd060dbf709b5b39610775fb88718ff8b19a5

vulkaninfo: Remove redundant platform guards

Change-Id: Id53c7c1cbee21f7113a8e82f64711915119d1d54

vulkaninfo: Tidy up code

* Initialize many variables
* Add const where appropriate
* Use any_of instead of a raw for loop

Change-Id: I7d0c1ff8c1798d7ff589b4d6a641dba32966a5e8

vulkaninfo: use trivial_optional type instead of pair

Replace the ad-hoc pair with a bool and a type with a proper class that
makes the interface clear. The `util::trivial_optional<T>`
type requires T to be trivially copyable and constructable, but otherwise
follows the conventions of C++17's <optional>.

Change-Id: I5b3b981be0af2ecd4b05380a5d030b9ecc0be1d0

vulkaninfo: Add check for printing memory budget

Added check for the `VK_EXT_memory_budget` before printing memory usage
and memory budget under VkPhysicalDeviceMemoryProperties.

Change-Id: I98bd22ede919087700950e67a206303026540d3a

icd:Consider DS when setting format bits

When setting device format properties, don't set color attachment-
related bits for DS formats and don't set DS attachment bit for
color formats.

Fixes #445

build: Update to header 1.2.178

- Update known-good
- Generate source

Change-Id: Iecb34b6221526dc5c6153c736fce894488989066

vulkaninfo: Add pipelineUUID printing to text,html

Change-Id: Ia9b1679188505a765c33b14aab46749f823f4a23

Update known_good.json

Updated to the latest MoltenVK tag.

vulkaninfo: Fix signed conversion warning in MSVC

Change-Id: I0ff94c9c09707fd6a49cefa62e32e719e207b721