scripts: Update comments in helper generator

Change-Id: Ie4171bb18ff499e54bdbd8ff7237e7026e6cf3e9

scripts: Complete safe_struct header generation

Change-Id: I87de80173ff414202f6e876db3da688466f3598d

scripts: Remove blank lines from end of helper files

Change-Id: Ie972f2d8b6bca633fbfb4745f29eab183cde7acb

scripts: Stub in safe_struct generator source

Change-Id: I060e8bb0eddda0c94bdaed7e04fddac8bf9d5822

scripts: Add generator options for safestruct files

Change-Id: Iffd19369976ac15ad54251e15a7f49e061a807e8

layers: Remove lock held across down-chain call

Unnecessary lock was grabbed just before an API call.

Change-Id: I0ef561d9b620f42333ba034dcbc87cba6daea776

layers: Reset Valid Usage database check status

Following the audit of existing Valid Usage enums in the LVL codebase,
we can claim that there are no Valid Usage checks in an unknown
state; the existing VU checks are either implemented or not.

This commit resets the check_implemented flags in the VU database,
such that any remaining "U" (Unknown) entries now become "N" (Not
implemented).

Going forward, the check_implemented status flag must be either "Y"
or "N"; "U" is no longer permitted.

Change-Id: Iaf1807cadf02d57e3fa5676108e290d3bab9b271

android: Fix typos in android-generate script

Change-Id: I31c32e56ae09134b7b91c610b546a9d46c20c007

build: Update travis config to catch build errors on intermediate steps

layers: Transition each aspect individually in TransitionImageLayouts

This fixes the possibility of spurious validation errors like the following,
when attempting to transition multiple aspects of an image after only a subset
of those aspects have previously been used in the command buffer:

  "Cannot query for VkImage 0x599 layout when combined aspect mask 6 has
   multiple initial layout types: VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL and
   VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL"

The specific case where this bug was encountered was where both the depth and
stencil aspects of an image were initially in the TRANSFER_DST_OPTIMAL layout.
The first command in the command buffer to reference the image was a
vkCmdClearDepthStencilImage on only the depth aspect of the image, followed
later by a vkCmdPipelineBarrier to transition both aspects to the
DEPTH_STENCIL_ATTACHMENT_OPTIMAL layout.

Since TransitionImageLayouts tries to look up the initial state for all
aspects at the same time, it was picking up the state based on the depth
aspect because of the previous use of that (which had both initialLayout
and layout set to TRANSFER_DST_OPTIMAL), and then calling SetLayout for all
aspects with only the new layout. Since the stencil aspect didn't have any
currently recorded state, it was being added with both initialLayout and
layout set to the new layout, causing a mismatch between the initialLayout for
the two aspects and therefore the spurious error above.

Fix by updating the state for each aspect individually.

scripts: Remove unused code from vk_helper.py

Struct size helper files are now generated by the helper_file_generator
script.

Change-Id: Ifcb0cceb9621f3f4bb41d948e9f0bb3fca7a0ea1

android: Add codegen for struct_size_helpers

Change-Id: I264d7ecca323089bda1583dbcf9eb751736146a7

layers: Update build dependency target name

Changed enum_string_helper target to be generic for multiple files.

Change-Id: I760eb634839228211554686aa92c9771602786bb

layers: Change struct_size_helper to use new codegen

Remove old references, add calls to new generator.

Change-Id: I64ab859e4ae6893870697f16ee6fd596b8a7908b

layers: Remove unused header from core_validation

This header is unused by the module.

Change-Id: Ife7a1b8eb344e2a670e1cfc9ace3e0697c97cd7e

scripts: Handle vk.xml latexmath in helper files

The vk.xml file contains latex-formatted metadata which must be
parsed for codegen.

Change-Id: Ie64c9974371d2bb66ba460165a0c23505991ebd7

scripts: Include chain_size helper in struct_size

Change-Id: I1afcee1ab90ad18d758fdb476d2d0012a891bdfc

scripts: Complete struct_size_helper source gen

This file will now output a complete vk_struct_size_helper.c file.

Change-Id: Ic87fd7d3c9a95daa666e8e4da63a21d31e8bfb86

scripts: Add gen for struct_size_helper source file

First level code-gen, move some code into functions.

Change-Id: Idf8ee44a1f74c4117aed74b8df5b548df618e790

scripts: Add generation of struct_size_helper header

Added generator-side plumbing to create the vk_struct_size_helper.h
file.

Change-Id: I5ff226131b8db2d773e4422877bdb2993ed76cf4

scripts: Add struct_size options for source, header

Added struct processing guts for building the struct_size_helper
header and source files.

Change-Id: I8172b921b670b47b8fd66499de6d76ddb3ef934d

scripts: Mod helper generator for multiple files

Added parsing of option, conditional file generation to helper file
generator.

