Remove last cmd machinery

This isn't useful now.

layers: Split CB_INVALID state

There are really two states here:
- Had a complete recording that was valid at some point, but not now.
- Fouled during recording

layers: Move validate_functions clear into resetCB

layers: Remove dead TODO for MT/CV merge

build: Enable linux 32-bit builds with ASFLAGS

Change-Id: Ic88fdb9d6c0f23a0b345c82852826a9c1abb8097

layers: Remove debug code from vk_layer_table.cpp

Change-Id: I146461055a1f925ed17c1433c8451aa8a65c5578

layers: Fix memory leak in instance destroy_dispatch_table

Change-Id: I8152960f30f73a997386aa9bf3561d2e0e8b408d

layers: Fix memory leak in device destroy_dispatch_table

Change-Id: I14aa005db87caab3cbb28a4e06b1d2a76c931341

build: Do not overwrite CMAKE_ASM-ATT_FLAGS

Allows passing flags to assembler in order to build 32-bit targets
on 64-bit platforms.

A 32-bit build can be produced by passing -DCMAKE_ASM-ATT_FLAGS=--32
to cmake.

scripts:Include string.h in ext helper

vk_extension_helper.h uses strcmp so add include of string.h to make
sure this dependency is correctly met.

layers: Clean up missing checks list for vkCreateBuffer

layers: Allow suppression of doctest output

Added checking in the doc validator script to look for a 'terse_mode'
option which suppresses its informational/warning messages.

Change-Id: I986fb6070b0de53d56e45f5da4b05211203a1b8e

loader: Fix alloc issue

We were allocating the wrong struct for the dispatch table object
when creating the instance.  This left us using invalid memory
when using the physical device extension trampolines.

Change-Id: I9939a8c9fe320b0d07592ab4beb5b6faaba40383

build: Fix ninja build

A previous change of mine broke the ninja build in the loader. This
commit resolves that by fixing the include path

Change-Id: Iaabf3488b323bb3d2d0e40a7d555769c528e61e3

demos: Modify correct uniform buffer for next frame

Change-Id: I78b8f5ef63fb2c14a3ce431f3e6f6a809e65972f

layers: Remove dead comment about interface block handling

loader: Fix unknown phys dev trampolines on linux

Change-Id: I6a8de7feb44587601cc72342f996a26f8ae6bcd8

loader: Fix unknown phys dev trampolines on win32

Convert unknown physical device extension trampolines to assembly
on windows. This is necessary because the previous method relied on
compiler optimizations that were not being run by MSVC, even with
full optimizations enabled.

Change-Id: Ibefa6ed24f8c91b1e723fab9c1cfd3039fabd64b

layers: Fix SC interface matching over complex types

layers: Use copies of layout data for descriptorsets

DescriptrSets used pointers to descriptorsetlayout data, which caused
invalid references if the layouts were deleted before the descriptor
sets. Changed the layer to copy the data.

Change-Id: I671f1efed2aa0986f3a370b51f2f96c07b555af7

external: update external revisions

Change-Id: I076eb49526ea8b54bde2d584c5779c8251e56a3e

layers: Use correct tracking for display planes query

layers: Untangle validate & update in CmdCopyQueryPoolResults

layers: Split validate & update of CmdWriteTimestamp

layers: Split validate & update of CmdEndQuery

layers: Untangle validate & update of CmdBeginQuery

layers: Convert all the std::bind to lambdas

layers: Untangle validateQuery some

layers: Fix parameter order for validateQuery

This was reversed from every case of these two parameters in the API,
which was confusing.

layers: Untangle validate from side effects in CmdResetQueryPool

build: Add CMake flag to suppress LVL content

Add CMake flag INSTALL_LVL_FILES (default ON) that when
set to OFF, suppresses the installation of LVL artifacts
when running "make install" on Linux.

This flag doesn't have much use in the LVL repo because
when turned off, CMake won't generate an install target
in the makefiles.

However, for the downstream VulkanTools and VulkanSamples
repos, it can be useful to avoid installing LVL files on
top of files that may have already been installed from
a previous "make install" installation of the LVL repo.

An example of such a use case would be the desire to
have a more up-to-date version of LVL on the system
than the one last merged into the downstream repo.

This new flag is meant to address the need mentioned in
LVL Pull Request #1844.

Change-Id: I421f37ea4e885fbf0a268eff363a28f0537b1953

build: Install smoketest with make install

smoketest is a standalone app that does not need
additional resources like textures or shader programs.

Change-Id: I335c0baad82343ffd9ff03763c418df11dcc385d

layers: Add in missing CreatDevice physdev obj check

Prevents doc validator from failing.

Change-Id: I0e70da954f0a7d7ed4a54d4bcefe996a6cc9cfb3

scripts:Add includes to extension helper

