Swapchain: Start keeping better track of surface-queueFamilyIndices.

Swapchain: Match validity rules for pCreateInfo->imageUsage.

Should not check that the value is non-zero, only that it's a subset of the
bits in pCapabilities->supportedUsageFlags.

misc: Update BUILD.md to point to latest ImageMagick.

We probably won't keep this link around much longer, but I wanted to
continue to provide a working link so people don't have issues installing
the appropriate version of ImageMagick.

layers: Fix 32-bit compilation warnings/errors introduced last night.

Several 32-bit compilation warnings and errors were introduced into
draw_state.cpp last night.  This fixes those issues and allows our builds
to work again.

layers: MR172, Add draw_state cmdbuffer lifetime checks to cmd pool destruction

layers: MR171, Validate pool cannot be reset when buffer in use in draw_state

layers: MR170, Validate queue paramters when getting a queue in param_checker

layers: MR169, Validate queueCount in param_checker

layers: MR168, Validate queue priority between 0 and 1 in param_checker

layers: MR167, Make sure queueCount is non-zero in param_checker

Conflicts:
layers/param_checker.cpp

layers: MR166, Avoid duplicate queueFamilyIndices in param_checker

layers: MR160, Fix Android build

vktrace: fixes to allow vktrace/replay to work with Hologram

- Protect strlen() against NULL strings
- Avoid freeing remapped semaphores in replay QueuePresent before using them
- Use a VkMemory total size instead of the literal VK_WHOLE_SIZE in trace buffers

layers: MR159, Add helper to get the value of an assumed integral constant

We need this in a bunch of array-handling code.

layers: MR159/LX309/LX310/LX304, Fix handling of OpTypeArray size in DrawState

Array size is specified via a constant instruction, not a literal.

layers: MR159, Collect offsets of instructions producing constants

We're about to need to be able to look these up by <id> for proper array size
handling.

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.