loader: Fix heap_realloc function to return newly allocated memory

LunarXchange issue #162

layers: Remove mid() & mdd() functions from DrawState

Just use the report_data directly off the already looked-up map entry for the given device/instance. This saves a map lookup.

layers: Add DrawState check for DescriptorSet availability in Pool

Prior to allocating descriptors from a Pool, verify that requested number/type of descriptors are available in that Pool, flag error if not.
When Pool is reset or when descriptors are allocated, release descriptors back to pool.

Added constructor and destructor for POOL_NODE to simplify allocation/cleanup.

Added layer documentation for new check and AllocDescriptorFromEmptyPool test to verify that error is correctly flagged.

layers: DrawState layout tracking improvements

Descriptor type per layout tracked via vector to simplify allocation/cleanup.
Added tracking for pipeline layout that can be used for further validation checks

layers: Added TODO comments to DrawState for missing validation checks

layers: Fix indices in DrawState error message

If imageView aspect for DescriptorSet Update has both Depth and Stencil set, top-level "WriteDescriptorSet" is governed by "i" and specific descriptor within that particular WriteDescriptorSet is governed by "j".

layers: Comment in DrawState for missing validation of DescriptorPool size limit

demos: Add mem barrier to change layout to present

This addresses LunarExchange issue 168

demos: check return value from vkQueueWaitSemaphore

demos: Handle paint events and call RedrawWindow to animate

This keeps MessageBoxs from validation on Windows from hanging the
demos, while allowing the demo to animate

Fix "_out64" -> "build" references accidentally removed

Fix formatting in BUILD.md

BUILD.md: changed "_out64" to "build"

demos: fix tabs and line endings in vulkaninfo

Demos: output api version in vulkaninfo

Clarify parameter usage in memory_type_from_properties

layers: LX161, validate attachment references in subpasses

For CmdClearColorAttachment, attachment ref must be in attachment
reference array of current subpass.

layers: LX160, Validate Descriptor Set Image Aspects

Validated that Descriptor Sets do not have both the STENCIL and
DEPTH aspect bits set in their imageViews.

loader: Clean up loader_platform file

Add subpass dependency validation.

layers: LX159, properly validate swapchain images

Swapchain images are implicitly created and deleted, but must
be considered valid VkImage objects.

layers: LX159, validate all descriptorSets in array

DrawState was not using array index in dsUpdate.

layers: LX159, pipelineCache CAN be Null.

Fix for ObjectTracker to not error on NULL pipelineCache objects.

layers: Remove MemTracker dead code

cmake: Need glslang for tests as well as ICD

cmake: Fix options for building on Windows

cmake: Make all sub-components optional

loader: Use loc instead of location for getting the directory

Without this, the VK_ICD_FILENAMES parameter ends up getting ignored

Updaing build to include vk_layer_utils

update_external_sources: Get Windows LunarGLASS build working again

Include the following changes:
- Enable SPIR-V rev 32 support
- Always delete previous LunarGLASS builds, just like glslang
- Disable Debug LunarGLASS build
- Change order of glslang build so that Release resides in install

layers: Bug #14850, enforce validation for semaphore states

Added creation of driver-side semaphore objects.

Fix image aspect value passed to demo_set_image_layout

Addresses LunarExchange issue #156

layers: Remove dependency between calls to vkCreateImage and vkGetPhysicalDeviceProperties

demos: Create a wider console on windows

Loader: Fix locking bug in CreateInstance

update_external_sources: move up to latest glslang

Also remove a few redundant redundancies.

loader: Remove unneeded defines

loader: Use correct 64bit printf format for Windows

Fixes LunarXchange issue #145

layers: Fix Windows compiler warnings

layers: Fix Windows compile warnings

layers: clean up includes

layers: Add DEBUG_OUTPUT support to swapchain

layers: Add DEBUG_OUTPUT support to shader_checker

layers: Add DEBUG_OUTPUT support to param_checker

layers: Add DEBUG_OUTPUT support to mem_tracker

layers: Add DEBUG_OUTPUT support to image

layers: Add DEBUG_OUTPUT support to draw_state

layers: Add DEBUG_OUTPUT support to device_limits

layers: Add support for Windows DEBUG_OUTPUT logging

demos: Add getting GetDeviceProcAddr entrypoint from GetInstanceProcAddr

Better test for the loader.

loader: Support GetInsTanceProcAddr(inst, "vkGetDeviceProcAddr")

layers: Suppress error messages for undestroyed descriptorSets

DescriptorSets are not explicitly destroyed. ObjectTracker will need to be
extended to handle these types of cases.

demos: Remove swapchain workaround code and don't assume 2 buffers

Addresses Lunar Exchange issue 153

demos: Change WSI references to swapchain

Addresses Lunar Exchange issue 147

demos: Animate tri via z-clipping

To address Lunar Exchange issue 146.  We still test/demo z-clipping while
displaying all of the triangle, at least some of the time.

layers: Add DrawState check to flag error on vkFreeDescriptorSets from ONE_SHOT Pool

Also added a layer validation test to flag this error

demos/tests: Stop calling vkFreeDescriptorSets on pools with usage ONE_SHOT

WSI Validation: Ported better logging approach from generic layer.

WSI Validation: Move some things between .h and .cpp files.

This will more-closely mirror what's being done in the generic layer, which was
the original source of this layer.

intel,layers,tests: move cmd_meta errors to validation

Add checks for alignment in DeviceLimits layer.
Add checks for type, format and sample count in Images layer.
Remove such checks from intel icd.
Add new validations to layer_validation_tests.

