layers: Validate subpass index in begin info is valid.

layers: Make sure fb render pass and secondary render pass are compatible.

layers: Validate that the framebuffer specified in the secondary command buffer is the same as the current one.

layers: Validate render pass compatiblity for executed secondary command buffers.

layers: Fix issues with some command buffer tracking structures being left uninitialized

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

layers: Fix 32-bit Windows build

A reinterpret_cast from a non-dispatch handle to uint64_t was failing on
windows 32-bit where non-dispatch handles are defined as 'typdef uint64_t
object'.  Changed reinterpret_cast to a C-style cast, as is consistent
with the rest of the non-dispatch handle to uint64_t conversions in
draw_state.

loader: GL120, move critical section to cover loader_get_icd_and_device

demos: Copyright update

layers: MR209 Fix swapchain's vkDestroyInstance to error about devices ...

not being previously destroyed, instead of physical devices not being
previously destroyed.

layers: Valdiate query is not active when cmd buffer is ending.

layers: GL122 Init queueFamilyIndex in command pool.

layers: Fix Windows build error in device_limits

layers: GL96 validateCommandBufferState should return skipCall

When validation checks were rolled into separate validateCommandBufferState
function, forgot to return the skipCall value from that function.

layers: GL64 Don't warn on descriptor set that's not updated

Previously was not handling the case of 0 sets in a layout.
Now catching this case by verifying that a descriptor set's
layout bindingCount is not zero before incrementing internal
descriptorCount variable.

layers: Check that queryFlags is valid if enabled.

layers: Add check to make sure inherited queries in enabled before use.

layers: Add occlusion query precise check.

layers: Improve in flight cmd buffer validation

Validation in flight cmd buffers are not reset.
Validate cmd buffer is not in use when freed.

layers: Add queue bit checks.

For each command check to make sure if the queue bits are set
correctly for that command.

demos: Minor cube/tri updates

Minor updates/fixes for cube and tri:
 - Break from demo_check_layers search loop when finding a match.
 - Skip memory allocation and layer/extension processing when 0
   layers/extensions are enumerated.
 - Fix potential NULL pointer dereference and exit with message when
   no accessible devices are available.
 - Fix device_extensions memory leak.
 - Remove trailing whitespace.

demos/tests: Enable unique_objects layer

Using VK_LAYER_GOOGLE_unique_objects with 'cube --validate' and when
running all tests with layers.

windowsrtinstaller: update estimated size for Programs and Features

loader: Fix bug on pass by value for created_instance

Revert "layers: Clang-format changes for files in layers dir of repo"

This reverts commit e260acfb96ea20945149ede6a250ef622ca7fd03.
The clang-format changes need to be postponed until just prior
to public release to facilitate MRs.

license: Update the main MIT license to reflect white space and copyrights

license: remove licenses for componenets that were moved to different repo.

loader:Update copyright and apply clang-format (no logic changes)

Update copyright info in python scripts

layers: Clang-format changes for files in layers dir of repo

windowrtinstaller: use new image files names in installer

windowsrtinstaller: changed images

layers: Validation Layer Copyright Updates

demos: Update copyright and apply clang-format

Update BUILD.md and README.md for the reduced scope of this repo

scripts: Remove icd generation specific code

androidBuild: remove unused layers  api_dump, generic, basic, multi

layers: Remove non-validation layers

Removes api_dump, generic, basic, multi.

misc: Remove vktrace and icd from this repository

include: Update copyright and apply clang-format (no logic changes)

windowssdk: Handle silent installer/uninstaller message boxes appropriately.

If Silent Install or Uninstall is enabled, we want all the message boxes we
have control over to automatically make a selection.  In addition, return
an error if we are supposed to reboot, and the silent install doesn't show
the message for the user to choose.

misc: Modify build_windows_targets.bat to build quietly.

Making these changes allows errors/warnings to be more obvious.

windowsrtinstaller: Don't display dialog when silent uninstall requires reboot

scripts: Fix python helper to find ext headers under vulkan dir

windowsrtinstaller: change windows icon

Merge branch 'marky_working'

misc: Modify update_external_sources.bat to build quietly.

This speeds up the build as well as makes warnings and errors more obvious.

Revert "misc: Change repo name references to LoaderAndValidationLayers"

This reverts commit 015e74cda0b9ad46bd6127d816f14d699697ec4a.

misc: Change repo name references to LoaderAndValidationLayers

windowsrtinstaller: Handle error conditions during install by uninstalling.

