build: Add FindVulkan, disable layers, ldr, tsts

Added FindVulkan cmake helper, disabled build of layers, loader,
and tests in cmake file.

Change-Id: I30a65e3a821c58b36971298ea6f7c7515cbda444

layers: Add object type string helper for cube

Change-Id: I4233528c4fa60a7729a8c2985e3b2a1097234ae3

icd:Add GPDP2 support for descriptor indexing

Set some default mock icd values for vkGetPhysicalDeviceProperties2
query of VkPhysicalDeviceDescriptorIndexingPropertiesEXT props.

layers:Moving to GPDF2

Update layer_data struct to track safe_VkPhysicalDeviceFeatures2 struct
instead of VkPhysicalDeviceFeatures. Currently just doing previous
checks using the VkPhysicalDeviceFeatures member of GPDF2 struct. Will
follow-on with checking against extension structs.

layers:Store GPDF2 struct

In PHYSICAL_DEVICE_STATE track safe_VkPhysicalDeviceFeatures2 instead
of the old VkPhysicalDeviceFeatures. Initially just passing the
VkPhysicalDeviceFeatures of VkPhysicalDeviceFeatures2 wherever this was
used. This is laying groundwork to start using/comparing extended
features in core validation.

icd:Fix Windows build

Make sure we generate vk_typemap_helper.h for mock icd.

layers: add missing VkFormatFeatureFlags checks

Checks for TRANSFER_SRC / TRANSFER_DST formats were added to:
- vkCmdClearColorImage
- vkCmdCopyImage
- vkCmdResolveImage
- vkCmdCopyImageToBuffer
- vkCmdCopyBufferToImage
These checks require VK_KHR_maintenance1 or Vulkan 1.1.

Existing checks were adapted to use the new helper function:
- vkCmdClearDepthStencilImage
- vkCmdBlitImage

layers: Add GetApiVersion() helper

layers: fix imageview 3D layout transition

Fix an oversight when recording a 3D image subresource layout transition
that is set by way of a 2D or 2D_ARRAY image view.

Change-Id: Ifbf7645fb2705fef6f82030f4e3e1da0e9da2f2c

demos: Add MacOS surface output to vulkaninfo

Added output for a MacOSSurface in vulkaninfo
and modified the CMake files to include the
new files needed for this change:

metal_view.m Objective-C code to hook into Metal
metal_view.h metal_view.m's header file

Change-Id: Ib7f9185a86703b2a300b2ce3be46f130c90d1b6b

icd:Add support for extended features queries

Update mock ICD generator so that if we have an extension that's been
promoted to core, the core version will call any existing custom
implementation of the extension function.

Update Features query to detect the two existing extension structs
(VkPhysicalDeviceDescriptorIndexingFeaturesEXT and
VkPhysicalDeviceBlendOperationAdvancedFeaturesEXT) and enable all of
those features when those queries are present.

loader: Fix gcc 8.1 cast warning

Change-Id: Id8f0301c4f208d185abe2cc3df71021cb4aa29b1

loader: Fix unhandled out of memory errors

Change-Id: I3b672a9d54e1509b550407f1b4ffae58b48beb85

loader: Remove duplicated delimiting chars in paths

Before loading a manifest file, its full path is strcmp() with already
loaded ones in order not to load the same file twice. While being simple
and efficent, this mechanism is not able to detect subtle duplicates
like these:

/usr/share/vulkan/icd.d/intel_icd.x86_64.json
/usr/share//vulkan/icd.d/intel_icd.x86_64.json

This patch ensure that searched paths do not contains such duplicated
directory delimiting characters in order to avoid this kind of problem.

Fixes #2331
Fixes #2629

tests: Fix min image transfer granularity test

This test as written was guaranteed to be skipped. It would skip the
test if the first graphics queue family found had a
minImageTransferGranularity with a width, height, or depth of less
than 4. The spec guarantees that any graphics queue family's min
transfer granularity will be 1 for the width, height, and depth. The
rewritten test is now only skipped if no queue supporting copy
operations is found with a large enough granularity.

Also the usage flags on the src and dst image needed to be fixed.
Additionally, the extent of the copies dst and src images were changed
to always be multiples of the min transfer granularity.

Finally, I added a device profile that has a queue with a large enough
min transfer granularity to not skip this test.

