layers: Add component awareness to SC interface matching

- Adds more tests for mismatch by component and by location
- Removes dead handling of builtins (we only care about collecting builtin /decorations/ to aid in the interface block handling;
    we don't really care about producing the set of builtins used overall.)

Some SC error messages change slightly, as interstage locations now include the component index.

Fixes internal bug 26609754

V2: Remove dead code

Signed-off-by: Chris Forbes <chrisforbes@google.com>

loader: Fix MSVS warnings

Apply branch 'fix-warnings' of https://github.com/null77/Vulkan-LoaderAndValidationLayers into null77-fix-warnings

layers: Fixed markStoreImagesAndBuffersAsWritten() in mem_tracker

It now correctly treats images as images instead of buffers.

loader: Update doc to add the Linux search path

$HOME/.local/share/vulkan/icd.d or explicit_layer.d or implicit_layer.d

loader: Add example code

loader: doc cleanup

loader: doc cleanup

Fix some awkward wording.

Conflicts:
loader/LoaderAndLayerInterface.md

layers: Make layer option names more consistent and update docs

Check boolean values in vkCreateSampler

demos: Fix vulkaninfo on win32 to use existing console

The vulkaninfo utility is compiled as /SUBSYSTEM:APPLICATION but
attempts to allocate its own console and print to it. Not only
does this appear to be unreliable (sometimes no output is printed),
it is unnecessary if the utility is compiled as /SUBSYSTEM:CONSOLE.

This removes unnecessary console manipulation code. By using
/SUBSYSTEM:CONSOLE, the ordinary main() entrypoint function can be
used.

The console is still enlarged if it was opened exclusively for the
process by detecting whether it was so opened. Likewise, the process
still sleeps forever if the console is exclusive to the process.
The process behaves like a normal command line process when executed
from the command line.

Fixes lunarg issue 350.

docs: fix bad link in layers/README.md

layers: Added TODO in drawstate for Khronos-github issue #30

layers: draw_state VK_WHOLE_SIZE in barriers

Allow use of VK_WHOLE_SIZE in barriers validation.

layers: Allow draw_state layout mismatch with UNDEFINED

Also allow CB initialLayout of VK_IMAGE_LAYOUT_UNDEFINED to match
any layout.

layers: Fix memory leak in mem_tracker

Certain codepaths could result in the validation_functions vector growing
unbounded, causing performance hits in QueueSubmit.

layers: Remove unnecessary null pointer checks

Fixes GitHub Issue #6

layers: LX232, Removed invalid object_tracker QueueBindSparse error message

layers: MR247, Snap start and end to page to compare for mem_tracker

loader: Fix typos, spelling, grammar in loader interface doc

build: Remove LunarGLASS from Win update ext sources.

demos: Properly destroy DebugReportCallback

Error resulted in warnings when shutting down tri --validate.
Also added DebugReportMessage setup.

Update build info to reflect ICD moving to VulkanTools

demos: Fix drawstate transition errors in tri.

Build: Remove LunarGLASS from update_external_sources.sh

LunarGLASS is no longer built from here since it not needed for this repo.
Rather, an update_external_sources.sh has been added to VulkanTools which
can be use to build LunarGLASS.

glslang: Update revision to pick up recent fixes

loader: Fix Wayland support

All that was needed was to fix some misplaced braces, fix a couple of
copy+paste problems from Mir, and add a lookup function.  Getting Mir
support working should be pretty easy.

Conflicts:
layers/swapchain.cpp

cmake: Add options for each of the linux wsi interfaces

We default everything except XCB to off since that is the only one that has
been well-tested up to now.

Don't use width=0,height=0 on window minimize

Vulkan doesn't support width=0 and height=0 so we just ignore the
minimize request on Windows.

demos: Add --validate option to tri

First tries to use standard_validation, if not found then a list of
validation layers.

loader: Fix standard_validation enumeration on instance chain

Fixed regression and still handle no layers found case.

winrtinstaller: update README.txt to better explain rt installer exit codes

layers: Allow draw_state layout mismatch with UNDEFINED

loader: Fix threading layer name in loader standard_threading

loader: Environment variable override got broken with recent change

This fixes that

Conflicts:
update_external_sources.bat

layers: Fix android build and update hash.

Update android build code to fix generated paramchecker.

Fix draw_state custom hash algorithm for ImageSubresourcePair.

loader: Fix windows build from last loader checkin

loader: Add a ICD and layer search path for loader to discover JSON files

path is $HOME/.local/share/vulkan/*

layers: Fix typo in mem_tracker

layers: param_check- enable android build

Update android build to use new param_check header files

layers: Fix possible crash in draw_state layer

Uninitialized descriptor slots are not filled this could cause
null pointer dereferences.

layers: Fix UpdateDescriptorSets crash in mem_tracker

Validation was handling TEXEL_BUFFER descriptorTypes incorrectly.
Removing for now.

demos: Adjust vulkaninfo failure cases

For Windows, after vulkaninfo allocates a console, it tries to resize the
console and change the title text.  Failure of either operation was
treated as console icreation failure, which led to vulkaninfo printing
to the console and then closing the console immediately.  Now treating
AllocConsole() failure as console creation failure and ignoring resize and
title text change failures.  With this change, resize failure will not
stop the app from pausing after printing info to the console.

winrtinstaller: change error message

winrtinstaller: Unique error codes

layers: MR243: NULL pointer and struct type checks

Verify that required parameters are not NULL and that structures with
sType fields have the correct VkStructureType values.  Messages are
logged for the following conditions:
 - A pointer is NULL and is not marked as optional in the XML
 - An array is NULL and is not marked as optional in the XML, unless
   its count is 0
 - An array count is 0 and is not marked as optional in the XML
 - A structre's sType value does not match the value specified
   in the XML

layers: MR243: Add param checker code generation

Generate param checker code to verify that required pointer/array
parameters are not NULL and that structures with sType fields
have the correct VkStructureType value.  The checks are generated
from the Vulkan XML API registry based on the optional parameter
tag and sType comments.

spirv-tools: update versions for 1.0.3

layers: Fix android build of safe_struct wrappers

Handle default case in switch statement for descriptor types.

layers: MR221: Correct name for unique_objects layer

layers: MR221: rename threading test

The threading validation layer has been basically
re-written, so updating the vendor name accordingly

layers: MR221: Android needs Enumerate*Properties

Android queries information about layers and extensions
from the layers via the Enumerate*Properties calls so
need them to function correctly. This patch adds missing
functions.

---------------------------

layers: MR221: Add Enumerate for Android

Android does not use json files to store layer and
extension information and needs to query it directly
from the layer.

layers: MR221: Update extension info to match json files

swapchain: MR221: Remove unused property info

swapchain: MR221: Use proper layer name

layers: MR221: Fix implementationVersion

layers: Special case VkWriteDescriptorSet safe_struct code

VkWriteDescriptorSet may have non-null ptrs that we don't care about.
Add a special case for this struct in safe_struct codegen that will only
check the array ptrs that we care about based on the descriptorType.

Also update safe_struct codegen so that all ptrs in structs are initially
set as NULL, and then only updated if src struct ptr is non-NULL.

Update unique_objects to base "if" guards around ptrs to be based off of
the shadowed safe_struct ptrs for any ptrs below the first level input
params. This is because safe_struct initialization will have already copied
down any ptrs so if there are any additional guards in safe_struct
shadowing, such as for VkWriteDescriptorSet, this allows unique_objects.cpp
to make use of only valid info and not have to replicate special cases.

loader:Force Cmake to build dev_ext_trampoline.c always relase mode

This makes sure this file always has tail call optimizations occuring

Update build info and references to new public GitHub names

Update build instructions to reference new repository names in GitHub

Update all references to the new public GitHub repository names, and
also switches from SSH access to the repositories to HTTPS access.

Update glslang_revision to reference new GitHub repo

layers: Fix Win32 build errors in draw_state

layers: Re-enable support for linux

loader: Add better checks for malformed disable_environment in layer JSON file

layers: Update debug_report spec version in json files to match vulkan.h

loader: move doc png file to loader current directory for use of same file in SDK

loader: more doc updates

loader: Add more info in layer interface doc, the layer section

loader: Doc review comments

loader: Add images to interface Doc

loader: Add line breaks to interface doc

Wrapped long lines to make it easier to edit in text based editor.
Also hoping that editing diffs could be smaller since a line is not
a whole paragraph.

loader: change so LoaderAndLayerInterface document is the sole document

layers: Fix Windows build with device_limits.cpp

loader: check for null pointers in loader_add_layer_property_meta

Fixes vulkaninfo when only RT is installed and there are no layers
installed.

loader: For dev ext trampoline error code, add error logging

loader: add support for standard_validation layer enabled from env var

Also fix memory corruption issue with the original std_validation patch.

loader: Add feature standard-validation layer from application

Adds a validation layer grouping VK_LAYER_LUNARG_standard_validation, which
is the set of validation layers in a fixed order.

This only adds support for enumerating the meta layer and enabling it explicitly
from the app.  Environment variable enablement of this layer is not yet added.

loader: Update the interface document
Mostly formatting and typos.

loader: rerun clang-format on some files

windowsrtinstall: Fix errors on vcredist not being present before install.

We install the 2013 VC redistributables, but we first read the registry to
determine if they were already installed.  That registry read returns an
error if they aren't already present.  So, we clear the error after reading
the registry.  We also added the 32-bit 2013 VC redistributables to the
64-bit OS install, just to be safe.

codegen: Add special case for DebugReportCallback inconsistency

windowssdk: Fixed Config powershell script to look at Windows Folder.

This replaces a hard-code to C:\windows instead of looking at the
proper windows drive letter <SYSTEMDRIVE and SYSTEMROOT>.

layers/loader: Fix string validation error

header: Update 1.0.3 header revision for inclusion of debug report extension

Removed vk_ext_debug_report.h (contents moved into vulkan.h)
Renamed debug report message enums

android: Add support for Android surface extension

param_checker: Check for pQueuePriorities

Verify that pQueuePriorities is set before using it.

layers: Validate wait event sourceStageMask.

license: move RenderDoc license from LoaderAndTools to SDK

layers: Use accessible id set when determining shader's descriptor use

- Eliminates bogus false conflicts between resources in a module, where the resources share a descriptor slot but
  no entrypoint refers to both.
- Eliminates false positives from shader vs descriptor set layout validation, where we would previously be upset
  about the layout not including a descriptor which was declared in the module but not used by the entrypoint.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Add spirv static call tree walker

With emphasis on finding all descriptor-based resources referenced in the static call tree of an entrypoint

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Collect OpFunction defs

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Pass the entrypoint down to collect_interface_by_location

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Use entrypoint interface specification to collect variables

The module may contain many others -- only the variables listed in
the OpEntryPoint are part of the interface.

Note that the shader resource interface is NOT listed in OpEntryPoint. For now we
continue to conservatively assume every OpVariable in the module with a suitable
storage class is in every entrypoint's shader resource interface -- that is, we're
not walking the static call tree from each entrypoint to determine what is and is
not used by it.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Look up entrypoint for each shader stage

Not doing anything useful with this yet beyond insisting that it is present, but
that will come in later patches.

V4: Add new enum to md

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Minor housekeeping in draw_state.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

demos: Fix a few layout transition bugs in cube

Re-enabled layout validation in draw_state revealed a few bugs in cube. This change fixes those bugs by starting tex image in PREINITIALIZED state with the HOST_WRITE_BIT set.

layers: Fix and re-enable layout checks.

Handle layout transitions for subresources and re-enable layout validation.

layers: Update unique_objects to make use of safe_structs

Stop overwriting const* data in unique_objects.
Instead, declare local safe_struct copies of data, update that
data in place with unwrapped objects, and pass down the local
safe_struct copy, which spoofs the original struct layout.

This includes code to build the layer using vk_safe_struct.h/cpp files.

layers: Update vk_helper to generate safe_struct wrappers

Create vk_safe_struct.h/cpp files that contain wrapped versions of vk structs.
The idea is to wrap structs that contain non-dispatchable objects so that
the non-dispatchable objects are no longer contained within or below "const" decls.
The safe struct wrappers are intended to exactly match the original structures
in how they appear, but they also contain some utility functions to contruct and
desctruct themselves so that ptr chains are automatically replicated.

For unique_objects, the idea is to use the safe_struct wrappers as follows:
foreach struct that contains non-dispatchable objects
   declare local safe_struct version of struct
   initialize struct to exactly match original
   unwrap non-dispatchable objects in place in the safe_struct

layers: Validate specialization entry data is fully contained within the specialization data block.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

Validate buffer barrier size and offset.

layers: Correctly handle VK_WHOLE_SIZE for dynamic offsets