loader: allow GAS on MinGW

Disabled by default, user must explicitly enable USE_GAS.

Fix CI sanitizer failures

build(deps): bump github/codeql-action from 3.24.6 to 3.24.8

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.6 to 3.24.8.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/8a470fddafa5cbb6266ee11b37ef4d8aae19c571...05963f47d870e2cb19a537396c1f668a348c7d8f)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.280

- Update known-good
- Generate source

apple: Updated framework version contents for appstore validation

build(deps): bump github/codeql-action from 3.24.5 to 3.24.6

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.5 to 3.24.6.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/47b3d888fe66b639e431abf22ebca059152f1eea...8a470fddafa5cbb6266ee11b37ef4d8aae19c571)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.279

- Update known-good
- Generate source

Add app packages to layers search path

Expand Windows system search path for layers to include AppX/MSIX
packages currently scanned only for ICD manifests.

build(deps): bump github/codeql-action from 3.24.3 to 3.24.5

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.3 to 3.24.5.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/379614612a29c9e28f31f39a59013eb8012a51f0...47b3d888fe66b639e431abf22ebca059152f1eea)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build(deps): bump github/codeql-action from 3.24.0 to 3.24.3

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.24.0 to 3.24.3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/e8893c57a1f3a2b659b6b55564fdfdbbd2982911...379614612a29c9e28f31f39a59013eb8012a51f0)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.278

- Update known-good
- Generate source

Remove current directory from DLL searching

By removing the current directory from the search paths for dlls, the
loader is less vulnerable.

build(deps): bump github/codeql-action from 3.23.2 to 3.24.0

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.23.2 to 3.24.0.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/b7bf0a3ed3ecfa44160715d7c442788f65f0f923...e8893c57a1f3a2b659b6b55564fdfdbbd2982911)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.277

- Update known-good
- Generate source

fix dEQP-VK.api.object_management.multithreaded_per_thread_resources.device_group crash

This commit reverts the a4ff6a54e4f06638fa8fb367913818bf9d189bdb, which
will cause this CTS case crash when build the loader with release type.

Fix the tests, so it's not adding multiple DXGIAdapters with same LUID

LUID is expected to be unique per D3DKMT_/DXGIAdapter.

Fix heap corruption in Windows tests.

Fix memory leak.

If we never find any other device after already_enumerated is recognized, the memory allocated in this function will never be freed.

build(deps): bump github/codeql-action from 3.23.1 to 3.23.2

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.23.1 to 3.23.2.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/0b21cf2492b6b02c465a3e5d7c473717ad7721ba...b7bf0a3ed3ecfa44160715d7c442788f65f0f923)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.276

- Update known-good
- Generate source

loader: Check apiVersion is not 0 for VUID-VkApplicationInfo-apiVersion

build(deps): bump github/codeql-action from 3.23.0 to 3.23.1

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.23.0 to 3.23.1.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/e5f05b81d5b6ff8cfa111c80c22c5fd02a384118...0b21cf2492b6b02c465a3e5d7c473717ad7721ba)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix lack of pthread linking in ubuntu 20.04

Ubuntu 20.04 saw a regression that caused the pthread library to no longer be
a part of libvulkan.so's linked libraries. This didn't cause the loader to fail
to load, rather any library that used pthread functions without linking to pthreads
themselves would fail to run, for example the Vulkan-ValidationLayers (VVL).
While it is currently unclear why VVL were trying to link to these symbols, this
would make loading validation layers impossible.

The issue turns out to be this line of CMake
`set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY)`. Specifically, that line of
code appearing *before* `find_package(Threads REQUIRED)` causes CMake to 'find'
CMAKE_HAVE_LIBC_PTHREAD successfully, whereas previously it would fail. Since
this variable indicates that linking to libc is enough to get threading
capabilities, CMake no longer links to pthread.

This commit is an effort to prevent breaking ABI due to changing the link library
list in Ubuntu 20.04. It should be noted that in Ubuntu 23.10, the location of
`find_package(Threads)` has no bearing on the value of CMAKE_HAVE_LIBC_PTHREAD.

loader: Miscellaneous platform-specific fixes

build(deps): bump github/codeql-action from 3.22.12 to 3.23.0

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.22.12 to 3.23.0.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/012739e5082ff0c22ca6d6ab32e07c36df03c4a4...e5f05b81d5b6ff8cfa111c80c22c5fd02a384118)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.275