Only check LogicOp if it is enabled.

layers: Migrate DrawState global data structs to layer_data struct

DrawState maps are all now stored per-device and per-instance.
Also fixed a bug where renderPassMap ptrs were not being cleaned up.

layers: DrawState migrate device and instance dispatch tables into layer data struct

This is first part of cleaning up DrawState. Need to migrate its global maps into the layer data struct as well so they are per device/instance and then single map look-up with get both layer data and dispatch tables.

layers: In DeviceLimits move dispatch table to layer data struct

All instance dispatch calls are made via instance_dispatch_table in layer data struct.
All device dispatch calls are made via device_dispatch_table in the layer data struct.
This saves map look-ups as a single call can be made to get the per device/instance layer data, including the dispatch table.

update_external_sources: Fix svn LunarGLASS checkout for Linux

Don't checkout HEAD or rolling back one commit will delete files.

misc: do not need pull if using git checkout

Also remove unused R32 variable and file now that
we can get latest glslang from git.

layers: Remove obsolete dbg message logging header

layers: Add support for debug_report extension in generic layer

Now Generic layer is better example of a validation layer

loader: Fix comile warning on Window

loader, icd: correct use of _aligned_malloc

Memory allocated with _aligned_malloc on WIN32 should be released with _aligned_free.
If icd/common/icd-instance is ever used on WIN32 it will now use _aligned_free.

loader_aligned_heap_alloc and loader_aligned_alloc would require additional
matching loader_aligned*_free functions to be correct for WIN32.
But both functions are never used.  Just remove those functions for now.

layers: Remove dead code from MemTracker

layers: Fix DeviceLimits error return codes

layers: Add validation for Image Aspect settings

Image layer gets validation for image aspect settings laid out in the spec.

loader: match free to alloc in dirent functions

closedir was using plain free instead of loader_tls_heap_free.

layers: LunarXchange #145: Fix windows printf format errors

Windows hates %zu. Changed to %lu and cast size_t to unsigned long.

loader: Fix GetInstanceProcAddr for CreateDevice

Demos: increase console buffer size for vulkaninfo

loader: Add warning message for unreconized layer names from env var

update_external_sources: get glslang from gitlab

This is the new way to get the tools for private SPIR-V rev32

update_external_sources: get glslang from gitlab

This is the new way to get SPIR-V rev32 source.

Only check blend state if blend is enabled.

Add basic validation for the shader module.

layers: Add DrawState checks for dynamically set Viewport and Scissors

Make sure that counts from dynamically set viewport and/or scissors match the PSO counts.
Added tests to verify these cases.

layers: Drawstate verify viewport and scissor state at PSO creation time

Make sure that viewportCount and scissorCount match
If viewport is not dynamic, make sure viewport state is supplied and if viewportCount is non-zero, that viewport data is non-NULL.
If scissor is not dynamic, make sure scissor state is supplied and if scissorCount is non-zero, that scissor data is non-NULL.

Added tests to hit these new cases.

loader: Fixes in trampoline code to support layers which wrap objects

Loader trampoline code may see wrapped objects.  Don't do value comparisons
of dispatchable objects but instead compare dispatch tables to find objects.
PhysicalDevice objects where may have multiple gpus with same instance dispatch
will be fixed in a later patch.

layers: Validation -- updated layer validation doc file

Corrected validation layer error code.

loader: fix use of freed memory

vkEnumerateInstanceExtensionProperties was using freed memory.
Avoid free of NULL pointers.

layers: Updated README.md and details doc

README.md was woefully out of date so this is a big improvement, but still more work to be done. May want to merge this doc with details doc in the future as they cover similar ground.

Build doc: Correction to the cmake command to be used for windows

vulkaninfo demo: Reduce the size of the console window so it fits on smaller screens

sdk: Update release notes for 0.9 SDK

loader: Fix vkGetInstanceProcAddr to handle debug_report extension

Need loader entrypoints for debug_report extension including the utility
functions. Don't call down the instance chain GPA for this extension.

Remove instance extensions decoding when GPA instance == NULL as don't want
to return extension entrypoints unless they are enabled.
This meant the WSI swapchain instance GPA was no longer used so remove it.

layers: Validate draw commands appropriately inside/outside render pass

Updated DrawState layer and validation tests to check that all draw commands
are correctly called inside or outside (or either) of an active render pass.

layers: Fix DrawState to correctly handle dynamic state from PSO and CmdSet* calls

Track which dynamic states are set via PSO and merge that with dynamic state set by CmdSet* cmds.

Updated viewport not set test to pass when scissor is flagged as missing. Currently scissor and viewport set together so scissor error masks viewport error.

layers: DrawState allows callback to set fail on DescriptorType mismatch

If the update DescriptorType does not match the layout DescriptorType, DrawState would previously bail automatically. This update flags the error and will only bail if the callback return code is VK_TRUE, like with all other validation errors.

Merge branch 'mlentine_layers4' into 'master'

Make reseting an unsignaled fence a warning.

See merge request !38

Set dynamic pipeline state as dynamic

Make reseting an unsignaled fence a warning.

loader: Change GetXXProcAddr to support NULL dispatchable object

Also change GetInstanceProcAddr to return the first entry down the chain rather
than global (trampoline entrys) all the time when a non-null instance is passed.

layers: DrawState handles dynamic state set via PSO

For dynamic states flagged in PSO, consider them set at draw time.
Clarify various dynamic state error messages to make them more consistent with updated dynamic state API.
Add test for scissor not bound.