loader: set DEFAULT_VK_{DRIVER,LAYER}S_PATH to "" on Linux

When loader_get_fullpath() gets an empty string as the 'dir' parameter,
it immediately writes out the original, unexpanded filename. Setting
the default drivers and layers paths to an empty string on Linux will
allow the unmodified filenames to be passed to dlopen(3) without having
to maintain a separate code path for Windows.

Now that there is no longer a default path for drivers or layers on
Linux, ${CMAKE_INSTALL_LIBDIR} no longer needs to be baked into the
loader binary. Note that any ICDs or layers that are currently getting
installed to the previous default paths (e.g. /usr/lib/vulkan/icd or
/usr/lib/vulkanlayer) will not be discoverable when specified as plain
filenames, unless these directories happen to be configured in the
loader's search path on the system in question.

loader: add helper for combining paths

Add a helper function to combine an arbitrary number of path elements,
separating them with the platform-specific directory separator, and
use it in places where path combining is currently achieved through
direct string manipulation.

As an additional cleanup, update loader_get_fullpath() to duplicate
the search path string internally, rather than relying on the caller
to do so. Since loader_get_fullpath() and loader_expand_path() no
longer modify input strings, and loader_expand_path() no longer
requires that the caller append the DIRECTORY_SYMBOL ahead of time,
acrobatics to duplicate strings to prevent modification and/or to add
directory separators are no longer necessary.

This change will result a behavior change on Windows: the existing
logic for loading an ICD that is specified as a bare filename, and
not as an absolute or relative path, is to search for a driver with
the given ICD name in the DEFAULT_VK_DRIVERS_PATH search path on
Linux, and to use the given ICD name as-is on Windows. This could
lead to DLL injection attack on Windows. Merge the Linux/Windows
behavior to limit the search for ICD filenames to the default search
path on Windows as well, which is also more consistent with the
behavior described in the loader spec.

Conflicts:
loader/loader.c

demos: fclose in cube and tri

Balance every fopen with fclose.

demos: windows fixes for cube and tri resize

Defer resize until after first prepare.
Record new window size from WM_SIZE.
Leave demo->swapchain.handle valid to pass to CreateSwapchainKHR.

Layers: MemTracker check vkCmdCopyQueryPoolResults

MemTracker check of buffer memory and usage flags in vkCmdCopyQueryPoolResults

demos: add occlusion features to vulkaninfo

Add reporting of VkPhysicalDeviceFeatures occlusionQueryNonConservative to vulkaninfo.

use conditional compilation based on _WIN32 instead of WIN32 and __linux__

demos/cube: Provide a non-null VertexInputStateCreateInfo

In the spec for VkGraphicsPipelineCreateInfo, some structs are explicitly
listed as being able to be NULL if they are not used.
VertexInputStateCreateInfo is not one of them.

demos/cube: Explicitly specify std140 layout on UBOs

Without this, glslang won't actually provide the offsets and strides
required by SPIR-V.

layers: Validate buffer view usage flags

layers: Validate number of layers in subresource.

Reviewed by Tobin.

tests: Added CopyDescriptorUpdateErrors test to verify 3 errors during descriptorset copy updates

layers: Add DrawState validation for copy descriptor updates

layers: Simplify descriptor validation in DrawState

Many descriptor validation helper functions were returning skipCall based on the potential for a bad update structure type. However, at the beginning of update validation, we validate that the update type is ok. Therefore, the other helper functions don't need to continually repeat this check, and can be simplified to return relevant values directly.

layers: Improve DrawState write descriptor update

Validate that stageFlags are the same for a single write update.
Validate that all sampler updates from a single write update are either immutable or non-immutable.
Refactor update contents check to have a loop per switch case instead of switching for each loop iteration.
Added two new validation errors:

DRAWSTATE_DESCRIPTOR_STAGEFLAGS_MISMATCH
DRAWSTATE_INCONSISTENT_IMMUTABLE_SAMPLER_UPDATE

layers: Remove DrawState g_lastBoundDescriptorSet variable - it's no longer used

demos: Clean up tri depth

We animated tri to make it clear it was rendering correctly based
on ISV feedback. It looks better, but still questionable.

This change makes the depth cutoff a straight line by giving the top
two vertices the same depth.

draw_state: Fix enum in validate_draw_state_flags

DRAWSTATE_STENCIL_NOT_BOUND -> DRAWSTATE_SCISSOR_NOT_BOUND

windows: Fix vector resizing in draw_state

demos: print new device limits in vulkaninfo

misc: Remove dynamic array definititions

Windows doesn't like dynamicly sized arrays. Replace
with new'd array of objects or object pointers instead.

misc: Add intrin.h to support ffs utility

misc: Windows does not like named structure initialization

Bug 14621: Change vkGet* functions to not return VkResult

Missed one function in prior patch.

tri/cube: Adjust demo_resize() to API changes.