- Update known-good
- Generate source

build: API parameterization changes

Set CMake properties consistently

Simplifies the code

Idiomatic way of setting visibility via CMake

Document BUILD_WERROR

BUILD_WERROR OFF by default

Consistent naming with other repos we maintain.

OFF by default to avoid issues with newer compilers, specific
generators (Xcode), etc.

Only require C compiler

By default the loader only requires the C compiler.

But by default the C and CXX compilers are enabled.

Only enable the C compiler. Move corresponding C++ code into tests
directory.

Add integration testing

Test find_package(VulkanLoader CONFIG)

Minimal vulkan.pc testing is also added.

Cleanup GitHub action script

build(deps): bump github/codeql-action from 3.22.11 to 3.22.12

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.22.11 to 3.22.12.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/b374143c1149a9115d881581d29b8390bbcbb59c...012739e5082ff0c22ca6d6ab32e07c36df03c4a4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Add VulkanLoaderConfigVersion.cmake

Provides versioning information for the CMake package

Improve clang on Windows testing

Test both GNU clang and clang-cl

Use Ninja for performance

Make sure VkSurfaceKHR is cast to uintptr properly

Fix wrong pointer casts in win32 surface code

The bad pattern was detected using clang/gcc but only on linux, so the
win32 code path never saw the fix.

Fix spelling mistakes in comments

These creep in over time, and make the codebase harder to understand due to spelling
errors causing confusion when reading comments.

Use memset to clear structs

Several places manually zero'd out each member of a struct. This makes it possible to
forget to zero out members, especially as structs change over time. Using memset
prevents that possibility.

build: Update to header 1.3.274

- Update known-good
- Generate source

build(deps): bump github/codeql-action from 2.22.9 to 3.22.11

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.9 to 3.22.11.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/c0d1daa7f7e14667747d73a7dbbe8c074bc8bfe2...b374143c1149a9115d881581d29b8390bbcbb59c)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Revert "Add simple timer of ICD preloading in live tests"

This reverts commit 2e502bf4c582a9fb2f1a5b863c74475c1a05972d.

Revert "Unload drivers which report 0 physical devices"

This reverts commit ba275bd829ee7f6f2682df4a84a9ae8481c5b801.

Revert "Use memset to clear structs"

This reverts commit 93855ba4e2dc1fba6a19fae4ee8bcf1a8462aad1.

Revert "Fix spelling mistakes in comments"

This reverts commit 0133778cb1c4493a9d580700b8a6e0fe493c601c.

Fix version installation of vulkan-loader

Currently libvulkan.1.3.0.dylib gets installed instead of
libvulkan.1.3.273.dylib

This fixes that while keeping the fix for the linker warning.

build(deps): bump github/codeql-action from 2.22.8 to 2.22.9

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.8 to 2.22.9.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/407ffafae6a767df3e0230c3df91b6443ae8df75...c0d1daa7f7e14667747d73a7dbbe8c074bc8bfe2)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build(deps): bump actions/setup-python from 4 to 5

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

ci: Remove macos-11 builds

CMake 3.28 is having issues with the macos-11 runner.

build: Update to header 1.3.273

- Update known-good
- Generate source

Fix spelling mistakes in comments

These creep in over time, and make the codebase harder to understand due to spelling
errors causing confusion when reading comments.

Use memset to clear structs

Several places manually zero'd out each member of a struct. This makes it possible to
forget to zero out members, especially as structs change over time. Using memset
prevents that possibility.

Unload drivers which report 0 physical devices

The loader did not unload any ICD's which contained zero physical devices, which
could cause premature exhaustion of memory in some circumstances, like 32 bit
applications. While the policy of the loader has been to keep things open for
the duration of the instance, these ICD's don't meaningfully participate in
anything due to the lack of VkPhysicalDevices.

This change adds a check after vkEnumeratePhysicalDevices where pPhysicalDevices
is not NULL such that all loader_icd_terms which reported zero physical devices
have its vkDestroyInstance called, and removed from the loader_instance's
icd_term linked list.

Add simple timer of ICD preloading in live tests

Adds a simple executable that times how long vkEnumerateInstanceExtensionProperties
takes over and over to see how well the ICD preloading functions.

Support Clang with GNU interface on Windows

By using CMAKE_CXX_COMPILER_FRONTEND_VARIANT, CMake will now distinguish
clang from clang-cl, allowing the correct command line argument formats
to be used.

