shader_checker: Also validate vkCreateGraphicsPipelineDerivative

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

shader_checker: Extract pipeline validation into own function

There's about to be a second caller.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Courtney Goeltzenleuchter <courtney@lunarg.com>

Windows: Add vkUpdateDescriptorSets into exports for vulkan.dll

Fix a bunch of mismatched new[]/delete.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

shader_checker: remove stray debug output from typechecker implementation

shader_checker: validate VS inputs against VI even when no VI

Now we'll treat the lack of a VI stage as the same as a VI stage
which provides no attribs and no bindings.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Tobin Ehlis <tobin@lunarg.com>

shader_checker: Only continue with pipeline config if no errors

The driver may crash in these cases.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>
Reviewed-by: Tobin Ehlis <tobin@lunarg.com>

shader_checker: current object & locking

Fix some issues with cross-thread access to the shader map, etc.
Also introduce a 'current object' as a hack to be able to support
the debug callback, which currently needs a dispatch table while
only having a VkInstance.

shader_checker: Add support for delivering layer messages via callback

This is just a direct copy of what all the other layers are doing.
Allows the layer validation tests to capture errors.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

ShaderChecker: do not crash if Shader output count == 0 and PSO attachments count > 0

(cherry picked from commit 80bb5d63774f2e11bc14d9c5291bb3273f0e79cf)

tests: Cmake changes for building on Windows

layers: Change MemTracker maps to contain structs

Remove one unnecessary level of indirection.

Conflicts:
layers/mem_tracker.cpp

layers: Tuning MemTracker

Reverse sense of fenceMap using VkFence instead of fenceID as key.
 This eliminates iterating over all of fenceMap.
Stop creating internal fences and let vkQueueSubmit pass NULL fence.
Track lastSubmittedFence and lastSubmittedQueue in command buffer info.
Use unordered_map instead of map for better performance.
Reuse iterator returned by map find whenever available.
 It is faster than repeating a find operation with "[]" notation.
Remove map entries at time of VkDestroy instead of other calls such as Wait.

Layers: fix MemTracker threading crash

updateFenceTracking was using iterator for deleted item.

layers: thread safe MemTracker

Remove unneeded getFenceFromId() that crashed outside of mutex.

layers: fast return of inactive MemTracker printing

v98: replace vkUpdateDescriptors() by vkUpdateDescriptorSets()

Only slightly tested.

Conflicts:
include/vulkan.h

v98: remove vk{Begin,End}DescriptorPoolUpdate()

Assume VK_DESCRIPTOR_UPDATE_MODE_FASTEST.

Conflicts:
icd/intel/desc.c
include/vulkan.h

v98: rename count to arraySize in VkDescriptorSetLayoutBinding

Conflicts:
include/vulkan.h

vulkan.h: V97 -- Remove multiple allocations. Bug# 13948.

Remove multiple memory allocation requirements from API, and supporting
changes in driver, demos, layers, and tests.

layers: Improved DrawState Descriptor Update validation

layers: Add some early returns in DrawState fail cases to prevent segFs in driver

layers: Fix drawState lock bug and don't call driver w/ bad pipeline

tests: Added some empty test functions with comments for DrawState layer testing

layers: Fix some DrawState checks

BUILD.md: Fix markdown language that didn't translate from github to gitlab

vulkan.h: V96 -- xglCmdBlitImage filter mode. Bug# 13759.

layers: Add additional check for RenderPass sampleCount

Was checking PSO MSAA num samples against FrameBuffer sampleCount. This change adds an additional check against RenderPass sampleCount.

layers: Build C++ version of generic layer

All layers are now implemented in C++.

layers: Convert object tracker to C++

wsi: Deal with drivers that don't allow query of swap-chain VkFormat.

wsi: Deal with drivers that don't allow query of swap-chain VkFormat.

wsi: Add queries for VkDisplayWSI's and VkFormat's to demos.

Show how to query which VkFormat to use for a WSI swap chain.  In order to work
correctly, changes are needed to existing ICDs:

- Changed the order of VkFormats for the intel sample driver, so that the
  desired value will be first (which is what the demos will use).

