layers: MR159, Rename get_type_def, type_def_index

These are just get_def, and def_index now to reflect the fact that they will
handle insts other than just type definitions.

windowsrtinstaller: Removed unneeded UNINSTALL reg values

Renamed "IC" UNINSTALL reg value to "InstallCount"

loader: Fix createInstance to pass pAllocator down

Also fix bug where createDevice failure removes logical device from icd
struct

loader: Don't have createInstance terminator code write the instance
object

loader: Moved missing #define needed by aligned_alloc.

The define needs to be before the includes.

layers: LX308, CreateGraphics/ComputePipelines support for multiple pipelines

Codegen needs special cases to create tracking structures when multiple pipelines
are specified. Moved these APIs into the explicit code section for now.

loader: Fixed missing #define needed by aligned_alloc.

This fixes the missing #define so Linux should build without warning.

loader: Update Windows uninstall to remove 32-bit vulkaninfo.

I recently added a 32-bit vulkaninfo to the install directory so
people could valiate the 32-bit loader.  Unfortunately, I wasn't
removing it.  Thanks to David for the catch.

loader: Removed CMake 2.8 files which broke CMake 3.x.

The CMake support files which I added as part of the FindImageMagick.cmake
commit, were not required when we rolled back the files to 2.8.  This
caused the XCB code to use the wrong version of the file on some machines
which resulted in XCB not being properly detected.

loader: Fixed issue with Linux when ImageMagick-5 installed.

Last changes made ImageMagick-6 work, but broke ImageMagick-5.
The issue is that the _ARCH_INCLUDE directories are only present
for ImageMagick-6 and newer.  So, Cmake was attempting to use an
undefined variable.

loader: Further ImageMagick detection fix

Karl discovered an issue with the last change to FindImageMagick.cmake.

FIXES: Fixed ImageMagick-6 detection with modified FindImageMagic.cmake file

The file I added was from Cmake 2.8 and didn't know how to detect ImageMagick-6 on Linux.

Fixes: Added missing newline after merge.

fixed typo

added gcc versions of aligned malloc

fixed custom allocator aligned memory allocations in tri sample.
fixed x86 compilation.

FIXES: Non-MSVC compilers on Windows were failing.

Fixes LunarXchange Issue #301.

BUILD: Modified CMake ImageMagick detection logic to work with CMake 2.8.

Modified CMake ImageMagick detection logic to work with CMake 2.8.

BUILD: Have git ignore the new build32 folder.

SAMPLES: Fixed memory leak in of presentMode data.

We were leaking the presentMode array data, now it's freed.

INSTALLER: Moved vulkaninfo 32-bit start menu link before 64-bit version.

Perform the 32-bit vulkaninfo start menu link creation prior to the 64-bit so that the 64-bit one appears by default.

BUILD: Fixed more compilation warnings on 32-bit Windows builds.

Fixed more compilation warnings on 32-bit Windows builds.

LOADER: Add 32-bit vulkaninfo link to start menu.

Add 32-bit vulkaninfo link to start menu.

Loader: Fix 32-bit Cmake detection of ImageMagick

Older versions of Cmake didn't like the newer script.

DOCS: Updated LoaderAndTools doc to indicate updated ImageMagick requirements.

Updated docs to indicate that both 32-bit and 64-bit installs of ImageMagick are required.

Installer: Clean up powershell header comment.

Clean up the ConfigLayersAndVulkanDll header comment.

Installer: 32-bit installer changes to get working on 64-bit system.

These changes allow the installation of 32-bit items onto a 64-bit system.

Installer: Added changes to detect 32-bit layers on 64-bit system.

Added changes to detect 32-bit layers on 64-bit system.

Installer: Minor cleanup

Modified size and added a clearer comment.

Installer: Changes for 32-bit installer

Installer/scripts should now support 32-bit OS.

Installer: More work towards 32-bit installer for 32-bit OS

Removed unnecessary code in update_external_sources batch, and added initial 32-bit path in installer.

Installer: Add support to install 32-bit vulkaninfo