Making tests work required fixing up windows_shim.cpp's usage of wcscpy
since that was triggering deprecation warnings. A simple for loop was
used instead.

Specify which layer type is being searched in log

Makes it clear which layer type is being searched for in any given set of paths.

build: Update to header 1.3.272

- Update known-good
- Generate source

docs: Remove comments about Travis/AppVeyor

Add clang-format to CI

Rename BUILD_STATIC_LOADER to APPLE_STATIC_LOADER

BUILD_STATIC_LOADER is ambiguous and implies it will work on all
platforms.

APPLE_STATIC_LOADER isn't ambiguous.

Furthermore don't document BUILD_STATIC_LOADER. Otherwise users
may use it when they shouldn't.

Mark APPLE_STATIC_LOADER as advanced so it doesn't show up in
cmake-gui by default.

Furthermore, make the build fail if someone tries to use
APPLE_STATIC_LOADER on a non Apple platform.

Test Xcode generator

Needed for iOS / simulator testing

build: Update to header 1.3.271

- Update known-good
- Generate source

build(deps): bump github/codeql-action from 2.22.7 to 2.22.8

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.7 to 2.22.8.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/66b90a5db151a8042fa97405c6cf843bbe433f7b...407ffafae6a767df3e0230c3df91b6443ae8df75)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Instance extensions unsupported by physical devices now return ERROR_EXTENSION_NOT_PRESENT

This is makes the code gen for VK_EXT_acquire_drm_display be in line with the wsi code
whenever calls to drivers are made which do not support the given extension.

Remove -fno-strict-aliasing from builds

This was added in commit 72199501cb01497ddc0e429d561e2e7e17a841c4

It was purported to fix a hang, but was never fully verified if it actually
fixed the underlying issue (or if the issue came from buggy optimizing
compilers or drivers).

The chromium build does not enable -fno-strict-aliasing and hasn't had any
issues known to be caused by strict aliasing optimizations.

This commit also fixes debug report/debug utils code that was not properly
casing to uintptr_t before assinging or dereferencing function parameters.
The same logic occurs in wsi.h which as it assigned a pointer of VkIcdSurface
to the VkSurface parameter.

Add Undefined Sanitizer support

Allows easy enabling of UBSAN.

Fixes one issue found with UBSAN in the tests.

build(deps): bump github/codeql-action from 2.22.5 to 2.22.7

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.5 to 2.22.7.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/74483a38d39275f33fcff5f35b679b5ca4a26a99...66b90a5db151a8042fa97405c6cf843bbe433f7b)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Remove API_NAME

Remove -fno-builtin-memcmp

Warn if apiVersion is below VK_API_VERSION_1_0 and above 0

Setting the apiVersion to an invalid value is an easy mistake to make.
But because it causes drivers to fail, the validation layers can not catch
it in time.

This commit makes it a 'fatal error' which forces log_message to write out
the message, regardless of whether or not VK_LOADER_DEBUG is set or if the
application has setup a debug callback in the pNext chain.

ci: Test universal binaries

proc_pidpath takes a uint32_t not a size_t

Fix settings file crash found through fuzzers

A few error conditions weren't properly handled whenever the settings file is invalid.
Added the exact json file which caused the errors, then fixed issues with releasing
memory properly.

Improve warning message from terminator_CreateInstance

Old messages were clunky and lacked helpful info like which driver failed
and the error code returned by the driver.

Cleanup loader_icd_init_entries

The logic of the function indicates that at a previous time, the
functions being loaded used dlsym/GetProcAddress to query functions.
Whenever that was changed to vkGetInstanceProcAddr, the error logging
wasn't updated to remove loader_platform_get_proc_address_error.
Trying to call dlerror() when no error has occured will return NULL,
which was dutifully passed onto loader_log, causing crashes.

The cleanup removes usage of that, as well as making the loading
logic uses 2 macros, one for required & one for non-required
function pointers.

Cleanup also meant streamlining the parameter list of
loader_icd_init_entries, since VkInstance and GetInstanceProcAddr
are both able to be gotten from the icd_term. And also meant cleaning
up the inst parameter being passed into loader_log, which was turning
the drivers VkInstance into loader_instance* erroneously.

A test was added to ensure that the error path when drivers do not
support the required functions correctly returns an error from
vkCreateInstance instead of crashing.

