repo: Remove interim REPO_ROOT CMake code

And a few other misc cleanups

cmake: Fix macOS CMake files to include dylib file

Fixed CMake files for macOS to include the vulkan dylib loader
file in the bundles instead of the vulkan framework file.

Change-Id: Ib59873fc392c8894d8a85b5a10bc0f17aa29061a

android: Fix CI for Android

repo: Reorg to remove Vulkan-Headers submodule

- Remove Vulkan-Headers submodule
- Add CMake code to add VULKAN_HEADERS_INSTALL_DIR and
  VULKAN_LOADER_INSTALL_DIR to search when locating
  Vulkan headers and loader
- CI adjustments for new model
- BUILD.md updates
- Travis-CI Android build temporarily disabled

Fixes #20

ci: fix travis syntax bug

build: Fix CI for updated Vulkan-Loader

Adapt to Vulkan-Loader getting rid of Vulkan-Headers submodule.
Get and build Vulkan-Headers and fix Vulkan-Loader build to use it.

cube: Fix separate present queue validation errors

From the spec:

Whilst it is not invalid to provide destination or source access
masks for memory barriers used for release or acquire operations,
respectively, they have no practical effect. Access after a release
operation has undefined results, and so visibility for those accesses
has no practical effect. Similarly, write access before an acquire
operation will produce undefined results for future access,
so availability of those writes has no practical use. In an earlier
version of the specification, these were required to match on both
sides - but this was subsequently relaxed.
These masks should be set to 0.

Change-Id: I495dc86ad62c0651fbc6acbfb0dfbb8245a324be

docs: Update build doc

Change-Id: I8764e470fe11646fc13cc62e73761c09e2779f6a

cube: Explicitly set QueueFamilyIndex(s) to QUEUE_FAMILY_IGNORED

The QueueFamilyIndex fields only come into play when you want
to use an image in two or more queues from different queue
families, which cube does not do.

Change-Id: I610aa8899539eeb25ca06254b88e6a6d0a2ffc97

build: Add cube(pp) to install target

- Modify cube and cubepp to obtain the texture image data from
  an include file instead of reading a PPM file at runtime.
  (This is the way it works for Android)
  This removes the need to install an image file or otherwise
  make an image file available in a repo build.
- Add cube and cubepp to the install target.

Note: The file handling code is left in place to make it easy
to add a "-texture_file <file>" option so a user can pass in
a texture file.

Fixes #5

cube: Remove last DebugReport

We still had the debug report detection code in cube.  This removes
the last bits of it.

Change-Id: I09559cd56ee4779453221c810d8328cb4c864aef

cube: Add missing free in IPD code

Fixes #11

cmake: Add WSI Build Support Options

Fixed CMakelists.txt for vulkaninfo
to generate build files on
Linux with the selected WSI options.

Change-Id: Ied83f0e24dbd08af2ef03066fdfa1443203aa4ad

build: Enable install targets on Windows

Note: cube is not yet being installed.

icd: Add a memory format that is unsupported

Hard code into the mock ICD that the memory type at index 3 is
unsupported. This allows the BindImageInvalidMemoryType test in the
validation layers to be run without skipping.

icd: Hard code unsupported image formats

Hard code that the mock ICD does not support
VK_FORMAT_E5B9G9R9_UFLOAT_PACK32, an arbitrary image format not
currently used in any validation layers test. Additionally hard code
that the mock ICD doesn't images created with a sample count of 64 bits.
These hardcoded unsupported image formats git the
CreateImageFormatSupportErrors and CreateImageMaxLimitsViolation tests
in the validation layers to run.

icd: Add external buffers, fences, and semaphores

Added calls to vkGetPhysicalDeviceExternal buffer, fence, and semaphore
properties that support for features and handle types.

build: Fix mock ICD install issues

- Use correct (icd.d) directory
- Make ICD installation OFF by default
- Fix up docs

Fixes #17

header: Update Vulkan-Headers submodule to 1.1.75

Change-Id: I7cfbb6e3ffb99308f3a77357f6184489aae322af

icd: Enable the VK_KHR_push_descriptor extension

Enabled this extension and updated vkGetPhysicalDeviceProperties2KHR to
fill out the push descriptor properties struct if it's in the pNext
chain.

cmake: Fix LOADER_REPO_ROOT pointing to SDK

scripts: Fix glslang fetch script for odd builds

Used scirpt location instead of current location as basis for glslang
component installation.

Tools: Better search for glslang + loader

Change-Id: I1652a5bc29623980e80dad168997fd75d08467e8

cube: Change glslang location logic for INSTALL_DIR

Now respects GLSLANG_INSTALL_DIR, for instance, if Vulkan-Tools is a
submodule of another repository. Also updated build doc.