Add support to install 32-bit vulkaninfo

LOADER/INSTALLER: Add support for 32-bit install in Win64 installer

Added ability to install 32-bit and 64-bit Loader into the Win64 installer.

Win32: Get 32-bit Windows build working
Also includes changes to allow simultaneous 32-bit and 64-bit Windows builds.

layers:  LX306, Fix NPE in draw_state

Fixed a case where getCBNode returned NULL but cleanInFlightcmdBuffer
used the result.

loader: Disable using CreateInstance allocation callbacks

Workaround for cts to work.

Added one more license to list of licenses

demos: Select queue family based on both graphics and present support

LAYERS: More 32-bit fixes caused by recent commits.

Had to fix more 32-bit build issues caused by recent commits.

Win32: Get 32-bit Windows build working
Also includes changes to allow simultaneous 32-bit and 64-bit Windows builds.

BUILD: Fix build of draw_state.cpp with VS 2015.

Added /bigobj to Debug build of all layer files when using VS 2015 or newer.

BUILD: Moved 64-bit source dir back to build from build64.

Moved 64-bit source dir back to build from build64 to be backwards compatible.

Win32: Get 32-bit Windows build working
Also includes changes to allow simultaneous 32-bit and 64-bit Windows builds.

BuildFixes: Hard-code Win64 in Visual Studio detect script

We don't officially support Win32 yet, so hard-code Win64 to protect the build.

BuildFixes: Fixed update_external_sources batch file for Windows build.

Code to detect Visual Studio version was encapsulated in a local section.

layers: LX300, Conditionally validate pInheritanceInfo in object_tracker

In BeginCommandBuffers, pInheritance may be non-NULL but should only be
validated if it is part of a secondary command buffer. Added status flag
for this setting in AllocateCommandBuffers and added code-gen for this
special case to the layer generation code and made the routine
(dereference_conditionally) general-purpose.

layers: LX295, fix draw_state vkCreateDescriptorPool range sizes

Was set to VK_DESCRIPTOR_TYPE_END_RANGE, should have been
VK_DESCRIPTOR_TYPE_RANGE_SIZE

shader compiler: map VertexIndex to VertexID

This is not a perfect implementation of VertexIndex. It assumes a base of zero.
Also pickup latest LunarGLASS that (truly) supports VertexIndex
Also re-enable ir_validate for undeclared variables

layers: Only set up tracking for new descriptors if allocation succeeded.

It's possible for this call to fail -- the pool is exhausted, or the request was simply bogus. In these cases, the returned descriptor set handles are meaningless. Don't shove them in our hashtables.

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

nulldrv: Set memory type mask for buffers.

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

nulldrv: fill out physical device memory info

Previously we just left this filled with junk.

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

layers: Fix draw_state validation of descriptor updates with images

Earlier fix was broken b/c we didn't have format of swapchain images.

This fix adds the format to imageLayoutMap IMAGE_NODE struct, which contains all images created via vkCreateImage() as well as all swapchain images fetched via vkGetSwapchainImagesKHR().

This required enhancing the SWAPCHAIN_NODE to include the createInfo. Since this has a dangling ptr for queueIndices, added a constructor and destructor to that struct to handle allocation and clean-up of that ptr.

misc: bump to header 1.0.2

layers: MR153, Correctly track inFlight cmdBuffers per queue

There were per-queue vectors of the cmdBuffers in flight, but they were not
being consistently used/updated.
This change keeps two sets of inFlight cmdBuffers: one globally for all
cmdBuffers on the device, and one for each queue.
Ideally we could just track per-queue, but secondary command buffers are an
exception that's considered "in-flight" from the time they're recorded in a
primary command buffer with the vkCmdExecuteCommands() call so having a global
list provides a way to account for that.
Command buffers are added into both the global inFlight set and the individual
queue set at QueueSubmit time.
When cleaning up command buffers based on fences or Idle waits, correctly
remove the cmdBuffers from the appropriate queue inFlight set. Also, if the
cmdBuffer is not inFlight on any other queues, remove it from the the global
inFlight set.
Removed the deviceMap as it was only being used to hold a vector of queues.
Since layer_data struct is already per-device, just moved vector of queues
directly into layer_data struct.