layers: Rework attachment use conflict checks

Previously buffer_validation covered part of this, but not everything.

V2: - Fix repeated ATTACHMENT_RESOLVE where I meant ATTACHMENT_PRESERVE
    - Fix camelcase noise in a bunch of places
    - Rename VkAttachmentReference& variables to attachment_ref to avoid
      attachment.attachment etc.
    - clang-format tidy-up
    - Move positive case to VkPositiveLayerTest

layers: Do renderpass dag validation before call

Previously this would contribute to skip /after/ the call, which
accomplishes nothing.

demos: Unify capitlization in json output

Change-Id: Ie74c686af656e346789b30ec1aba41457170c6d2

demos: Add help options to vulkaninfo

Change-Id: I43886920e5ce1feb2e293ed46a3e8c23a217ca69

layers: fix a multiplane imageview bug

Change-Id: If11059dca9716a90134d9d362351634fa07a8ecd

layers: Track display and display mode objects

Add VkDisplayKHR and VkDisplayModeKHR tracking to
VK_LAYER_LUNARG_object_tracker.

Change-Id: I63004cfe734793593bea12700ea9cd1bdefeab0d

layers: image transfer granularity check fixes

- Use appropriate VUIDs for all checks
- Update tests to check for VUIDs instead of message substrings
- Image granularity tests should be also be run on compressed images.
  They're *mostly* redundant with existing compressed-format-only checks,
  but I can't convince myself that they're *always* redundant, and the
  spec doesn't say to skip them.
- Removed comments that the checks should be moved elsewhere; IMO they're
  fine where they are, but I welcome second opinions.
- Compressed-format alignment checks should also apply to _422 single-plane
  formats.

layers: Fix down-chain null funcptr crashes

Modified dispatch table helper to replace null device extension
function pointers with an appropriate stub function.

Change-Id: If5e22333f084d3f41c0eefdf3e82c9060b872e8c

demos: Fix cube ActualTimeLate function

The ActualTimeLate function in cube.c is supposed to check if a
VkPastPresentationTimingGOOGLE structure indicates that the
presentation happened within a refresh-cycle duration of the
desired presentation time.

However currently ActualTimeLate will always return false, because
it is calculating the presentation deadline as `actual + rdur`.
This change calculated the presentation deadline as
`desired + rdur`.

Reviewed-by: Andy Ritger <aritger@nvidia.com>

tests: Add tests for indexed draw checking

New tests to exercise new checks for indexed draw count and offset.

Add VkRenderFramework::DestroyRenderTarget() and call it from
VkTriangleTest so that VkTriangleTest cleans up after itself,
allowing it to be called multiple times within a single framework
instantiation.

Introduce VkBufferObj as a typedef for vk_testing::Buffer.

Fixes #2448

Change-Id: Iab513c7f50ef390c210ea76e7ef3f67d7e3d40f8

layers: Add checks for indexed draw count/offset

Check that the first index and index count are within the bounds
of an index buffer for DrawIndexed, taking into account any
offset specified in the binding of the index buffer.

Fixes #2448

Change-Id: I47a1acbf0bb0e586692c75380d8d43a1c72450fd

layers: remove duplicate GetEnabledExtensions() function

tests: Clean up MapMemWithoutHostVisibleBit test

Simplified this test by removing an uneccessary creation of a VkImage.
Instead, the test now just directly allocates unmappable memory and
attempts to map it.

layers: Remove Fill Memory warnings

Implementation was incomplete, and had been reduced to producing
non-specified warnings.

Change-Id: I34a4bbf1f572f337b5f9f9316e9717f52645d8dc

layers: Fix segv during DebugReportCallback setup

Change debug log message to use correct VkObjectType to
VkDebugReportObjectTypeEXT method, avoiding the index out-of-bounds
lookup that caused a SEGV.

Change-Id: Ibb6e485eccb36ff0e9b49f0722d8f2d0bf660982

layers: PreRecord Free/Destroy calls

