layers: Add todo flagging destroy while on queue

This starts with the case of early VkDestroySemaphore seen in demos.
It is a much larger general case of objects that are active on queues.

demos: vkDestroySemaphore only after it is unused.

Semaphores should not be destroyed when in use.
Call vkQueueWaitIdle before calling vkDestroySemaphore.

layers: Handle bad filename in APIDump settings file

If APIDumpLogFilename requested in vk_layer_settings.txt file has a bad path or cannot be opened for some other reason, then APIDump will write out an ERROR message w/ the bad filename and notify user that STDOUT will be used instead.

vktrace: Add support for debug_marker_lunarg extension

layers: Fix ObjectTracker to handle dyanamic array params

ObjectTracker will now check for valid objects across all entries of arrays that are passed as params. This includes all objects embedded in arrays of structs such as the pCreateInfos array in vkCreateGraphicsPipelines().

vk_platform: Update to upstream vk_platform.h

Gitlab bug #9: https://gitlab.khronos.org/vulkan/LoaderAndTools/issues/9
noted that the SDK's vk_platform.h did not match the upstream vk_platform.h.
This patch makes them identical updates impacted code.

demos: Fix compile warning setting renderPass

nulldrv: Make build work on Windows with destroydynamiclinewidth

loader: Fix SegFault if ICD JSON file doesn't exist to return gracefully

vktrace: GLAVE file/directory renaming

Glave/glv files and directories renamed. Default trace files now named
*.trace.

Remove extraneous temp file

layers: Fix APIDump to correctly set output file name from settings file

Merge branch 'check-cube-arrays-complete' into 'master'

check that cube array views are complete cubes

See merge request !29

check that cube array views are complete cubes

v156: Bump API minor version

v156: Bug 14451 - Rename dynamic state objects

Bug 14474: Remove *_USAGE_GENERAL

v155: Bug 14435 - Remove abbreviations in VkClearColor

v154: Bug 14417 - Added VkShaderStage to VkShaderCreateInfo

v153: Minor version bump for bugs 14323, 14400, 14339, 14406

Bug 14406: Add support for front/back stencil reference dynamic state

Bug 14339: Add the missing alphaToOneEnable to VkPipelineColorBlendStateCreateInfo

Bug 14400: Add FORMAT_FEATURES for blits

Bug 14323: RasterLine and RasterDepthBias changes

loader: Add support for relative pathnames in the ICD manifest file lib_path

Fix formatting for GitLab web interface

layers: Initial commit of Validation Details md file

This is intended to be a comprehensive list of validation checks in the Vulkan validation layers. For each layer, a table is included that describes the checks performed by that layer, the relevant API calls, the associated ERROR/WARNING code, any tests that exercise that check, and further notes or TODOs. Also each layer has a "pending work" section so that checks to be added in the future can be documented here.

This is intended to be a working document where the document stays in synch with the code. Also, if missing validation checks are identified, but cannot be immediately implemented, they should be added to an appropriate "pending" section. Finally, if someone wants to contribute to the validation efforts, they can identify existing "pending" work to implement.

loader: Make so layer manifest files are retained at CreateInstance

layer manifest files were rescanned after CreateInstance as needed for
CreateDevice and GetPhysicalDeviceXXXProperties. Now stored the device
and instance layer list in instance object.

Also fix mem leak of extension list in some cases.
Also fix bug where deep copy of instance  to device layer properties was
not being done. Only a shallow copy was being done.
Also remove the lib_info struct from layer properties structure as only the
lib name field was being used. Other layer library info is stored in the
layer_library_list once the layer libraries are opened.

loader: Make the ICD library scan happen as needed rather than once

Add a one time loader init routine also since this was done by ICD scan.
Add icd library struct to instance object.

loader: Remove the device_extension list from from scanned icds, unused

loader: Remove the global extension list from scanned_icds struct

This list is regenerated as needed and stored in the instance object.

loader: Make the global/instance ext list once for each instance

Also fix some bugs in the heap_alloc/heap_free.
Also fix some memory leaks.

loader: Remove global instance extension list and rebuild it as needed

This will be necessary once ICDs are rescanned at CreateInstance and prior.

tests: Fix tests that re-submit Cmd Buffers w/ ONE_TIME_SUBMIT flag

Added a way to pass VkCmdBufferBeginInfo through test framework.

For tests that do multiple submits, update BeginCommandBuffer() calls to not set ONE_TIME_SUBMIT flag.

layers: Add validation check to enforce ONE_TIME_SUBMIT of cmd buffers

Track createInfo and beginInfo independently per cmdBuffer & check begin flags at QueueSubmit time to verify cmd buffer not submitted multiple times if VK_CMD_BUFFER_OPTIMIZE_ONE_TIME_SUBMIT_BIT is set.

Add CmdBufferTwoSubmits test to verify this check.

cube: Set pipeline renderpass

LunarXchange issue #111:
https://vulkan.lunarg.com/app/issues/55cbc6d77ef24d0001000001
noticed that the pipeline's renaderPass was not being set.

Revert "demos: Add dependency for SPIRV shader components"

This reverts commit 7dffaa146e1ec138b1c6a0b02a3b54d352636683.

layers: Change Threading layer dispatch and logging

Use new log_msg with per instance and device dispatch

layers: Remove unused layer-dispatch target

layers: Optimize DrawState when info is not shown

Return quickly from synchAndPrintDSConfig when info is not requested.

layers: Remove add_dependencies of .def files

The generated .def files should not be listed with add_dependencies.

layers: Remove active flags with removed callbacks

layer_destroy_msg_callback was including flags from just removed callbacks.