layers: MR150, Fix render pass graphics checks

Make sure that VK_SUBPASS_EXTERNAL is handled properly.

layers: MR139, draw_state VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT checks

Added a set to track which secondary cmdBuffers are contained within a primary cmdBuffer.
Added new DRAWSTATE_INVALID_CB_SIMULTANEOUS_USE enum value for new error/warning
cases, along with line in documentation.
New validation issues and state updates are listed below.

While tracking cmdBuffers at QueueSubmit time:
 1. Flag an error if a cmdBuffer is already in flight and does not have
    VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT set
 2. Account for tracking of secondary cmdBuffers (and their in_use resources)
    that are submitted within a primary command buffer

When submitting secondary command buffers into primary command buffer at
    CmdExecuteCommands time:
 1. Flag an error if secondary cmdBuffer is in flight and doesn't have
 VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT set.
 2. Warn user if secondary cmdBuffer doesn't have
 VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT set but primary cmdBuffer does.
 This causes primary cmdBuffer to be treated as if
 VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT isn't set.
 3. Add secondary cmdBuffers to set off of primary cmdBuffer for tracking purposes
 4. Add secondary cmdBuffers to inFlight set

layers: MR137, Flag error if set updated or freed while in use

It is illegal to free or update a descriptorSet that is in use.
Updated SET_NODE to inherit from BASE_NODE for in_use tracking.
At the time that Draws are recorded into the cmdBuffer, capture any active sets
for that cmdBuffer into std::set<VkDescriptorSet> activeSets.
At the time vkCmdBindDescriptorSets is recoreded in cmdBuffer, flag descriptor
sets as in use. At the time a set is freed, flag an error if set is in use.

glslang: bump up glslang and spirv-tools versions for ISV release

shader compiler: add new VertexIndex

Make cJSON case-sensitive

debug_report: Integrate review feedback

Conflicts:
include/vulkan/vk_ext_debug_report.h

Conflicts:
include/vulkan/vk_ext_debug_report.h

layers: Fix screenshot to intercept CreateInstance so it canbe on instance chain

vktrace: Get partially working again with the new loader/layer interface

layers: Remove obsolete VkbaseLayerObject

loader: remove resolved TODOs

loader: Remove CreateInstance and CreateDevice

The new init mechanism always uses GetInstanceProcAddr
to query the next layer's CreateInstance or CreateDevice
function. Thus no reason to cache those function pointers
in the instance dispatch table.

loader: Use ICD's GetInstanceProcAddr

Appears there is some difference in behavior between
vkCreateDevice addr that we get initially vs. one
later. Not sure what difference is yet.

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

loader: Add util to strip loader extensions

loader: implement new layer init method

New layer init method requires the construction of
Link information for CreateInstance and CreateDevice
that is accessible to layers via the CreateInfo.pNext pointer.
The layer can then use the Get*ProcAddr from the Link
structure to initialize their dispatch table if the
call down the chain returns successfully.
This removes the need to do special initialization work
at Get*ProcAddr time.
Layer Get*ProcAddr now return their internal function
pointers regardless of the value of instance or device.
Only need to have valid instance & device when looking
up extensions or when passing the request down the chain.
This mechanism allows us to remove object wrapping used
by the loader previously. Also simplifies the dispatch table
setup.

Conflicts:
layers/device_limits.cpp
layers/draw_state.cpp
loader/loader.c
loader/trampoline.c

layers: fix threading destroy instance

The threading layer was using the generic destroy_instance
utility when it should be using the utility that takes
a map.

layers: Add utility functions for new layer init

These utilities will find the appropriate CreateInfo
extension that defines the layer link information.

layers: Update layer info to match json

loader: Add structures to support new init method

These structures will be used to extend CreateInstance
and CreateDevice to support layers. The loader will
create a chain of these structures - one per layer -
that tells the layer the Get*ProcAddr for the next
lower part.