Change-Id: Ia7c51d50fb1785f3002dc2868f98d8e384d0e36e

scripts: Pass parameters to helper file generator

The helper file generator will be used to output multiple helper
files. Added passing of parameter and test call for this feature
to the generator options block.

Change-Id: I1e417ff50628f395c589282ed6f96f0340758f23

layers: Add valid usage enums to core_validation (VL-60)

Update valid usage enums in subsection of core validation from
validate_and_capture_pipeline_shader_state() to
checkGraphicsOrComputeBit() inclusive.

Change-Id: I8b71c63de26e42eb073a1eac1abd121d5e57f5c7

layers: Remove redundant code

This is already being checked in the object tracker layer.

Change-Id: Ie1d5776810c3f8a3df7b9ef920d02e90c0d239b4

layers: Update valid usage for Jira VL-59.

Change-Id: I9aba0480aad5b565eb59f642def7a11e46822566

layers: Update Valid Usage enums (VL-69)

Update Valid Usage enums in parameter_validation.cpp.

Update VU status in the database.

Completes Jira task VL-69.

Change-Id: Ie16c41dd0ad8f264b62b3d5b60d100d7b9b3f229

layers: Tag internal helper functions as static

Reduce the scope of internal helper functions by
declaring them static.

This is only changes lines related to VL-69,
rather than a full sweep of the code.

Change-Id: I3593e519d11066cd8631f56131de39e6e3b73f50

layers: Canonicalize function names in log_msg()

For consistancy, Vulkan function names in log_msg strings should
include the vk prefix and end with parens.

Also fix some typos and cut/paste errors.

This only touches lines related to VL-69; it is not a sweep
of all instances in the code.

Change-Id: I41a8f0c9d3b93203f67aef3fa570eb8d95725d31

layers: Update Valid Usage enums (VL-63)

Update Valid Usage enums in a subsection of core_validation.cpp.

Refactored some helper functions to add parameter for msgCode.

Update VU status in the database.

Completes Jira task VL-63.

Change-Id: Iba2cfb978a044d6affb21df9b02a2ac852dfba4e

scripts: Remove debugging code from PV

Change-Id: Ia58e87a07e0e182b6dd447f4d6dc2c7fb2a9f346

scripts: Fix handle tracking in parameter validation

Handle list was reset for each feature -- accumulate handles across
all Vulkan features to allow exension handle validation to proceed.

Change-Id: Id240404d9cc6757d741f575b9152b0aa82a0c1c2

layers: CmdDrawIndirect is not an indexed command

Change-Id: Ic5e947382f64db366c72fbf058bbbd1118f79e01

layers: GH1342, Correct objecttype in object_tracker

The IndirectCountAMD extension had the wrong object type specified
for validating its command buffers.

Change-Id: I36707455c6e59f9f0f8456b67b02a320fa4631fa

loader: gl167 - Remove unknown inst extensions

Change-Id: I24aecbdab452a04514fd42a9a9ae7f3bcf1cd2ca

demos: Make spacebar pause work in cube demo (Linux).

Fixes #1316.
Rework XCB event processing code to handle events
properly and avoid hangs when pausing.
Fix polarity of decrease/increase spin rate keys.  I
suspect that this was an artifact from fixes made
to the transforms a while back.

Change-Id: I74adf7309227fafd175d8972ca930a304c58a1dd

build: Add libxkbcommon to Travis config

libxkbcommon is needed to build VulkanSamples

docs: Fix README to show Travis master br status

Better to show status only for master branch Travis builds.  Other branches frequently fail and cause the badge to not reflect the status of the master branch.

scripts:Adding duplicate error id exceptions

Adding three more exceptions to stats script for duplicate uses of
unique error ids.

layers:Remove redundant checks

Remove descriptor set check to make sure that src & dest descriptor
sets of copy update are valid. Object_tracker already verifies this.

layers:Remove redundant checks

Remove duplicate checks from core_validation that are already handled
in object_tracker.

layers:Add TODO comments for valid usage updates

Note a few unique error enum duplicates that should have their own
unique ids added through spec updates.

layers:Remove redundant check

Core_validation has a shared to check to verify image layout for both
vkCmdClearColorImage() and vkCmdClearDepthStencilImage(). Removing the
same check for color images from image layer.

layers:Remove redundant validation check

As was noted in a comment, the core_validation check for a valid
instance in EnumeratePhysicalDevices() was replicating the same check
in object_tracker. Killing the core_validation check.

demos: Fix CMake file to allow WSI selection

Define DEMOS_WSI_SELECTION CMake variable so it can be set
from CMake command line or GUI.

Change-Id: If2531d102fe9b054a1eee4da28431779b173fc10

external: Back up one spirv-tools commit