tri/cube: Add WM_SIZE to Win32 event handler to resize a window.

cube: Removed some #if 0's that had bee left in the code.

tri/cube: Handle window resizes (from WSI or XCB).

If either vkAcquireNextImageKHR() or vkQueuePresentKHR() return
VK_ERROR_OUT_OF_DATE, a new demo_resize() function is called, that replaces the
current state, command buffers, swapchain, etc, with new ones that reflect the
new size of the window.  As part of this, the app picks up the window size from
vkGetSurfacePropertiesKHR().

The same occurs if the application receives an XCB_CONFIGURE_NOTIFY event, with
a changed window size.

layers: Improve DrawState Descriptor validation

Adding further validation for the contents of vkUpdateDescriptorSets() call.
Make sure that provided sampler/image/buffer data is consistent with type and layout of the update.

For imageView updates, make sure that aspect bits are set correctly for Color and DS images and make sure that the imageLayout is compatible with the underlying image format.

This results in 4 new validation checks:
DRAWSTATE_SAMPLER_DESCRIPTOR_ERROR
DRAWSTATE_IMAGEVIEW_DESCRIPTOR_ERROR
DRAWSTATE_BUFFERVIEW_DESCRIPTOR_ERROR
DRAWSTATE_BUFFERINFO_DESCRIPTOR_ERROR

Each of the new checks has a basic check to verify that the underlying Vk object (sampler, imageview, bufferview, buffer) for the respective type of check is valid. The ImageView error also includes cases where format of the image is not consistent with the layout (Color vs DS).

Also added some enhancements to existing check:
DRAWSTATE_INVALID_IMAGE_ASPECT - verify D&S bits not both set, but that one is for a DS layout, verify COLOR bit set for Color layout

This change included adding and/or enhancing the tracking in DrawState for image[View] and buffer[View].
There is documentation for each of the new checks.
There are new tests added for each of the new error cases as well as the ASPECT error case.

layers: Modify Image layer to store IMAGE_STATE structs instead of their unique_ptrs

loader: Check for NULL ICD in CreateDevice

Related to lunargxchange #163

misc: Bump to header version 183

draw_state: Remove obsolete check

bug-14827: Remove DescriptorInfo union

The DescriptorInfo union is invalid in C++

bug-14258: Remove maxCommandBuffers

commit #53c2f7e

layers: Update ObjectTracker to handle double-embedded loops of array struct params w/ object arrays in them

ObjectTracker still only goes 1-level deep on object validation so if an array struct param has an embedded struct that then has an object, validating those objects is not yet done by ObjectTracker.

layers: Fix objectTracker to validate DSs in CmdBindDescriptorSets()

bug-14258: Make descriptor pools and command pools more consistent

misc: bump to header version 182

bug-14575: normalized queue priorities

Bug 14789: Clean up limit structure

layers: Fix windows build error

Image layer had a list initialzation inside a member initializer list

layers: Add state-dependent image aspect/format checks

misc: Update header to v181

bug-14827: Make DescriptorInfo a union of bufferInfo, imageInfo, and bufferView

bug-14786: CmdBufferOptimizeFlags -> CmdBufferUsageFlags

bug-14786: Addressing layered subresources in vkCmdCopyImage

commit: ad27e2b Attempt at naming consistency

bug-14786: minor renames

misc: Change clear rectangle to include array layer/count

commit: 53a511d

bug-14542: Remove VK_UNSUPPORTED from success return codes

WIP bug-14815: VkShaderStage

WIP bug-14815: VkImageAspect

misc: Bump header version to v180

misc: change is unneeded?

bug-14746: add batched submit for semaphores and command buffers

misc: match upstream header v179

Bug 14621: Change vkGet* functions to not return VkResult

bug-14580: add dataSize to vkGetPipelineCacheData

commit: aaa2ae0

bug-14809: Need VK_ERROR_FEATURE_NOT_PRESENT

misc: make enum boundary names consistent

bug-14808: report number of valid bits in timesamps

misc: upstream header v176

misc: Remove WRITE_COMBINED

misc: Fix compile warnings about undefined _MSC_VER

commit: a500fc4

bug-14737: Move alphaToCoverageEnable to MultisampleStateCreateInfo

bug-14646: remove VK_SUBPASS_DESCRIPTION_NO_OVERDRAW_BIT

commit: 854f367

misc: bump header version 174

bug-14084: Replace abbreviations in VkQueryPipelineStatisticFlags

bug-14531: Expand tess abbreviation to tessellation

misc: Bump header to align with upstream

bug-14642: Rename queueRecordCount to requestedQueueCount

misc: Bump header to align with upstream

bug-14291: combine color and depth/stencil clears

misc: Rename depthClip to depthClamp

Implementations will need to invert their behavior.
commit: 60a6646e24e58b1e2c44ea8eeb1017fc133593a1

demos: Check all return values

Return empty string instead of null

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