android: Removed building of struct_size helpers

Change-Id: I2e3fee12a8b25f5ae3251687b64d7359a6572db6

layers: Removed build of struct_size helpers

These are no longer used in this repository.

Change-Id: I07c5402d3e7f9b4578eabe274d0e7b6b76ca6186

layers:Check secondary command buffer renderArea

Fixes #2480
Fixes #1501

Add delayed function call to verify secondary command buffer renderArea
against vkCmdClearAttachments() rects at vkCmdExecuteCommands() time.

Updated the lambda function container for delayed secondary command
buffer checks to include a reference to the primary command buffer.
Add lambda call at vkCmdClearAttachments() for secondary command buffer
and pass in primary command buffer reference so that area restriction
can be appropriately checked at vkCmdExecuteCommands() time when
renderArea of renderPass is known.

layers:Consistent parens for function names

Add "()" where missing on a couple of output strings for
vkCmdClearAttachments function.

layers: Downgrade maxResourceSize check to warning

layers: Update VkViewport check to 1.1

loader: Remove old declaration for KHX extension

scripts: helper_file_generator.py cleanup

- Remove list as set anti-pattern
- Remove conditional that could never evaluate to False

scripts: Merge upstream verbosity changes

From Vulkan-Docs /src/spec/genvk.py

demos: Fix cube build on android

- Add generated files to include path
- Use messageSeverity to determine android log priority
- Prefix ANativeWindow with struct to match vulkan_android.h

Change-Id: I1c29c41a95614701455074cdf7be49f47505a0fc

header: 1.1.70 update

Change-Id: Ifa07a957e3c0e51c9b5c00b6c9d7fff04905c8ab

scripts: Roundup strings sizes in struct to multiple of 4

vktrace layer needs strings sizes to be multiple of 4 for
data alignment in the trace file.

layers: Fix CmdDispatchBaseKHX rename in PV

layers: Update unique_objects changes for 1.1

layers: Make unique_objects ID map global

Increasingly, instance objects are referenced by device-level APIs.
Made unique ID map global, shared across all instances and devices.

Change-Id: I5b62e7b95ec25a2a8b390d46024b43c29f2199ce

scripts: Remove unreleased extension names

Change-Id: Ia0270edcd17a8b8a33e390f781b7357208e7a676

build: Avoid using CMAKE_BINARY_DIR

Using CMAKE_BINARY_DIR was breaking the use of this repo as a
submodule because the top level binary directory is different when
build as part of another repo.

layers: Fix typo in CV downchain call

Change-Id: I8d7ecbb1f4e63b9c4bc1ee9b53ea65c34fb2352d

demos: Fix cube note building on windows

layers: Add remaining core_validation aliases

vkBindBufferMemory2
vkGetBufferMemoryRequirements2
vkGetImageMemoryRequirements2
vkGetImageSparseMemoryRequirements2
vkGetPhysicalDeviceSparseImageFormatProperties2
vkBindImageMemory2

Change-Id: I1fb1a476e680c2d601a9b72029358b8b42950ed1

demos: Fix compile error in cube.c

Change-Id: I9ff2429f6f2b79795bbc57e376decae8c0dae630

helper: Fix compilation warning

Adding vk_enum_string_helper.h produced a warning that
GetPhysDevFeatureString was unused.  Made it inline (as all
the other commands are in that header) and it made the compiler
happy.

Change-Id: I9452ddfc8de4af4c88b95e380b9520d9f59796da

cube: Use string_VkObjectType

Use the existing string_VkObjectType utility found in
vk_enum_string_helper.h instead of the manually defined
DebugAnnotObjectToString function.

Change-Id: I9e9ecfc1c512ec1b9b98cce2f846c565ac98ecbf

cube: Update to use VK_EXT_debug_utils

Update cube to use VK_EXT_debug_utils instead of
VK_EXT_debug_report.

Change-Id: I238072855abdf09c9d33a66b3a7cc0beecc0e8c7

layers: Update VUIDs for 1.1