As a workaround for the assert running CTS with validation, back
up one spirv-tools commit until SPIRV-Tools issue 517 is resolved

Change-Id: I4b173add841a67654caabd4fd534fe00a8970009

scripts: Fix dynamic array calc in vk_helper.py

VkSubmitInfo->pWaitDstStageMask should be treated as a dynamic array
with a count of VkSubmitInfo->waitSemaphoreCount.

Change-Id: I42da296eeb849f5baabc12d011c938f0f322274a

build: Cleanup warnings when compiling with clang

Travis-CI is building this repo with gcc and clang.
This commit fixes warnings reported by clang but not by gcc.
All the fixes involved adding an extra set of braces in
initializers.

Change-Id: I88e7841d9ab13afe2e45362b8f85261863d5b7fa

scripts:Exceptions for duplicate unique ids

Added a number of exceptions for duplicate uses of unique error enums
that are valid. Also added some comments to a few cases that need new
unique ids.

layers:Remove redundant check

Object tracker layer already makes sure that cmd buffers passed into
vkCmdExecuteCommands() are valid so removing the duplicate check from
core_validation.

layers:Remove redundant image compatibility check

The format compatibility check when creating an image view handles case
of color-color check as well as compressed image compatibility check.
Killing the redundant check and comment as this is all handled above.

Update README to point to Khronos AppVeyor CI

scripts: Remove hack from safe-struct generation

Some special-cases were added to safe_struct genration so that it'd
spit out unneeded worker routines for normal functions. Removed the
offending code and the workaround.

Change-Id: I9394b06e3cdda011f417c725622eec3dd224a989

layers: Remove bad call to safe_struct worker fcn

One of the displaymode functions was calling a safe_struct worker
function incorrectly, and a special-case hack was added to have
unique_objects generate this unneeded function.

Change-Id: I9939b56cb254e50b60d48945a14ad223efad775e

build: Return status immediately if a test fails

build: Cache external build directories

build: Move to latest Android NDK

build: Update Android target revision

tests:Update tests to use unique ids

Update a bunch of tests to use the unique id to check for expected
error. Update documentation for a few tests that were incorrect.
Also a couple of minor test updates to fix minor issues.

layers:Only flag one aspect error at a time

For image aspect bit errors, first flag the error if correct bit is not
set, else flag error if additional bits are incorrectly set. There's
not a real benefit to flagging both errors and it complicates testing.
Generally the first error should be sufficient to allow a user to
correctly update their aspect bit usage.

scripts:Check for use of error enums in tests

Update stats script to gather all error enums that are used by tests to
check for validation errors. If the database reports that a test is
checking a specific enum, but that enum is not being used in the
SetDesiredFailureMsg() for that test, report it as a warning.

Ideally all of the checks will start using the explicit enum value to
check for the presence of that error, so this warning will highlight
tests where it should be easy to update them to check for error enum.

docs: Update README for AppVeyor Windows CI builds

build: Add config file for AppVeyor Windows CI

docs: Update README for Travis

external: Update Glslang/Spirv-Tools revisions.

Update Glslang to commit 807a0d9e2f4e176
and Spirv-Tools to commit 6993fc413d8da97.

Change-Id: I468ec5ec3bee13f4d1549439bef9b40908245f33

build: Add Android build to travis

Change-Id: I51232194642bd2f36e212b4a0ac1e81272eae450

loader: Switch variable appropriately

A delete was being done using a variable that wasn't directly
tied to the array.

Change-Id: I59ba4bc86a72fc8158e3a7c46e13c54fb2409873

layers:Variable renaming for style and clarity

layers:Don't pass active binding set around

The active bindings set was being passed around but the data is only
used in a couple places and an alternate form of it is available in the
binding->requirement map. Also, the binding set was wrong in that it
stored bindings for all bound sets at draw time, but was being used to
check bindings for individual sets.

layers:Clean up descriptor data struct

Stop passing around hideous vector of descriptor state. The data is
already contained in various bits of the cmd buffer struct so just
re-create any data at the point it's needed.

layers: Add VU enums to core_validation.cpp

(Completes Jira VL-62)
Add new VU enums to core_validation (part 4 of 5). Update database
to reflect changes. Small edit to InvalidPushConstants test in
layer_validation_tests.cpp to update error message text.

Change-Id: I05800c9e783d66ef41e92c4af1d2f2e9eb8b9270

layers: Add valid usage enums to core_validation

Update valid usage enums in subsection of core validation from
validateEventStageMask() to validateSecondaryCommandBufferState(),
inclusive.

Update tests for vkCreateFramebuffer to work with refactored error
messages.

Change-Id: Ie6138fbfe7422b3a6affac45faeeb198a0ab6bcc

build: Compile with both clang and gcc on Travis

Fixes #1310

Change-Id: I430089a362b70e104ad3274a76b1033b2e94ac2a

