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

layers: Move PV layer data structs into header

Change-Id: Ie8fb823f1bfa97047cb2d361bc0daea5ee504673

Layers: Add code-gen'd calls to val exts

Modified PV code gen to insert and make calls to validate
extension dependencies.

Change-Id: I90103b16d36fb868b32df796b1dae7741dfcba34

layers: Remove duplicate check and dead code

Change-Id: Ib25a9fc9052897eccb244d85122b451bda02f892

layers: PV code gen for FlagBits enum parameters

Extend validate_flags to work in instances where a single Vk*FlagBits
enum value is valid instead of a bitmask with multiple values.

Change-Id: Ib8d34551923f7ecb2549500dfde674fa1094ac8b

layers: Add VU offset checks to BlitImage

Added 10 VU checks to CmdBlitImage around the offset values in the
VkImageBlit struct.  Cleaned up a few tests that were tripping
these checks inadvertently. Added BlitImageOffsets test.

Also updated the appended error message number to match the new
8-char hex format.

Change-Id: I75d319cce6bbe96e8686e0bad3c4a0d58922b7a4

loader: Emulate surface capabilities 2

Add loader emulation for VK_KHR_get_surface_capabilities2 commands

Change-Id: I1de99d1bc94d6ec1191601e75b79c5c32fa81fdc

layers: Add VkObjectType conversion

Add conversion from VulkanObjectType internal enum to the
VkObjectType core enumeration.

Change-Id: I17158722faffaf088e07219a5bf596ee02d2338e

build: Speed up AppVeyor builds.

- Build only VS 2015
- Cache the external directory

tests: Split up ImageLayerViewTests

This test name didn't come close to describing what it was trying to do.
Split into focused tests that cover each case of CreateImageView.

tests: Remove problematical KHR_mnt1 check

Remove KHR_Maintenance1 check and conditional code from test
CopyImageTypeExtentMismatch(). Disable two clauses that are
specific to the extension, for the time being. Update the VU
database to replace missing test names.

Change-Id: I0c1442b943c3ed8a105b771868946ae68e543b50

layers: gh1649 - Fix clang warning

The function pointer definition for the Physical device extensions
was incorrect.  Updated to work properly and remove clang warning.

This was also causing an issue with Tony's latest test.

Change-Id: Ibdd19754a4394d1b88d3a20268238baaf3ec0907

external: Update external revisions

Change-Id: Ie168ac55cc12d317a2c189dbaee735718b295c04

loader: Fix env usage and cleanup messages

Fix the environment variable usage of VK_LOADER_DISABLE_INST_EXT_FILTER
so that it will also ignore the unknown extensions during
vkCreateInstance and clean up a few more messages.
The previous messages were confusing and didn't indicate what
was actually wrong.

Change-Id: Iccd0a4b2cfd673187cb7e92fe9cec23deb058ff2

loader: Extend emulation to VkDevice creation

Add support for VkPhysicalDeviceFeatures2KHR and
VkDeviceGroupDeviceCreateInfoKHX structs in the pNext chain
of VkDeviceCreateInfo for emulated physical devices

Change-Id: Ib00e35546f064f57f987d8eda14fba17bceab52c

loader: Add additional extensions to ext emulation

The loader will now handle structures from VK_KHX_multiview and
VK_KHX_external_memory_capabilities in the pNext fields of commands
from VK_KHR_get_physical_device_properties2.

Change-Id: Ic7175e9334c56c7642b75360b35a6b9ff240ee98

layers: Add new exts to device_extensions hdr

Change-Id: Iddaf6cda599d3d98f678f74fd206cf6b670a275f

layers: Add tracking of VkDescriptorUpdateTemplateKHR

Add tracking of the new VkDescriptorUpdateTemplateKHR handle.

Also, update to include proper VUIDs for the corresponding checks
in object_tracker.cpp with help from Mark L and Mike S.

Change-Id: I4a665cfa2e8b4d8faf2a0ebd583c759e399b1599

scripts:Fix doc validation for 8-col DB

Update doc validation to read in the new VUID string and core|ext
columns of the vk_validation_error_database.txt.

tests: Fix test names in database

Change-Id: Ib96edb3a15a8496040c7ccb0db019ccbd92e94e7

external: update external revision for glslang

Pick up a fix for built-in redeclarations.

Change-Id: I2b506e24b2cf619b07ff1c75ab0a9a05143379ca

layers:Pipe in disable for shader validation

