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

vktrace: Don't put VkAllocationCallbacks in trace file.

Allocation callbacks don't make sense in the trace file
because the callback code isn't available for replay.
Instead, put NULL in the trace file so that replay uses the
default allocators.

This fixes a problem with tri demo which uses allocator
callbacks for create and destroy instance.  Replay would
crash when playing back destroy instance because it jumped
to an invalid location. (create instance hard-coded a NULL)

Note that space is still reserved, although unused, in the
trace buffer for VkAllocationCallbacks.  This should not
be a huge space issue, since the Vulkan calls that use these
callbacks are relatively infrequent.
This commit does make the buffer allocation and the use of NULL
more consistent across the API functions.

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.

tests: Fix out-of-scope vector initialization in test framework

This issue caused random param-checker errors in the tests using the
test framework.

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

vktrace: Fix CreatePipelineCache to handle InitialData.

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.

Relicense vkjson from 3-BSD to MIT

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

vktrace : MR178, Correctly interpret queue priority in VkDeviceCreateInfo packet

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

tests: MR159, Add a test for an array size mismatch between VS and FS

Demonstrates some weird behavior seen in LX#309 LX#310 LX#314.

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

vktrace: Fix CmdPushConstants to save data in the trace buffer.

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.

vktrace: Handle SpecializationInfo in ShaderStage create correctly.

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.

vktrace: Allow AllocateCommandBuffers to work with multiple buffers.

Added one more license to list of licenses

FIXES: Fixed cmake generation issue discovered by Tony.

Fixed a cmake generation issue in the ILO driver caused by my previous changes.

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.