demos: Updated vulkaninfo for new features/limits entries

Also cleaned up output

layers:  Fix windows build for multi layer .def file

loader: Fix bug in heap_free function to not do double free

layers: Make multi layer functional again  as a multi layer per library

Pretty much a total rewrite of this layer.

Make multi1 a Device layer and multi2 an Instance layer.
Remove all extension and layer property stuff, rely on manifest file.
Remove all old common stuff, rely on newer loader/layer interface which
can handle multiple layers pre library via manifest file.

loader: Handle multiple layers descriptions per manifest file

Bug 14407 - Add texelCoords to Sampler state

Bug 14339 - Add alphaToOne as an optional feature

v151: Bug 14386 - Change VK_NULL_HANDLE from a 0 to a nullptr

v150: Bug 14384 - Add a subpass field to VkCmdBufferBeginInfo

v149: Fix header ordering bug

nulldrv: Change to new WSI query functions.

layers: mem_tracker: Change to new WSI query functions.

demos: Change to new WSI header ...

Change to:

- Use new WSI query functions.
- Set new VkSwapChainCreateInfoWSI members.

loader: Change to new WSI query functions.

vulkan.py: Change to new WSI query functions.

WSI: Update swapchain headers to to version 13 and 45.

Update "vk_wsi_device_swapchain.h" to version 45.
Update "vk_wsi_swapchain.h" to version 13.

build: rebuild LLVM when LLVM overlay files change

layers: Don't output DrawState dynamic state info for null handles

This was causing random segfaults in the layer validation tests.

loader: Remove one time scan of layer libraries; do it as needed

Everytime get a Vulkan API call which needs layer library info rescan
all layer manifest files to get the info.  This way if they change the list will
be updated.

layers: Handle uninitialized QueueFamily values in APIDump

If queueFamilyCount or pQueueFamilyIndices were uninitialized, APIDump
would persist in printing this info resulting in a segfault. Added
check for sharingMode before examining these values.

build: advance glslang and LunarGLASS versions to SPIR-V rev 31

layers: port ShaderChecker to SPIR-V v31 (from ChrisF)

WSI: Remove old "vs_wsi_lunarg.h" file.

layers: Restore information messages to the MemTracker layer

tests: Move DestroyShaderModule calls after pipeline creation

vktrace: Build vktrace as part of the main build

codegen: Bring in fixes from glave codegen scripts to vulkan.py vk_helper.py

Add a new target to vk_helper.py for "struct_size" so vktrace can use that.
Otherwise target "struct_wrappers" builds a bunch of files not needed by vtrace.

Add debug_report as an extension in vulkan.py. Vktrace needs this to intercept
debug_report extensions.

layers: Reenable informational messages in DrawState layer

include: Change debug report pUserData parameter to non-const

This was inconsistently defined.

tests: Fixed a few errors in Tri caught by ObjectTracker

Update ICD links in BUILD.md.

Update the ICD loader configuration links in the BUILD.md to point
to the master branch.

demos: Initialize render pass in pipeline create

windows: Fix various warnings

Categories include:
 ASSERT_TRUE(int) changed to ASSERT_TRUE(0 != int)
 Change uint32_t to DeviceSize
 Stop adding with bools
 Add some explicit casting
 Remove unreferenced variables
 Add 'f' suffix to float literals

Conflicts:
tests/vkrenderframework.cpp

demos: Fix trivial bugs in tri

Update stale info in BUILD.md.

loader: Change layer manifest file location env var to VK_LAYER_PATH

bug #14394

v148: Bug 14368 - Minor fixes for VkPhysicalDeviceFeatures

v147: Bugs 14352 and 14300

v146: Bug 14290 - Move typedefs from vk_platform.h to vulkan.h, make pSampleMask an array

v145: Bug 14160 - Remove VK_DEVICE_CREATE_VALIDATION_BIT and unused flags field

v144: Bug 14292 - Rename VkRenderPassBeginInfo::attachmentCount to clearValueCount

v143: Bug 14277 - Follow pointer naming convention in VkSubpassDescription

v142: Bug 14275 - Remove image layouts in VkFramebufferCreateInfo

v140: Bug 14251 - Inconsistencies in the definition of queries

v139: Bug 14257 - Remove pCount parameter from vkAllocDescriptorSets

layers: Fixed formatting error in DrawState

layers: Fix freed memory use in layer_destroy_msg_callback

Defer free in layer_destroy_msg_callback until after last use of data.

layers: DrawState validation cases for BeginCommandBuffer

Framebuffer and Renderpass must be NULL for PRIMARY CBs, non-NULL
for SECONDARY command buffers. Test cases also added.

loader: Remove extension origin and use Vulkan ext prop structure

Origin was set but unused.  This allowed tghe loader_extension_property
structure to be reduced down to the VkExtensionProperties.

loader: Make sure layer list from manifest file is all cleaned up properly

loader: Move long DEBUG_REPORT comments into debug report source code

loader: Add accessor functions to add/delete from layer property lists.

Use these to handle building and destroying instance and device layer property
lists.

loader: Convert layer versions from manifest file to Vulkan int version

loader: Allow null string for CMAKE_INSTALL_PREFIX to prevent extra paths

NULL string was making loader search extra local paths such as /share/...
Also expand log buffer .

Initialize VkDesciptorSet to VK_NULL_HANDLE

Compare the handle object to VK_NULL_HANDLE, not its internal integer value

layers: Validate renderPass.

cmake: Set CMAKE_INSTALL_PREFIX to "" so loader will be built with sys paths

CMake seems to default to non-null but for system SDK builds want this to
be empty string.