- Add new VUIDs to database and vuid_mapping.py
- Remove deleted VUIDs from database
- Update vuid_mapping.py to disregard KHR/KHX suffix
- Update spec.py json comparison to update more fields from existing
  VUIDs and delete entries for removed VUIDs
- Update layers and tests that were pointing to deleted VUIDs to use
  replacements
- Revert vuid_mapping.py and run_all_tests.sh workarounds for undefined
  private VUIDs

Change-Id: Ie6c5b01237d5917be66a35337e3dbaf42fea3b00

Revert "loader: Only return ext pointers if ext is enabled"

This reverts commit 4537090525b430e2bc059cc59c7de3c80a7e7e6e.

layers: Bump layer json files to 1.1.69

loader: Fix debug_report not returning NULL

Fix a bug where debug_report would be treated as an unknown extension
if it was used without being enabled.

loader: Only return ext pointers if ext is enabled

The loader previously returned valid pointers to functions from
VK_KHR_swapchain and VK_KHR_display_swapchain even if the extensions
weren't enabled. This change fixed that.

header: Update to 1.1.69 private header

Change-Id: If9fc0f065f77ecff14123f665e603e6f2595d571

layers: add GetDeviceQueue2() support

This change add GetDeviceQueue2() support to object tracker and core
validation layers.

layers: Fix wrong format specifier in layer log

loader: Fix MSVC uninitialize variable warning

loader: Extend pre-instance intercepts for 1.1

Add the ability to intercept vkEnumerateInstanceVersion through the
pre-instance intercept mechanism

scripts: Fix xlib_xrandr platform name

Change-Id: I3aa8e929d394322ebfcfbf2a59846b2adae057e3

Device group extension rename

VK_KHX_device_group/VK_KHX_device_group_creation are now KHR

Change-Id: I2ade2e7162718a4683c9365ae177891285f177b4

scripts: Update codegen to work with new aliasing

Change-Id: I3739f7b7b6eb17a5e3e1bc9cc18e74569cb50f95

Fix vkGetDeviceQueue2 segfault when returned vkQueue is VK_NULL_HANDLE

This affects the Loader.

Change-Id: I05a3feb87781be773f30327601a2bf86a53ae115

Implement initial VK_EXT_debug_utils changes

This affects the loader, scripts, and layers and introduces the
changes to support the VK_EXT_debug_utils extension.

Change-Id: Ia5336f63e85b00f1e59416c06aacd4ae331fd692

build: Bump linux loader version to 1.1

docs: Update loader doc for 1.0 ICD handling

The 1.1 loader has to handle 1.0 ICDs specially during instance
creation. This commit documents that behavior.

loader: Update 1.0 checking mechanism

The loader will now get vkEnumerateInstanceVersion from the ICD by
calling the ICD's GetInstanceProcAddr function.

loader: Disable emulation for 1.1 core functions

Disable emulation for physical device functions when they are used
as core 1.1. The emulation will still be used if they are used as the
original instance extensions.

loader: Check vkEnumerateInstanceVersion in ICD

The loader will now check vkEnumerateInstanceVersion in each ICD
before creating an instance to determine if the loader needs to
change the apiVersion field.

demos: Update Vulkan Instance Version reporting

layers: Enable 1.0 exts moved to 1.1 core

Also plumbed api_version into CV and PV for validation and enabling
the appropriate 1.0 extensions by default in 1.1.

layers: Update core_validation for aliases

Also refactored for pre/post architecture.

CreateDescriptorUpdateTemplate[KHR]
DestroyDescriptorUpdateTemplate[KHR]
UpdateDescriptorSetWithTemplate[KHR]
GetPhysicalDeviceQueueFamilyProperties2[KHR]
EnumeratePhysicalDeviceGroups[KHR]

layers: Updated unique objects layer for aliases

UpdateDescriptorSetWithTemplate[KHR]
CreateDescriptorUpdateTemplate[KHR]
DestroyDescriptorUpdateTemplate[KHR]

layers: Updated Object Tracker layer for aliases

GetPhysicalDeviceQueueFamilyProperties2[KHR]

layers: Handle aliasing in parameter validation

loader: Remove repeated function from vulkan.def

loader: Re-add old 1.1 loader work