To avoid Destroy/Create and Free/Alloc race conditions (when a driver
returns a destroyed(freed) handle between the PreCall and PostCall
phases of a validation intercept routine, PostCallRecord functionality
has been moved to PreCall.  This affects the following calls

FreeMemory
DestroyFence
DestroySemaphore
DestroyEvent
DestroyQueryPool
DestroyBuffer
DestroyBufferView
DestroyImage
DestroyImageView
DestroyPipeline
DestroySampler
DestroyDescriptorSetLayout
DestroyFramebuffer
DestroyRenderPass
FreeDescriptorSets

Did not alter the destroy functions for Instance, Device, or the Debug
or layer extensions.

Change-Id: I5133982eaa1a0c94283fa922d9c3bcc90b5a6223

core validation: extend CLEAR LOAD_OP check to new DS image layouts

Extend check for load operation vs first time use image layout to
new read-only depth stencil image layouts.

The new layouts were added in VK_KHR_maintenance2 / Vulkan 1.1
and follow similar restrictions to VK_IMAGE_LAYOUT_DEPTH_STENCIL_READ_ONLY_OPTIMAL.

Spec:

"
For any member of pAttachments with a loadOp equal to VK_ATTACHMENT_LOAD_OP_CLEAR,
 the first use of that attachment must not specify a layout equal to
VK_IMAGE_LAYOUT_DEPTH_READ_ONLY_STENCIL_ATTACHMENT_OPTIMAL.

For any member of pAttachments with a stencilLoadOp equal to VK_ATTACHMENT_LOAD_OP_CLEAR,
the first use of that attachment must not specify a layout equal to
VK_IMAGE_LAYOUT_DEPTH_ATTACHMENT_STENCIL_READ_ONLY_OPTIMAL.
"

tests: test behavior when dualSrcBlend feature is disabled

layers: validate dual-source-blend factors

Fixes #2614

Fixed the following issues with the previous checks:

- Only alpha blend factors were checked
- The checks did not use the individual VUIDs for
  each blend factor field
- The check was being run at pipeline bind time,
  not pipeline creation time

layers: Install into the data root directory

The layers files fit are not really configuration files and are rarely
edited by users. So, let's move the install location into the
data root directory.

Signed-off-by: Nick Sarnie <sarnex@gentoo.org>

layers: Add missing promoted device extensions

Add three missing device extensions to promoted device extensions list
in vk_extension_helper codegen.

    VK_KHR_relaxed_block_layout
    VK_KHR_shader_draw_parameters
    VK_KHR_storage_buffer_storage_class

Change-Id: I392155d318ca580e576436ef46695218e5eb446e

layers: Sort promoted extensions list

For ease of maintenance, sort the promoted extension lists.

Change-Id: Ib1d690d55c6b9c3a156c9682fbe96bbebed17c14

layers: multiplane view format compatibility

Add check for compatible format when creating an imageview of a
plane of a multiplane image.

Add test to exercise.

Change-Id: Id3388aa49b42a16e84dc1d459121ee908b2e5f0f

demos: Sort most vkinfo output alphanumerically

Sections left unsorted: Presentable Surfaces, VkQueueFamilyProperties,
and Format Properties

Change-Id: I075a6897c3070f70d7073c22e7100c49c19073a3

layers: Implement extension dependency validation

Add CreateInstance and CreateDevice validation checks for extension
dependencies.

VALIDATION_ERROR_1fc00ad6
   VUID-vkCreateDevice-ppEnabledExtensionNames-01387
VALIDATION_ERROR_21200ad8
   VUID-vkCreateInstance-ppEnabledExtensionNames-01388

Change-Id: Ia78f15494f10a1eca6a2a33e3a22d769b5d38785

layers: Add string_join utility

Added utility modelled on script languague "join" command for
concatenating collections of strings (of various types).

Change-Id: Ia6ee7a144e7abc49a3356e9e20802706191947c1

layers: Add dependency info to vk_extension_helper

Add additional metadata to *Extensions structs to enable extension
dependency validation at CreateInstance CreateDevice time.

Refactor the code generator for the extension helper prior to
refactoring the generated code to address an incomplete.

Change-Id: I79ae680d9c41f1f153ee8108d4deac8ec8c5a886

layers: Refactor codegen for vk_extension_helper

Refactor vk_extension helper prior to adding extension dependency
information to code gen. Refactor to simplify feature addition.

Change-Id: I7b2902dd1a4070c83810f57c0836b50cb6ecfb86

loader: Eliminate warning for uninitialized value.

Added default value to severity to kill a Linux warning.

Change-Id: Iaf3e62a0fa01d74874aaec4951f9a2aa38479ca0

layers: Sort extension order for extension helper

Sorted the extensions by name s.t. the output of the vk_extension_helper
generator would be consistent run-to-run.

Change-Id: Ief3d822b91260e469f7f2d563076d8a79f1a1bf0

Image bind count is associated with pImageBinds rather than opaque binds

header: Update to version 1.1.74 of Vulkan hdr

- update layer json files
- update vulkan_core.h
- update vk.xml
- update vulkan.hpp
- update validusage.json
- update vk_validation_error_messages.h
- update vk_validation_error_database.txt
- maintain local changes to reg.py

Change-Id: If0c52d13f78ddfcc7c9d316386c0d20399e97668

layers: Add handling for NULL in core post call

Added handling in
PostCallRecordGetImageSparseMemoryRequirements2
when a nullptr is passed to 'reqs'

Change-Id: I444ec733d4a172329aceb0fcbb68568700b8f535

layers: Fixes from PR #2566

layers: Add VALIDATION_ERROR_32a00272

This adds VALIDATION_ERROR_32a00272 to core validation layer to catch
reset of descriptor pool in-flight. Also adding an accompanying test
case and an update of the validation layer database to reflect new
validation and test.

build(win): Add dep to create dir before use

Add a CMake dependency for Windows builds to create
the layers/$CONFIG directory before running any of the
other steps that copy the layer json files to this directory.

This failure was sporadic and often did not fail because
other build steps that compiled the layers often created this
directory before the copy steps executed.

Because of the difficulty in reproducing this problem, it is not
certain that this fixes the problem.  However, the mac Xcode
builds were already using this pattern to achieve the same goal.

layers: Validate against nonCoherentAtomSize

FlushMappedMemoryRanges and InvalidateMappedMemoryRanges check
size and offset against nonCoherentAtomSize.
Github issues 329 & 2584
Gitlab Issue 611

Change-Id: I4cbc6c21f6fe4b6187b381b2565a90c61417bee5

loader: Fix a 1.0/1.1 versioning bug

Fix a bug where when mixing a 1.0 and a 1.1 driver, if the loader
stores the 1.0 driver first, the apiVersion field in vkCreateInstance
will be rewritten to 1.0 for both drivers. Now it will only be
changed for the 1.0 driver, as intended.

Change-Id: I9e7c5e0b0b7c0eb660082319dcddd9e8ef8adc61

scripts: Add spec.py check for ext changes

When the surrounding asciidoctor conditionals of a VUID change it may
impact validation even if the VUID text does not change.  Detect this in
spec.py and flag for manual intervention.

Change-Id: Idf802955ba9bbfe911edd4114fb28bdddf5fcc9b

demos: Enable all required exts for present caps

vkGetDeviceGroupPresentCapabilitiesKHR requires VK_KHR_SURFACE and VK_KHR_SWAPCHAIN
in addition to VK_KHR_DEVICE_GROUP

loader: Clarify loader warning

Clarify several loader warnings about layers so that they include
the name of the layer that is causing an issue.

Change-Id: Ie48d73f6cc8466c3114597831fef451b3ab32bf7

macos: Specify deployment target version

This allows using the build on macOS 10.12 even if built on 10.13.

Change-Id: I21939163551bb9a0d0088513a23dc5d2cf7f5206

scripts:Account for extnumber key to fix warnings

With the addition of the 'extnumber' key update comparison to skip over
the extnumber as well as extname in order to avoid a bunch of
irrelevant warnings.

demos:Add const to fix build warning

Explicit "const" for ppEnabledExtensionNames to fix build warning.

fixup

loader:Fix build warnings

Add "fall through" comments on intentional fall through switch cases to
avoid compiler warnings.

layers:Capture rect by value to validate 2nd cb

Fixes #2587

The lambda function was only capturing the pRect pointer by value so
the underlying rect value could still be changed between when the
command was submitted to a secondary CB and when the rect was validated
at CmdExecuteCommands() time.

Copy the actual rect value so that it can't change.

layers: Validate subpass dependency access vs stage masks

demos: Remove undesirable gpu_count assert

to allow the useful error message to be shown instead

layers:Update comments to avoid compiler warnings

The simple "// fall through" comment causes the compiler to omit a
warning.

loader: Bump loader copyright to 2018

Change-Id: I59b80b7f4bff37ac15fd30acc3f4600164afee90

demos: Add vulkaninfo.rc file

Add an rc file for vulkaninfo, which allows the version number and
copyright to be encoded into the file. This is necessary to support
new runtime installer changes.

Change-Id: I95a1b4338e3071162cca028d75b7dbd5a35d0393

externals: Update to latest glslang

Change-Id: I06c15a028941fc60bd6ddcd830a8401d83193753

vulkaninfo: Fix array length skew

Added new struct pNextChainBuildingBlockInfo
to store sType and memory size of structs for
function buildpNextChain.

Change-Id: Ia49084d2e303336f0305a68303edaaf60f74d669

vulkaninfo: Add core 1.1 extensions

Added support for the following extensions:

VK_KHR_multiview
VK_KHR_device_group
VK_KHR_device_group_creation

Change-Id: I103fd988518339f95f0eeb7e7408ef49dec8ec22

macos: Pass argc/argv through to main cube code

This allows users to specify arguments like "--c 100"

header: Update to version 1.1.73 of the Vulkan hdr

- update json files
- update vk.xml
- update vulkan_core.h
- update validusage.json
- update vk_validation_error_messages.h
- update vk_validation_error_database.txt
- update vulkan.hpp
- removed checks for the following deprecated VUIDs related to
  ImageCreateInfo and FormatProperties:
  - VALIDATION_ERROR_09e007ae
  - VALIDATION_ERROR_09e007b4
  - VALIDATION_ERROR_09e007ac
  - VALIDATION_ERROR_09e007a6
  - VALIDATION_ERROR_09e007b2
  - VALIDATION_ERROR_09e007a4
  - VALIDATION_ERROR_09e007aa
  - VALIDATION_ERROR_09e007a8
  - VALIDATION_ERROR_09e007a2
  - VALIDATION_ERROR_09e007b0
- modified CreateImageFormatSupportErrors test to remove killed checks
- modified ImageLayerUnsupportedFormat test to remove killed check

Change-Id: I8f1577747d87fdeb6f68b9db0340443a83c44630

layers: fix PR2567 formatting errors

Change-Id: Id7c479d7a2421c468990125dcfe815363a440a39

layers: add setIndex to log_msg output in ValidateDrawState()

icd:Add new physical device query funcs to ICD

Add mock ICD support for vkGetPhysicalDeviceSurfacePresentModesKHR,
vkGetPhysicalDeviceSurfaceFormats[2]KHR,
vkGetPhysicalDeviceSurfaceSupportKHR,
vkGetPhysicalDeviceSurfaceCapabilities[2]KHR,
vkGetSwapchainImagesKHR and
vkAcquireNextImagesKHR.

tests: Fix typos in layer_validation_tests.cpp

Includes fixing a corresponding typo in a descriptor_sets log_msg call.

Change-Id: Ib876482357ff2b945fe411fbef68d421fdbfc802

Don't clobber immutable samplers in update

The copy cases were already doing the right thing; the basic updates
missed the check for the sampler being immutable.

Fixes #2569

layers: Add support for VK_EXT_descriptor_indexing

Change-Id: Ieeaf9cbdff1c9b383d4f47ef678fe4a7467113cb

android: Add GetPhysDevFeatures2 to vulkan_wrapper

Change-Id: I186def1470aa0423f9bde273860df3c01bf1d4f4

travis: Avoid empty Travis CI job

Change-Id: Ifb917ab3cf72f0c89f3c7bc9b647156bf1259a50

layers: maint2 image usage bit validation

Add partial support for VK_IMAGE_CREATE_EXTENDED_USAGE_BIT in
vkCreateImage(), currently just avoiding the usage bit checks when
EXTENDED_USAGE is specified. Full support will require an
exhaustive search of all compatible format's usage bits, which
seems pretty expensive. TODO for now.

Adds support for checking for a chained VkImageViewUsageCreateInfo
struct when doing usage bit validation in vkCreateImageView().

Change-Id: I79a5853dc93dfa54040ace162170f1226a978c2d

demos: Remove unused var from vulkaninfo

Change-Id: I31c161117ec773240df006cac4e06dffff05c7c6

travis: Set up to test commit messages

Change-Id: I64adcdb59329eed77cd7c0dd4ff89362f73f72f4

scripts: Add commit message format checker

Change-Id: I94efe0326e3f68e19af605a1ff381da59d2d7d1d

layers: use VUID in ValidateImageAspectLayout log msg

This commit also updates the corresponding test to expect
the VUID instead of a log message snippet.

layers: add level/layer to ValidateImageAspectLayout log msg

This helps clarify the case where a transition fails due to a
specific subresource being in an unexpected layout.

Fix missing validation errors when a CB never bound any descriptor sets

In this case, the last bound state would never be properly configured
and so we'd skip over this whole block.

macos: Remove update_and_draw from cube demos

Adopting this change from MoltenVK.  The wait and update steps
are no longer needed.  Calling just demo_draw now works as well
on MVK as it does on other platforms.

layers: multiplane image allow color aspect bit

There is a case (memory barrier) where a multiplane image can have
the color aspect bit set, so the general aspect check should allow
it. In the common case where only VK_IMAGE_ASPECT_PLANE_n_BIT_KHR
bits are allowed, there are specific VUIDs in place to enforce.

This fixes an "unexpected error" result that recently appeared for
the CopyImageMultiplaneAspectBits test.

Change-Id: I766da2940955d8a7b72ec26e4dbefe9e60eb0b08

scripts: exclude vulkan headers from clang-format

Travis CI shows linux build failure on every header update because
upstream Vulkan-Docs files are not formatted with the clang-format style
of this repo.  This change excludes the vulkan headers directory from
the CI clang-format check.

Change-Id: I6d1bee486d06b690f65aa2c3098de14f3c5931c9

header: Update to version 1.1.72 of the Vulkan hdr

- updated layer json files
- updated vulkan_core.h
- updated validusage.json
- updated vk_validation_error_database.txt
- updated vk_validation_error_messages.h
- updated vuid_mapping.py
- updated vk.xml
- updated vulkan.hpp
- moved vuid 1ae00652 to 1ae00f06

Change-Id: Id6ba342841cca04c0669b7feb39d82e2b8052443

layers: Add text-string VUID version of log_msg

Allows adding VUIDs for checks that are not yet public, or that do
not yet have numerical VUIDs assigned.

Change-Id: I9f3a92d8e085f20452a52439517e7cdf677631d3

layers: Add text vuid error message map

Change-Id: I71efc6127960a8fdff16a8279201f7e59be73bf3

scripts: Output vuid-string->message map

Change-Id: Id4123053da62fb8d2b4d32393d11e4aa6c030f92

layers: fix typos

Change-Id: I93a01dfe0e7f49ba3f6992a5f6c8d312b524774a

tests: Unit tests for CreateDescriptorUpdateTempla

Added unit test for vkCreateDescriptorUpdateTemplate(KHR) validation
checks.

VALIDATION_ERROR_052002bc
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00350
VALIDATION_ERROR_052002be
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00351
VALIDATION_ERROR_052002c0
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00352
VALIDATION_ERROR_052002c2
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00353

Change-Id: I55bebc50757dd537f0be849aec38120fae41e181

layers: Add checks for CreateDescriptorUpdateTempl

Added four validation checks for vkCreateDescriptorUpdateTemplate(KHR)
and removed a false positive.

VALIDATION_ERROR_052002bc
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00350
VALIDATION_ERROR_052002be
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00351
VALIDATION_ERROR_052002c0
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00352
VALIDATION_ERROR_052002c2
    VUID-VkDescriptorUpdateTemplateCreateInfo-templateType-00353

Updated autogen of object tracker s.t. that logic for noautovalidation
handles matches parameter validation, and spec generation, eliminating a
source of false positives.

Change-Id: Ib831eabd45a5083168355bca6b0e2beda3a76222

docs:Fix my name spelling in README

macos: Update vulkan framework version to 1.1

layers: Avoid reading pointer to array when count is zero

Take for example VkSubmitInfo. It has, amongst others, two fields:
waitSemaphoreCount and pWaitSemaphores. The specification states that
if waitSemaphoreCount is zero, the application may leave pWaitSemaphores
uninitialized (in essence, that field is ignored).

The layers read that value anyway, triggering uninitialized read errors
by memory sanitization tools.

Fixes uninitialized read in QueueSubmitSemaphoresAndLayoutTracking.

layers: Destroy object tracking after reporting undestroyed objects

Fixes a memleak in VkLayerTest.LeakAnObject.