If an error occurs during install, uninstall the portions that have been
installed, and exit with an error code.

layers: MR182/GL113 Fix dynamic offset validation

Dynamic offsets are bound to CB, so move the tracking struct into CB struct.
At draw time, iterate over all of the active sets and verify that each
dynamic set does not overstep its buffer based on the corresponding dynamic offset.

windowsrtinstaller: change icon used by installer

docs: remove old obsolete documents

windowsrtinstaller: change licensing and change logo displayed in UI

layers: MR181/GL102 Fix unique_objects to correctly assign local vars under ptrs

If a variable that needs to be restored is below a ptr, then need to declare
it at the top level, but only assign it if the ptr(s) that it's below are
non-null.

layers: MR178, Merge-related fix

layers: MR178, Fix descriptor set in_use tracking

This breaks descriptor set validation into two cases:
1. If a descriptor set is being used in an in-flight command buffer, immediately
flag an error if that set is destroyed or updated.
2. If a descriptor set bound to a command buffer (but not in-flight) is updated
or destroyed, mark that command buffer as "INVALID" and flag an error if such a
command buffer is submitted.

Fix vkWaitForFences() code to not do any cleanup for the case where some, but not
all, command buffer fences have completed. In this case the app will have to either
check individual fences or use waitAll flag so we will clean up at that point.

layers: MR178, Various fixes to inFlightCmdBuffers tracking in draw_state

For clarity change the name of device-level set to "globalInFlightCmdBuffers".
Correctly handle validation errors and inFlightCmdBuffer clean-up when waiting for device idle.
No need to do another queue node look-up when inserting inFlightCmdBuffers at
trackCommandBuffers time, just use data from earlier look-up.
Correctly set queue when creating QUEUE_NODE.
Add locking around resetCB.
Refactor large set of checks at vkQueueSubmit time into validateCommandBufferState() function.

Conflicts:
layers/draw_state.cpp

winrtinstaller: change wording in license file

misc: Create a script to allow building 32-bit and 64-bit Windows.

Create a script to allow people to build both versions of Windows
targets from a command-line.

layers: Fix issue with api_dump when const char* is null.

The pEngineName variable in Hologram was NULL and crashed on my 32-bit run
with the api_dump layer enabled.

layers: Correct names of settings api_dump looks for in vk_layer_settings.

The settings text file was using a mixed-case prefix "Api" while the layer was
looking for upper-case prefix "API".  This caused the layer to ignore changes
made to the vk_layer_settings file.

loader: Fix 32-bit loader exception when layers enabled for Vulkan apps.

The loader_create_device_terminator had an incorrect calling convention.  This
caused the ESP pointer to not get properly restored after the function was
called by a 32-bit application.

layers: Fix 32-bit Win compilation issue in swapchain layer.

The layer was using an unordered_map<void*, SwpSurface> ... in several
places, but VkSurfaceKHR (unlike many other Vulkan types) isn't a pointer
but a uint64_t.  On Win32, it's trying to push in a uint64_t into a 32-bit
pointer, which the compiler is choking on.  Instead, make the unordered_map
use VkSurfaceKHR.

Swapchain: Add Google copyright

Swapchain: Fix typo

Swapchain: Ensure Destroy{Device|Surface}() don't core dump.

There are multiple pieces of code that can try to delete an SwpSwapchain struct:

- DestroyDevice()
- DestroySurface()
- DestroySwapchain()

This tries to address the various paths through the code, so that
Destroy{Device|Surface}() won't also try to destroy an already-destroyed
swapchain.

Swapchain: Validating supported surface for presenting with a queue.

This validation involves multiple levels of indirection as vkQueuePresentKHR()
receives a queue parameter, and a pPresentInfo->pSwapchains array.  For each
swapchain, it's surface must have been seen to be supported with a
queueFamilyIndex that is associated with the queue parameter.  Info must be
recorded at various previous calls, and be correlatable at this point in time.
Very complicated!

Swapchain: vkDestroyInstance check if child objects were destroyed.

Swapchain: Check "compatible" pAllocator for creation/destruction.

Swapchain: Keep/check relationships between surfaces and swapchains.

Swapchain: Validate that queueFamilyIndex is within bounds.

Swapchain: Add TODOs for difficult-to-do validity language.

Swapchain: Check that pCreateInfo->compositeAlpha has one bit set.

Swapchain: Check that pCreateInfo->preTransform has one bit set.

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.