icd: Remove unused reference to vk_sdk_platform.h

vkjson: initial commit

layers: MR148/GL82, Don't memset PIPELINE_NODE in draw_state

Remove memset of PIPELINE_NODE from Compute Pipeline creation.
Default constructor handles initialization so no need to memset and this was
causing a crash due to corrupting the active_sets std::set.

layers: MR149, Cleanup CMD_NODE use in draw_state

Move pCmds vector<CMD_NODE*> to be cmds vector<CMD_NODE> instead. This simplifies
cleanup of the data structure as we don't have to manage the ptr new/delete lifecycle.

WindowsRTInst: Layer registry values are created/removed correctly on Windows7

layers: GL84, Reset vkQueuePresentKHR semaphore state in mem_tracker

layers: GL92, Properly remove cmd buf metadata

When FreeCommandBuffers was called the helper routine was not
properly removing the command buffer tracking data.

layers: MR145/GL67, Correctly mirror command buffer inheritance info

Embedded InheritanceInfo struct within command buffer BeginInfo was not correctly
mirrored in draw_state. Correctly copy this data as appropriate and correctly
clear it when command buffer is reset.

Conflicts:
layers/draw_state.cpp

layers: MR144, Fix layer names

layers: MR143/GL85, Fix draw_state implicit command buffer reset

On vkBeginCommandBuffer an implicit reset occurred late and would overwrite the
beginInfo. Moved implicit reset earlier so that beginInfo is maintained
regardless of whether or not implicit reset is triggered.
Also removed some dead code from resetCB().

debug_report: MR141, Change pUserData to match type used elsewhere

Other uses of pUserData in Vulkan do not have a const qualifier,
remove it for this pUserData.

layers: MR142/GL89, draw_state: check swapchain images during descriptor updates

When vkUpdateDescriptorSets() is called, if an imageView was being updated, we
were not including swapchain images in the check to verify a valid image.
Adding these to the check to avoid false positives.

layers: Fix object_trackers dereferencing of invalid pointers

vkUpdateDescriptorSets->VkWriteDescriptorSet: object_tracker should only validate
array specified by VkDescriptorType.

layers: Fix problems with screenshot to get it working again

- Set the layerCount member of VkImageSubresourceLayers
- Add image transition memory barriers (needed for some drivers)
- Add dispatch table to command buffer object created by this layer.
  This lets it get dispatched by a downstream layer, if present.

layers: MR140, Add get_type_def helper in DrawState

layers: MR140, Add SPIRV instruction stream iterator helper

layers: MR140, Pass whole shader_module to build_type_def_index

layers: MR140, Remove some commented merge remnants

layers: LX257, Validate Uniform/Storage buffer offset alignments

loader:  Return the loader created instance rather than from the instance chain

Fix's issue with layers that wrap instance.

installer: Fix LunarXchange issue #287.

Apply David's fix to force the registry to be created.

layers: Fix json file for multi layer

laoder: Fix phys_dev to use unwrapped version in loader_CreateDevice

layers: Add minimal actual analysis of interface blocks to DrawState

There's definitely still room for improvement here (SPIRV-Tools might have some stuff that will make some of these more complex module walks a little clearer); however, this adds enough support for interface blocks for us to generate all the same kinds of mismatch errors we were previously generating for loose inputs and outputs.

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

layers: Fix mishandling of VI vs VS validation (Gitlab#69)

This wasn't quite correct. We'd end up trying to run it_a off beyond attribs.end(),
which is invalid and upsets the MS debug stdlib. This is most likely the root cause
of the weirdness that caused people to add _at_end, _first, etc to this function long ago,
so that can all disappear -- but for now, let's just deal with the actual bug.

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

nulldrv: Initialize image format properties

Previously this was just left uninitialized. Image layer would then randomly fail image creation based on comparisons against junk.

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

nulldrv: Add more proper support for ShaderModule objects

Again not OK to claim success but return junk rather than a proper handle. Fixes various layers exploding when running against nulldrv.

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