Generated extension helper has dependencies on vulkan.h and
utility (for std::pair) that weren't declared.

layers: Remove swapchain json files

Change-Id: I0e513ed3054d0d5719ba95b198a489814262b138

docs: Remove swapchain refs from docs/text files

Change-Id: I88d9c5ae82159ad6b1976b0b498fbe4ee7f5e2d5

layers: Remove obsolete entries from layer config

Change-Id: Idd86d217020047efabc4634bfdb441f238a07519

layers: Remove swapchain source and header files

Change-Id: Ifdaa3136883b1129cc435509bf9288b1b5ddf723

layers: Remove swapchain.cpp from vuid validator

Change-Id: I67301721b064cbc4f3d2a55ff42e10a014d86f3c

build: Remove swapchain layer build from cmake

Also removed from android makefiles.

Change-Id: Ib4be1c737e9bd9fda21210ed7b234b9242b7b8f5

demos: Remove swapchain layer from demos

Change-Id: I8628686d3caad6bed2261431776ac9a55d0c076f

loader: Change std_val to ignore swapchain layer

Change-Id: I560dda7156d8550af6b7073eb7fcd0a14aa97131

layers: Move remaining Swapchain checks into CV

Also refactored modified routines for pre/post call structure.

Change-Id: I192e11c1d04effa8d2f3478c1341f6eedea7138a

loader: replace UTF-8 “” with "

layers: Move validation of CreateShaderModule

Now we have no spirv details left in core_validation.cpp

layers: Add new source file to stats script

Missed this one.

layers: Move shader validation plumbing out into own file

layers: Rename whitelist checking routines in UO

Change-Id: Ifa1f5a8ad95c9c2578d5444764f66c5f552078e8

layers: Unique objects GPA handling updates

Change-Id: Id14c4b5ecb96eed9091ee13aceeb1701adb1f511

scripts:Add header protection to dispatch helper

layers: Remove unused PV routines

Change-Id: I0b4a2d1db79a00adca9159345402717d8aafab1b

layers: Make PV GPA handling more consistent

Change-Id: I28d4642ca924928b9a76d6ddeb3a4e9ef106710b

layers: Simplify threadchecker GPA handling

Change-Id: I11cdc2f6524926714d22abd08fba0a670a3a4a97

layers: Remove dead/unused object_tracker functions

Change-Id: If0acd6013b205be973db334f874408d3f4e06c35

layers: Remove extension enable checks from OT

Change-Id: I926836e429a2f38c35291702ab9f71b6d1834d7f

layers: Simplfy object_tracker GetProcAddr handling

Change-Id: Ie38e89f67b293cdfda3ade5ce979464206145226

layers: Update subrange checks+tests to new VUIDs

i.e. disambiguate extension VUIDs from similar VUID in core

layers: Fix some valgrind issues

layers: Fix leak of descriptor pool states

layers: Use unique_ptr for descriptor set layout map

This fixes a leak where PostCallRecordDestroyDescriptorSetLayout erases
from the map without calling delete on the erased element.

layers: Free pipeline state memory in vkDestroyPipeline

Pipeline states are allocated in vkCreateGraphicsPipelines and
vkCreateComputePipelines.

layers: Destroy physical devices in object tracker's vkDestroyInstance

The objects are allocated in EnumeratePhysicalDevices, but were never
destroyed after use.

layers: Free per-device and -instance data on destroy

Instance and device data are allocated by the first call to
GetLayerDataPtr in the corresponding vkCreate call. We need to delete
them when we're done with them. Introduce a FreeLayerDataPtr helper
function that assists with this task.

Deletes for dispatch tables were missing in a few places too. Add them.

layers: Remove dead functions from core_validation

Change-Id: Ifb2d79136bba86bb6655f400a2e46cf23e81dfe3

layers: Clean up core_validation GPA def & handling

Got rid of artificially separations in calling GetProcAddr, remove
ext enable-checking at GPA-time -- this happens in CV.

Change-Id: I89815c86d7f9187537efd3568edd11a9ffffa44f

layers: Remove unused extension enable data structs

Change-Id: Id70d38c4e2d6f0d78b7e2d6e2108b0f39f3e99db

layers: Unify extension enable tracking

Switch parameter validation code-generated extension dependency checks
to use the existing data in vk_extension_helper.h.

Change-Id: I6e3c83f68430c1d3de915e05a84944782b4a55a2

layers: Rename PV device extension enables

Change-Id: I4078b279dd812694c358fdcc9f23f5620befaca0

layers: Remove device_extensions.h from repo

Replaced by code-generated vk_extension_helper.h

Change-Id: I399181eb118db1077cf6681951a68ed0bf4622aa

layers: Move PV to use new extension helper