vkEnumerateInstanceVersion got removed, as did the checks for layer
versions when we rebased off of github. This adds them back in.

loader: Fix crash in VK_KHR_surface

The VkSurfaceKHR object in vkGetDeviceGroupSurfacePresentModesKHR was
not getting properly unwrapped. As a result, an invalid surface handle
was being passed to the drivers (usually resulting in a crash).

loader: Fix loader using stale VkApplicationInfo

Fix a bug where the loader could crash because it used a
VkApplicationInfo struct after the struct had gone out of scope.

loader: Remove version check from vkCreateInstance

loader: Fix crash with null VkApplicationInfo

Fix a crash when creating an instance where
pCreateInfo->pApplicationInfo is set to null.

loader: Fix crash in aliased extensions

loader: Update loader for Vulkan 1.1

loader: Update functions names

loader: Update vkEnumerateInstanceVersion for 1.1

loader: Add new core 1.1 entry points

loader: Fix windows build

loader: Fix loader marking 1.1 cmds as required

loader: Move manual KHR exts to core

loader: Update vulkan version location

scripts: Update generators for 1.1

layers: Fix unique_objects for renamed safe structs

layers: Fixed issues due to safe struct renames

scripts: Avoid duplicates in enum string helper

scripts: Fix loader extension generator

Was incorrectly creating device/instance dispatch tables.

scripts: Private repo VUID workaround

Workarounds for private codegen and tests so they can run without having
to update the VUID mapping or database

Change-Id: Idf71df859f55888103f93f863fc73a0c7a01de2c

header: KHX->KHR name changes

scripts: Update lvl_genvk.py with 1.1 changes

scripts: Allow for skipping version_1_1 features

Treat 1.1 features as 1.0, instead of as an extension.

docs: Update Contributing.md for msg output info

Change-Id: Iab9831c798970334808c5d0784e1e5128f05c6f6

vulkaninfo: Add extension output

Added output for extension structures:

VkPhysicalDevice16BitStorageFeaturesKHR
VkPhysicalDeviceSamplerYcbcrConversionFeaturesKHR
VkPhysicalDeviceVariablePointerFeaturesKHR
VkPhysicalDeviceBlendOperationAdvancedFeaturesEXT
VkPhysicalDeviceBlendOperationAdvancedProper-
tiesEXT
VkPhysicalDevicePointClippingPropertiesKHR
VkPhysicalDevicePushDescriptorPropertiesKHR
VkPhysicalDeviceDiscardRectanglePropertiesEXT.

Change-Id: I5718ab765f48fb0d65ddecd6718f0c14e814e715

layers: Add support for StorageBuffer storage class

Add basic support for storage buffers using the StorageBuffer storage
class rather than UniformConstant + BufferBlock decoration.

layers: Require appropriate feature enabled for writable descriptors

layers: Determine whether a shader uses any writable descriptors

layers: Param validation checks multiplane bits

Update parameter validation checks to recognize the new aspect flag
bits added by VK_KHR_sampler_ycbcr_conversion extension
(VK_IMAGE_ASPECT_PLANE_n_BIT_KHR). Update core validation checks to
avoid false positives when copying multiplane images.

Change-Id: I6fb19ff76386ce55f3b44b0301ba7cd7f7e88547

layers: Cleanup naming for layout compatibiity

Type and variable names have been clarified for the pipeline layout
array of descriptor set layouts based on reviewer feedback.

Change-Id: I6b191b2121db87285f7fd50810991340c02475c4

layers: Improve pipeline layout compat updates

Replaced existing "compatible for set N" and descriptor set binding
update/disturb logic with one using the cannonical form dictionaries and
implementing all of the disturb before and after rules.  Also applied
this update/disturb logic to PushDescriptors recording.

Change-Id: I950c8e5d56c2dbc81fc52136af5a22882cfbc7a4

layers: Add canonical/unique ID to pipeline layout

Add unique ID's to pipeline layout needed for pipeline layout
compatibilty revamp in subsequent commit.  Refactor unique ID support
into common Dictionary implementation.

Change-Id: I0d864c8ef3b3406d6444aed4d73078d25d5eb26f