cmake: Rename uninstall target to prevent collisions

Cmake does not have an uninstall function, and multiple repositories
cannot have targets with the same name. The 'uninstall' target is to
generic, so renaming this target to 'uninstall-Vulkan-Tools'.

docs: Update BUILD.md to reflect CMake changes

Updated BUILD.md to reflect the changes in the
MacOS build workflow after the repo split.

Change-Id: Iff5f89c960b73fec0d1f0b08c7aa448da4873384

cmake: Update CMake files to build for MacOS

Updated CMake scripts to add building for MacOS

Change-Id: I413f79e337d33f37891092da97380597e28c8257

docs: Clarify loader build requirements

cmake: Change name of helper-file cmake target

Cmake won't tolerate multiple build targets withing a project.
Generate_helper_files has been renamed to resolve conflicts with
other repos.

docs: Update build.md for glslang location

cube: Download glslangValidator binary

Obviates the need to set GLSLANG_REPO_ROOT or to have glslang
otherwise installed for building this repo.

android: Update vulkan_wrapper, add source script

* Finally check in the script that generates the wrapper
* Update the script to handle split vulkan files
* Update the checked in vulkan_wrapper.*
* Add a README descrbing how to use the script

android: Fixes for cube

This restores files that were dropped during the split.

Also migrates support files to this repo, which we will remove
from Vulkan-ValidationLayers and treat like a submodule.

Updates to travis to build cube.

build: Enable install target on Linux

Now installs vulkaninfo and mock ICD.
Not yet installing cube(pp) pending decision on issue #5.

docs: Update build.md w/loader repo root info

docs: More BUILD.md updates

Added some TODOs.

repo: Updated .gitignore file for new structure

vulkaninfo: Moved new macOS files into new location

repo: Turn on AppVeyor CI

repo: Turn on Travis-CI

Add test to run vulkaninfo with mock ICD.

cmake: Fix GLSLANG_REPO_ROOT for Linux

cmake: Use GLSLANG_REPO_ROOT to find glslangValidator

cmake: Add FindVulkan.cmake for older cmake installs

cmake: Add LOADER_REPO_ROOT to repo

Also added in warning if git submodule update --init is not run.

repo: Fix git submodules and AppVayor badge

scripts: Add helper file generator back in for icd

Mock ICD now uses the typemap helper, added the helper file generator
back in, along with associated lvl_genvk mods.

vulkaninfo: Fix some merge conflicts

build: Android mods to remove layers

docs: Updated markdown files for new repo content

repo: Delete include directory and subdirs

headers: Switched cmake files to Vulkan-Headers

headers: Added vulkan-headers as a submodule

vulkaninfo: Move vulkaninfo.rc file

Change-Id: Ic974500a0334a9de0bb7317f6aa77693cfa36869

repo: Delete unused files

Change-Id: I4a8985b087ac999fdcebfc61c812ff0c1f6b91bf

cmake: More cmake fixups

Removed duplicate vulkaninfo directives, renamed project, killed
some dead comments.

Change-Id: Ibf580ac0fb5f8ebec0a68fe9c36f7905dbccc594

repo: Delete unused code generators

Change-Id: I1ef619a117c6ec9c2befe0458eb8502894ecff6c

repo: Delete smoketest

Change-Id: I44b5b0c5e54a1fdbbfbce6b3ebb4788f0aa5871c

repo: Delete glslang files

Change-Id: I42e66cf9dc4a4db7a86bc2f29086fad532b9f629

repo: Delete gtest submodule

Change-Id: Ib366f7550753cc2de3a65285e2a79afec48200a0

repo: Delete loader, layers, tests files

Change-Id: I28b8833df82946457325353cc03d3f91e6a776cf

android: Build cube for android

Change-Id: I0ff87c345098d51a24115bbb33cbb60a488efb45

cmake: Clean up main cmakelists.txt file

Delete unnecessary cmake, move non-common stuff to consitituent cmake
files.

Change-Id: I158c50511a2b9810f6b2fa198d4d8aee5a23ac93

cube: Move cube directory up to top level dir

Change-Id: Ie1116b0b6de742b4f85834a4c9d1927dd641e19b

vulkaninfo: Move vulkaninfo from demos to top level

Change-Id: Ifb946c4eacd950f8c021399b87e9437edf5533bf

cmake: Remove building and setup of glslang

Change-Id: If982a9c517bf2a180ba2d1cf853d49f46412b13d

scripts: Remove generators from lvl_genvk.py

Also remove unneeded include from common_codegen.py.

Change-Id: I1b6833acb5654217d85bbc4284713aa12fa1a104

demos: Remove smoketest demo from build

Change-Id: I2a958901a8ac84b782a62b80245999725f4cff3e

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