Change-Id: Ia11632d241b0c717364cc8a2a26d3daaa66bcde5

layers: Remove device_extensions.h from unique objs

Was not being used.

Change-Id: I498b40bd0ee29d4e1ef478b0570947ef2315144e

layers: Renamed extensions struct for devices in CV

Struct contains instance and device extensions, changed name to cover
both.

Change-Id: I34d315b02b426673acb4fc069d8a6a4112f12178

layers: Move CV to use new extension helper

Change-Id: Idb18d0f358fd359c1c5862fba394c833e2fa13f1

build: Add extension helper generation to cmake

Change-Id: Id93ead148d5b0beb23ee5d9f1bc7e8fc727aa46e

scripts: Add codegen for vk_extension_helper.h

This file replaces device_extensions.h and will help to unify
extension enable handling across the validation layers.

Change-Id: I00eb187423e5d912bae64c0e026f34496be185de

scripts: Add generator options for extension helper

Change-Id: Ice9f1361c56de3cc342dd40a589580f4565bbc9e

layers: Fixup VUID ref in comment

Change-Id: I7baf4a5979bd3acf67fc9c3d722ed8c9584f27a0

layers: Fix script crash for special-cased err enum

Change-Id: Ic054c35fd777e66e85ed56978ad1628ec24b94a6

scripts:Update bad line in database

The required extension for one VU is causing a multi-line entry in
database file. Manually updated the line and will pursue permanent
fix by updating spec.

layers: Update vuid scripts for Windows, python3

Change-Id: I1ec5ae94677412a8a6318f99f09b1fa095bf97ab

scripts:Update VUIDs for spec 1.0.51

This is the first spec update using the new json process.
Updated text for a number of IDs and also added 9 new IDs.

Update command was "python spec.py -update"

scripts:Update json parsing and vuid update

Account for new "validation" top-level in json file when parsing.
Also fix a couple of bugs: wasn't correctly translating ID to hex for
new new IDs and wasn't setting implicit state for new IDs.

header: Update to 1.0.51 Vulkan version

- updated include/vulkan/vulkan.h
- updated scripts/vk.xml
- updated layers json files
- updated tests json files
- built and included new vulkan.hpp file

Change-Id: I4f1d5609c8377047165e2ba14e89cba9877436db

layers: Fix bad behavior in repeated calls to CV GetSwapchainImagesKHR

layers: Fix some typos

layers: Remove FRAMEBUFFER_STATE::referencingCommandBuffers

Nothing used this.

layers: Remove MT_FB_ATTACHMENT_INFO::mem

Nothing used this.

layers: Remove dead device queue tracking from swapchain

layers: remove useless check from GetSwapchainImagesKHR

This is a relic from a /very/ old version of the WSI extensions, and
appears to be checking driver correctness more than app correctness.

layers: Remove check from swapchain already done by OT

loader: Fix buffer overflow in loader_add_to_dev_ext_list

The size argument to a call to memcpy is for the wrong struct. The type
of `ext->list[idx].props` is VkExtensionProperties.

Credit goes to AddressSanitizer for spotting this bug.

loader: Add emulation for EXT inst extensions

Change-Id: I481ff8a2ae0ec5a82773fddd2b59c750126adc11

demos: Fix fence sync after new throttling scheme

f06ab9 cube: Throttle rendering rather than presentation didn't
remove the AMD workaround and didn't consider the VK_ERROR_OUT_OF_DATE_KHR
path through resize.

Change-Id: Ib29bffb5b3c2a82456ad734424af3a9348e897c5

tests: Add ExceedMemoryAllocationCount using profile layer

Change-Id: I363acfff6cffa5966813dbfd30bb0c28810bd145

layers: Add todo for pNext ext dependency codegen

Change-Id: I5754ceeccca23723282b834e6fe9feadc1d74ae1

layers: Add missing/disabled extension validation

Many of the main validation checks were ifdef'd out or were missing
entirely.

Change-Id: I18cd977a6a790cc12988c55093d826ab9ec50556

layers: Remove redundant device ext dep checks

These are now handled through code-gen.

Change-Id: I1430f25fbe151b5581bb57292e733e8ba66cbdf7

layers: Remove redundant instance ext dep checks

These are now handled through code-gen.

Change-Id: I773a2a3a5e531ba24108aeadf3fd67239a20c5e3

Layers: Implement PV per-API ext dependency checks

As each API is used, PV will now us a code-generated check to verify
that all of the required extensions for this particular API call have
been enabled via CreateInstance or CreateDevice.

Change-Id: I1c11d0b8322edf005b2b197a415c92a82e0cb810

layers: Pass layer data into PV validation functions

Previously just passed in report data, but need access to more stuff.

Change-Id: I2af7b2ae73d3e2ecd3409f32e8f1a42e5e736d8c