layers: Store PushConstant ranges in dictionary

For rapid compatiblity checks, create unique ID's for PushConstant
ranges.

Change-Id: Ie33372e2428e453e610e292b2917801225c11020

layers: Give compatible DSL unique ids

Store compatible DescriptorSetLayout definitions in a dictionary with
unique IDs.  Use IDs for trivial accept compatibility validation.

Change-Id: I1085a354d36d1a733f79adb4459decf8cf0a55c1

layers: Split DescriptorSetLayout data structure

In preparing for use of "common definiton" id's split the
DescriptorSetLayout data structure into handle specific and common (def)
components.  Object interface is preserved.

Change-Id: I5c6b1e726aa70de548c65a5f5ad08726b0a7507b

layers/vk_layer_config.cpp: fix win32 include case

Fixes mingw-w64 compilation on case-sensitive file systems.

docs: Update brew package list

Starting around Feb 27 2018, "python" installs python 3.x instead of
2.x.  Also remove git which should be present from the Xcode command
line tools and python2 which should be present in the base macOS
install.

tests: Add unit tests for dedicated memory binding

Added unit tests for the following valid usage checks w.r.t. binding
dedicated allocations to buffers and images.

VALIDATION_ERROR_17000bc8 VUID-vkBindBufferMemory-memory-01508
VALIDATION_ERROR_17400bca VUID-vkBindImageMemory-memory-01509

As same code path checks the BindImage2/BufferMemory2, but has no
matching valid usage, no tests are added for those cases.

Change-Id: Id8845d3cdb85731142ca0dca381a5e9157118a25

layers: Add VU checks for dedicated buffers/images

Add validation for the following valid usage for binding dedicated
allocations to buffers and images.

VALIDATION_ERROR_17000bc8    VUID-vkBindBufferMemory-memory-01508
VALIDATION_ERROR_17400bca    VUID-vkBindImageMemory-memory-01509

The same code path checks the BindImage2/BufferMemory2, but there are no
matching VUID at this time.

Change-Id: I8b700059d8c0acdfabf0b30be3dca20f616b3be1

macOS: Add macOS support

layers:Add VUIDs for VK_KHR_incremental_present

Ran across this old TODO when cleaning up branches. Add in the
appropriate VUIDs, update DB, and kill TODO comment.

build: Fix -Wmicrosoft-cast warnings with clang-cl

MSVC allows implicit conversion from function pointers to void*.
The C++ standard doesn't allow this, and clang-cl emits a warning
when this is done.  This change fixes these warnings in
Vulkan-LoaderAndValidationLayers.

Arguably, loader_platform_get_proc_address() should return a
FARPROC so that the compiler can help checking calling conventions,
but this is the simplest fix that unblocks us.

This is needed for https://crbug.com/550065

tests:  Add tests for vkCmdSetScissor

+ increase coverage of `VkRect2D` tests.

layers: Rewrite pv_vkCmdSetScissor checks

- tweaks to completeness, correctness and quality of msg logs
- add `multiViewport` related check
- protect against `NULL`  ptr `pScissors`
- make overflow check more robust (remove u32->i32 cast)
- correct veid db

tests: Add missing dyn. multiVieport tests in db

layers: Conform to gstyle in pv_vkCmdSetVieport

layers: Make unique_objects ID map global

Increasingly, instance objects are referenced by device-level APIs.
Made unique ID map global, shared across all instances and devices.

Change-Id: I5b62e7b95ec25a2a8b390d46024b43c29f2199ce

scripts: Mod spec.py to use local validusage.json

This script by default used the latest version of validusage.json from
the web. Changed to use the local copy checked into the repo. Used the
json-file option to fetch from a non-default location, removed the
-json arg and the URL option entirely.

Change-Id: Id088b08c8e66a3cdfca2b9e771797bffe941e5d3

scripts: Add validusage.json to LVL repo

In the future, this file will be updated as part of the Vulkan
header update process, and will be used as part of managing VUID
identifiers and lifetimes.

Change-Id: I6e29c1148e8752147310649a5ccfd99f385c0f46