- Enhanced the nulldrv to support the
  VK_PHYSICAL_DEVICE_INFO_TYPE_DISPLAY_PROPERTIES_WSI query.

Add win32 condition variables.

layers: Make threading layer provide thread-safety

Add mutual exclusion behavior to threading layer after reporting collisions.
This will allow applications to continue beyond first threading error.
It can't help reentrant calls.

tests: Expand layer validation test coverage

Added layer validation tests for several ObjectTracker and
MemTracker validation cases. Also improved error handling
in object tracker layer to handle validation failures that
could cause unrecoverable driver errors.

Merge branch 'master' of
https://gitlab.khronos.org/vulkan/LoaderAndTools.git

Revert "demos: Add msaa-tri.c for msaa rendering example modified from tri.c"

This reverts commit 0e83c4d651d5602a35b5c54eb59f2543310b5f31.

demos: Add msaa-tri.c for msaa rendering example modified from tri.c

loader: On Windows disable global optimization for API entrypoints

Without this change, on a Windows Release build, mhook is unable to hook the entrypoint functions.

layers: rework Threading layer to match version 91

Handle multiple thread checked parameters.
Thread-check parameters in some calls that start with a VkDevice parameter.
Use more recent WSI name pattern.

layers: fix calculation of intercepting functions.

Assign _gen_layer_get_global_extension_info to intercept.
Remove duplicate calculation of lookups.

vulkan.h: V93 -- fine-grained memory binding synchronization. Bug# 13464

New names and behavior for QueueBind* APIs.

shader_checker: check types between FS output and CB

shader_checker: add type checking between VF and VS

shader_checker: add helper to walk a type tree and return the basic type

shader_checker: fix fs output/cb attachment walking after assert fixes

layers: remove assumption that there's a glslang source tree alongside

We have an in-tree spirv.h now, just use that.

Windows: Fix build from recent loader changes

spirv: Update to matching header

vulkan.h: V94 -- remove vkCmdCloneImageData.  Bug #16550.

layers: Remove use_count from ObjectTracker

Removed ll_increment_use_count from layer
Fixed some issues with thread safety
Simplified handling of special-case APIs
Removed GetExtensionSupport dead code

misc: Add vkGetInstanceProcAddr() entrypoint

Adding this entrypoint as this has been discussed in khronos  and is needed
for the loader/layer/extension proposal.

Caveats:
1) Have not updated layers or any tests/demos yet that will come later;
2) No one including loader is using this call yet;
3) Leaving exisitng vkGetProcAddr() as is for now; later once loader
is using vkGetInstanceProcAddr call can switch vkGetProcAddr() to
vkGetDeviceProcAddr()

loader: No longer generate code, instead directly check in source code

layers: Fix bug of GetGlobalExtensionInfo not being added to layer_intercept

loader: logging on Windows to both debug output and stderr

layers: update object tracker memory ref validation

Allocate enough space for queueInfo.
Record queueNodeIndex to allow setGpuQueueInfoState between addQueueInfo and validateQueueFlags.
Report case when queue flags remain unknown to application and layer.
Move expanded common code into validateQueueFlags.
Initialize g_pQueueInfo to NULL. Handle case when it is NULL.
Handle when malloc or realloc fails.

Merge branch 'master' of https://gitlab.khronos.org/vulkan/LoaderAndTools.git

Removed warning in cube demo.

layers: Add object type validation to object tracker

Validate that objects match the objectType any time both are specified.

layers: Use native Vulkan object type definitions

ObjectTracker was using locally-defined object types as the vulkan
ones were unavailable. Replaced them with ones from Vulkan.h.

loader: Fix bugs where non-dispatchable objects overwrite dispatch table

Also fix the GPA for non pass through cases

loader: Fix bug in vkGetProcAddr so trampoline code is returned as needed

loader: Separate out the python generator script

Loader is a reference component. Intel driver is a sample component. Loader
should not be sharing files with driver. This is a first step in that direction.
Also loader has special cases that are not in driver or layers. So it simplifies
their scripts not to have loader baggage in them.

loader: rename a few dispatch utilties

bug 13854: Remove memory references

header: 0.93.0
svn: 30980

bug-13690: Clarification on vkFlushMappedMemory