layers:Mark latest buffers and images as written

Fixes #1311

In the Draw-time refactor the marking of store images and buffers as
written was incorrectly moved before the list of images and buffers was
updated for the current active bindings.

This fix restores correct order of operations so that the buffers and
images of interest are first added to cmd buffer active list, then
they're appropriately updated as written.

scripts: Correct author emails in helper script

Change-Id: I53196ed512f5465e6ebfb88b93b727e29a7e9809

scripts: Remove enum_string_helper from vk_helper.py

This functionality is now in helper_file_generate.py.

Change-Id: Iea20fe5a9882de7001154d2a809106c5fb63f3cb

android: Update build scripts w/enum_string codegen

The enum_string_helper header is now generated from vk.xml.

Change-Id: I6adf09663c87b21623993a939a8ad60e343a805d

layers: Mod cmake for new enum_string_helper codegen

Remove old-style codegen invocation and replace with new vk.xml
based stuff.

Change-Id: I4f3dc70bfa3426141b8aaba12b09090eeea05085

scripts: Update lvl_genvk.py w/new helper generator

Change-Id: Ib3afef2e275d2f7b4906d0ca95022aeb4fd2e674

scripts: Update helper gen to ignore disabled exts

Header file description contains unimplemented/undefined extensions.
Filter these out when processing enumerated types.

Change-Id: I1a83dc932a10fe9b3565a4b1462f4ab7e934168f

scripts: Add helper file generator

This vk.xml generator will create the enum_string_helper header file,
as well as the other vk_helper.py-generated files in the future.

Change-Id: Ia19a2c6fd23c1df2af3ded66e2e80b88159a7d85

layers: Fix VkGraphicsPipelineCreateInfo flags check

"pCreateInfos->flags | VK_PIPELINE_CREATE_DERIVATIVE_BIT" is always true

scripts: PR1314, Remove trailing whitespace/newlines

Change-Id: I41f7703540af202b94ebd78ec03b282abb321e35

layers: Make comments consistent across file

Change-Id: I1a92a25ae113142fd8293a1e30d6c0afff2cb759

layers: Remove inconsistent comment specifiers

Change-Id: I6909e27ee88b5362801bc88953746d827e3f05db

layers: Make comments consistent

Change-Id: I07a249f02956e2cda57548a62f8993631d2b334a

scripts: Remove unused code from vk_helper.py

Mostly the string_struct_helper related stuff, but also some
additional orphaned remnants.

Change-Id: Ibdd124d10351c694b02917c81e97c56dc9645e6b

layers: Update 'string' layer header files

Remove struct_string_helper header files, add in enum_string_helper
and <sstream> as these were included in the removed header file.

Change-Id: Ice0842ce2f06b5581a5add739534d59d834edbf6

layers: Remove struct string helper from cmake

Change-Id: I0c1ede8bb65d8926bbb3d83d5d320d2a72639781

loader: EnumPhysDev fixes

Found a few issues, and I had some concerns about the physical
device values enduring over multiple queries.

Change-Id: Ifaa94a4ecf9edfc79bdd3b3d473db068952e3264

layers:Fix CmdDispatch post-call function

Cut-and-paste bug was incorrectly calling CmdDraw post-call function
instead of CmdDispatch.

layers:Fix unique id for CmdDispatch

scripts:Add more duplicate error enum exceptions

Allow various other unique error enums to be used multiple times.
Addressing these on a case-by-case basis and documenting the exceptions
in the script.

scripts:Add duplicate id exception

Add duplicate exception for VALIDATION_ERROR_00648 which is for
VkMappedMemoryRange struct to have valid memory objects. That struct
is used by both Flush & Invalidate MappedMemoryRange.

scripts:Allow duplicate exceptions for some ids

Added global duplicate_exceptions list at the top of the script that
is used to store any IDs that are allowed to be used multiple times
in the source. An explanation comment of all exceptions should be
included.

Initial exception included in list is VALIDATION_ERROR_00942 which is
a descriptor set update error to make sure a valid sampler is included
for relevant write updates. This is re-used for two copy updates that
may have invalid samplers.

layers:Remove redundant check

Remove valid buffer check from descriptor_sets.cpp that's already
handled in object_tracker.

layer:Add unique error ids for memory binding

Update all of the cases to validate that memory is bound to non-sparse
images and buffers to use the correct unique error id.

layers:Add valid usage error ids

Update descriptor set cases where no memory bound to buffer or image to
use the correct respective unique error ids.

scripts:Update stat script output

Improve stats script to output the filename and line numbers where a
unique error enum is being re-used. This makes it simple to pick
through the duplicates and evaluate them.

layers:Fix unique error enum

ResetCommandBuffer valid commandBuffer handle is id *00094, not *00090.
This was causing 90, which is for allocation, to be incorrectly reused.