Fix typo in LoaderDriverInterface.md

Linux drivers use the ".so" extension, not ".dylib"

build: Update to header 1.3.270

- Update known-good
- Generate source

cmake: Fail if Fuchsia

Fuchsia builds are only properly supported via the Chromium
build.

Remove vulkan.symbols.api not actually used anywhere

Allow building a universal binary

My earlier error checking was too agressive and doesn't
allow for building a universal binary at all.

Move the error check inside of USE_GAS so it's clear to the user
what to do (IE disable GAS if building a univeral binary)

cmake: Document why BUILD_STATIC_LOADER doesn't install

Implement VK_MSFT_layered_driver extension support

This extension reorders physical devices enumerated through the windows
EnumerateAdapterPhysicalDevices whenever multiple drivers exist for the same
LUID. This is so that if a driver is considered a 'layered implementation',
eg Dozen's Vulkan on Dx12, the physical device corresponding to the native
vulkan driver is preferred.

The implementation of this extension cause some renaming of structs, variables,
and parameters to make the code easier to understand, such as renaming
the struct loader_phys_devs_per_icd to loader_icd_physical_devices.
During the review process there was quite a bit of confusion about what the
struct's purpose was that should be much clearer now.

tests: Create seed corpus data

Enable layer log messages in DebugUtils

Debug Utils messengers wouldn't receive the layer or driver messages.
Mofified the messages so that the they would be logged in layers or
other debug utils messenger.

Fix Debug callbacks in layers in vkCreateInstance

After a layer calls down to vkCreateInstance, it may need to create a
DebugUtils Messenger (or Report object) for capturing debug information.
With the old design, it would get handled as a "during vkCreateInstance"
debug utils messenger object because the loader trampoline was still
in the middle of "vkCreateInstance".  However, treat it separately
if it was created with a call to vkCreateDebugUtilsMessenger and that
allows it to capture info all the way until the instance is destroyed.

build(deps): bump github/codeql-action from 2.22.4 to 2.22.5

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.4 to 2.22.5.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/49abf0ba24d0b7953cb586944e918a0b92074c80...74483a38d39275f33fcff5f35b679b5ca4a26a99)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

cmake: Fix x86 / x64 check

closes #1356

Fix instance create swallowing error

docs: Document QNX build

build: Cleanup QNX build

Move QNX build scripts into the scripts directory. This is
consistent with how we deal with the chromium build.

Provide a useful error to CMake users trying to cross-compile
for QNX. Although CMake does support QNX we want to point users
to the official method.

cmake: Move include(CheckIncludeFile) into loader/CMakeLists.txt

Currently it's confusing placed in the top level CMakeLists.txt

cmake: Fixing unsecure environment warnings for MinGW/Apple

WIN32/Apple doesn't support these functions

build: Move fushcia source files into loader directory

Mainly cosmetic to have 1 less top level directory

ci: Add iOS / tvOS builds

scripts: Update update_deps.py

Check for nullptr in vkCreateInstance trampoline

pCreateInfo must not be NULL, but the loader was dereferencing the pointer
before checking if it is NULL. Moving the dereference below the check for
NULL ensures no nullptr dereferencing occurs.

Dont assert in vkAllocateCommandBuffers

The loader_init_dispatch first checks if the dispatchable handle has the
ICD_MAGIC_VALUE before setting the handle to the correct dispatch pointer.
This is useful to check that driver's are correctly creating the handle
using VK_LOADER_DATA.

Because layers are capable of wrapping handles, the loader_init_dispatch
function should only be called in terminators which run before any layers
have a chance to modify the ICD_MAGIC_VALUE. Otherwise the assert fails.

vkGetDeviceQueue & vkGetDeviceQueue2 uses loader_set_dispatch, which does
the same code as loader_init_dispatch but without the assert.

This commit changes vkAllocateCommandBuffers to use loader_set_dispatch.

build(deps): bump github/codeql-action from 2.22.3 to 2.22.4

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.3 to 2.22.4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/0116bc2df50751f9724a2e35ef1f24d22f90e4e1...49abf0ba24d0b7953cb586944e918a0b92074c80)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

build: Update to header 1.3.269

- Update known-good
- Fix hard-coded tag index for getting extension names from vk.xml
- Relax regex for correlating kVulkanObjectType* with
  VK_DEBUG_REPORT_OBJECT_TYPE_* by making the trailing _EXT optional
- Generate source