Bug 13690 - Clarification on the behavior of vkFlushMappedMemory
svn ID: 30914 & 30888

bug-13751: Update vkCmdClearColorImage color parameter

svn ID: 30918
Change the VkCmdClearColroImage color parameter
be a pointer.

intel: support only VK_DEPTH_MODE_ZERO_TO_ONE

Fail pipeline creation when VK_DEPTH_MODE_NEGATIVE_ONE_TO_ONE is specified.
Update viewport transformation for depth range [0, 1].

Demos are updated too.

demos/cube: use GL conventions

Use GL conventions in vertex data and contents.  Tranform to VK conventions
in the vertex shader with

  gl_Position.y = -gl_Position.y;

demos/tri: fix facing and enable culling

tri: Fix command buffer use for staging texture

layers: Move OT validation calls inside mutex

ObjectTracker generated validation code was being added outside the
mutex-protected area compromising thread-safety.

layers: Add memory manager queue validation to object tracker

Validate that sparse-memory binding operations only take place on queues
with MEMMGR capability.

layers: Add another routine to the list that don't need use counts

demos: Add validation via --validate cmd line option

layers: Eliminate compiler warnings on Windows.

This includes suppressing compiler warning C4065, where VS complains if a
switch statement has a "default", but not "case" statements.  This was true in
many functions of the auto-generated "helper" header files.

This also includes removing unreferenced local variables, and dealing with some
type differences (sometimes with casting, sometimes with changing the type).

demos: Eliminate compiler warnings on Windows.

loader: Eliminate compiler warnings on Windows.

Win: Stop using the temporary "xcb_nvidia" code.

Now that we have a better WSI solution, this older code is no longer needed.

demos: Improve tri/cube error if incompatible driver found.

toolchain: Update to newer glslang and LunarGLASS versions

LUNARGLASS_REVISION=1070
GLSLANG_REVISION=30939

mem_tracker: Fix error from fixing MSVC asserts

demos: tri/cube give error if can't find WSI extension.

demos: Improve tri/cube error if incompatible driver found.

demos: Use macros for the app name, and window name.

layers: Fix msvc++ iterator issues

demos: Change app name for the console window.

draw_state: Fix another MSVC++ debug assert

draw_state: Fix Microsoft C++ iterator assert

The Microsoft Visual Studio vector library does not like this code:
delete (*ii).second->pCmds.back();

Internally, pCmds.back() does a pCmds.end() - 1 and since the
vector we are using (*ii).second->pCmds is not local, the library
throws an assert about the iterator being non-incrementable.
The assert if fixed by making a local copy of the vector and
working on that.

mem_tracker: Do not increment iterator after delete

The loop was blindly incrementing the iterator and
it should not. In the delete case the loop should
resume with the iterator returned
by the delete as it's the next element in the list.

cube: Disable rendering once we enter cleanup

Code was getting a segfault after clicking on the close
window button and a refresh is attempted as things are getting
shut down. If we use same demo->prepare flag to indicate
rendering is ready during shutdown we avoid the issue.

draw_state: Fix Microsoft C++ assert

The C++ runtime library was issuing an assert on the following line:
for (vector<CMD_NODE*>::iterator ii=pCB->pCmds.begin(); ii!=pCB->pCmds.end(); ++ii) {

It was complaining that the iterator ii was not compatible with the
pCmds.end(). Tracing down the assert, the C++ library did not like the
fact that we were accessing pCmds through a pointer (pCB). If we made a
local copy of the pCmds vector the assert went away.

Shader Checker Layer: make sure iterated list is of non-zero length

On MSVC++, need a check to make sure a list is of non-zero length
before attempting to iterate on it.

loader: Terminate chain of GetProcAddr() in driver if gpu obj is base

Fixes issue when driver doesn't implement an entrypoint of recursing
and unwrapping gpu object too many times.

layers: Fix bug in APIDump where some void* members were having their address printed when APIDumpNoAddr was TRUE

icd: Add support for one bit alpha BC1 formats

layers: Add another routine to list that don't need use counts

layers: Add 3 more routines to list that don't need use counts

layers: Fix APIDump corner cases to not print addresses w/ NoAddr option