Allow VK_VALIDATION_CHECK_SHADERS_EXT enum to disable the spirv-tools-
based shader validation in core_validation.

There were a few other pieces of internal shader validation and
tracking that were tied to the shader_validation bit but I re-enabled
those bits for now as we lose the active descriptor binding tracking if
those features are disabled. That would lead to a various other
features that need to be disabled to avoid crashes. Specifically the
connection between command buffers and active samplers, buffer, images
that are present in descriptor sets.

To test this I ran layer_validation_tests with the shaders disabled
and most tests work fine with only the ones testing internal shader
validation pieces failing as expected.

scripts:Remove cruft from spec.py script

Kill a bunch of old stuff from spec.py now that we can use json file
for updating/tracking the VUIDs.

Update the error DB & header to be sorted by the new IDs.

Also add step to VU txt parsing to remove any backslashes which will
break compilation of vk_validation_error_messages.h.

layers: Add PVRTC formats to format utils

Update vk_format_utils to classify PVRTC formats correctly.

Change-Id: Ie48883885725a28d62dbafa66151971e0f33a45e

layers:Migrate to new validation error IDs

Migrate valid usage IDs to new numbers based on the string IDs that are
integrated into the spec build process.

The formula for mapping the string ID to the new VALIDATION_ERROR enum
value is in vuid_mapping.py script.
The mapping between the old IDs and the new IDs is in the file
"old_vuid_to_new_mapping.txt."

The vk_validation_error_database.txt file has the new IDs as well as
the string-based ID for each element and a column indicating if the
valid usage is in the core spec or if it requires extension(s) and is
therefore in the extension spec.

The error messages were shortened and no longer include a section
reference. Now the link goes directly to the line in the spec where
the valid usage text occurs. The base of the spec link is the core spec
unless the VU requires an extension in which case the spec with all
extensions is used as the base.

scripts:Add core/ext column to error database

Update the validation error database to have a column that indicates if
any extensions are required for a given VU to be valid. If an extension
is required then the spec link will point into the spec with all
extensions, otherwise the core spec will be linked.

scripts:Add string-based VUID to db file

Include the new, string-based Valid Usage Unique IDs in the validation
database file. This will be use to map between the old ids and these
new ids.

These script updates are a bit hacky as the migration is underway and
function is being favored over form. Once migration is complete I can
strip down spec.py script to core functionality which should be much
cleaner and simpler than either the original code or this current form.

scripts:Preparing to migrate VUIDs

Initial changes to spec.py in order to parse json file of VUs
and convert string-based VUs into unique integer IDs that will be
used for the enum values.

Still need to get a clean json file and parse all of the VUs to
confirm that they're unique. Once that's done the actual conversion
will be a separate step.

The VUID string to unique ID mapping code is in vuid_mapping.py.
This includes tables for mapping from api/struct/param names to unique
ids and the convertVUID() function that takes a string and maps it to a
uniqueID.

loader: Fix extension surface params

Change-Id: I310d6d356b5c9377ff8285ee123def989e95b142

external: update external revisions

Update external revisions for glslang, spirv-tools, spirv-headers, and
shaderc.

Change-Id: Ie1f4afe2d36a6d8e3d1e166c23372f8f6523cf59

tests: use built-in gl_Position

This change is required for upcoming revision of glslang, which requires
location on input/outputs but which does not allow location on
built-ins.

Change-Id: I9ba8989e5ac69bf422e16d4e7ffc7d33b9f71760

header: Update to 1.0.50 Vulkan version

- updated include/vulkan/vulkan.h
- updated scripts/vk.xml
- updated layers json files
- updated tests json files
- build new verison of vulkan.hpp
- code fixups for deprecated DebugReportError enum

Note that this does NOT include any shader-checker support for the
VK_AMD_texture_gather_bias_lod extension, as this requires glslang
and SPIR-V updates.

Change-Id: Ie01aed143f2fad065c86e2a27e46fe32ccb9e1f7

scripts:Update Validation IDs for spec 1.0.50

I'm attempting to migrate to the new VUIDs that are integrated into the
spec this week so this will hopefully be the last update of this type.

Migration plan is:
-Update existing VUID database to include new, integrated, string VUIDs
-Update VUID db to include core/ext data from json
-Update links/messages in VUIDs
-Move all old VUIDs in the source to use new VUIDs

Command to perform this current update was:
python spec.py -update -remap 3076-2049,2

And I manually restored 911 & 912 which are in the core spec but not in
the extension spec.