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.

build: Parameterize def file name

Change-Id: I697c856b7c7532a38823302a4decf78cdea583ae

layers: remove unused variable

layers: Update Valid Usage enums (VL-61, final)

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

Refactored some helper functions to add parameter for msgCode.

Removed check that was redundant with object_tracker.

Update VU status in the database.

Fix a couple other typos.

This completes Jira task VL-61.

Change-Id: Ie7f11741892c68331b70bb1b611f5c80fe930018

layers: add VU checks in image.cpp (Jira VL-66)

Updated existing validation checks in image.cpp to new VU
enums with minor refactoring only. Left TODOs for other
VUs that will require more extensive redesign. Updated VU
database to match. Also minor updates to
layer_validation_tests to account for changed error
messages.

Change-Id: Ib6cc0e360ceeaae256b0cd0f1d272fe127c95419

layers:Refactor CmdDispatchIndirect()

Refactor CmdDispatchIndirect() to use pre/post pattern. Use shared
validate/update functions along with extra code to handle the
validation and state update of the buffer used for indirect call.

layers:Refactor CmdDispatch()

Refactor CmdDispatch() to use pre/post pattern. Updated the DrawType
helper functions for use with Dispatch functions as well. For the
update helper, broke that out into a separate DrawDispatch helper
that does common work, and then a Draw helper that calls the
DrawDispatch helper and then does some extra work only relevant
for draw calls.

layers:Refactor CmdDrawIndexedIndirect()

Refactor CmdDrawIndexedIndirect() to use pre/post pattern.

layers:Refactor CmdDrawIndirect() to use pre/post

Refactor CmdDrawIndirect() to use pre/post pattern and helper functions
for DrawType validate and update. There's a bit of extra validation and
state update that goes along with the buffer passed to this function vs
the Draw & DrawIndexed versions.

layers:Refactor CmdDrawIndexed()

Refactor CmdDrawIndexed() to use Pre/Post pattern. Added validate and
update helper functions that should work for all four CmdDraw*
functions and initiall changed CmdDraw() and CmdDrawIndexed() to use
the helpers. Will update remaining two functions in follow-on CLs.

layers:Improve perf warning for unused vtx buffer

Fixes #1282

Track if a vtx buffer is ever used by a cmd buffer and only flag perf
warning if cmd buffer has a Draw added with a bound vtx buffer that has
never been used.

layers:Kill synchAndPrintDSConfig() function

Delete synchAndPrintDSConfig() and related functions.
Lots of informational prints that aren't needed.

layers:Split ValidateAndUpdateDrawState()

Split ValidateAndUpdateDrawState() to isolate validation and state
updates into ValidateDrawState() & UpdateDrawState() functions.

Refactored CmdDraw() to use Pre/Post patter with all state updates now
occurring in the Post function. Made some initial updates to other
Draw/Dispatch functions to call each new function in-line, but will
refactor these functions to use Pre/Post in follow-on commits.

layers:Split validatePipelineDrawtimeState()

Split validatePipelineDrawtimeState() into validate & update functions.
Also rename validatePipelineDrawtimeState() to
ValidatePipelineDrawtimeState().

layers:Rename two functions to CamelCase

layers:Add PostCallRecordCmdDraw()

Add PostCallRecordCmdDraw() to perform state updates for CmdDraw()
function. Don't quite have all state updates in there yet. Remaining
state updates will be in follow-on commit.

layers:Separate state update from addCmd()

Update addCmd() function to only perform validation and rename it to
ValidateCmd(). Separate out the state update of last_cmd in cmd buffer
to new UpdateCmdBufferLastCmd() function.

Made sure that everywhere ValidateCmd() is called now also calls
UpdateCmdBufferLastCmd(). Currently just calling them back-to-back but
will continue with further updates that put the ValidateCmd() call into
the PreCallValidate*() function and the UpdateCmdBufferLastCmd() into
the PostCallRecord*() function.

Also included some comments on work that needs to be done while
breaking up CmdDraw() for pre/post functions.

layers:Pass cmd buffer as log_msg object

Fix some old TODOs where cmdBuffer was not being correctly passed into
log_msg as the src object.

layers:Remove unneeded return value

MarkStoreImagesAndBuffersAsWritten() only updates state and the return
value was a constant false. Update function to remove the return value.

loader: clean up comment and variable

Make one variable's name plural to clarify that it
contains more than one layer.  Also, clean up a
comment that wrapped prematurely to the next line.

Use Python instead of greadlink to get full path on macOS

Python is built-in on Macs; this avoids an unnecessary dependency on greadlink

loader: Fix out of memory in enumerate phys devs

Change-Id: I9f25875f065f7db58f9c8841965ba1f2f974de90

layers:Update dispatchable object printing

Fixes #1188

Uniformly print all dispatchable handles in error messages as pointers
with the "%p" print specifier.

loader: Update term phys devices dynamically

Change-Id: Ib474a7af431746d9e820e188681d2a9310e94a0d

loader: Update tramp phys devices dynamically

Change-Id: I693598b5c9f72ecee10661e3af9420941b2bd67c

android: Preserve toolchain build if present

Similar to a2e6ddd1 which preserves it for desktop.

loader: guard gcc feature appropriately

Macro __GNUC__ should be used to guard gcc-specific code.
Also exclude clang because though it defines __GNUC__ it
doesn't accept this particular gcc feature.

Change-Id: I703289cad58e7675f4c4ca5e7635fc8fb35ae77e

Merge pull request #1300 from KhronosGroup/161221-fix_string_validation

loader: Fix validation error

loader: Fix validation error

The string length validation will not detect strings
that exceed the max length.
For example, when i = max_length-1 and utf8[i] is a valid
character (>= 0x20 and < 0x7f) no error is thrown and
the next iteration will end the loop.
This change extends the loop and the if check to
catch this issue.

loader: Fix Clang warning in table_ops.h

This fixes the following warning:

error: cast between incompatible calling conventions 'cdecl' and 'stdcall'; calls through this pointer may abort at runtime

This was revelated by building the loader with Clang on Windows in
32-bits mode. In 64-bits this warning doesn't appear, presumably due
to different calling conventions.

Change-Id: I7018be38859695436487f582f515bfb08ca28e06

layers: Add missing <string> dependency

Change-Id: Ib3c74be1e1d3c168720aaef4dafed1a6a04c7d9b

cmake: speed up the build

Remove old dependencies from cmake, which was causing long build times
even when nothing had changed.

Change-Id: Ie162606b54352306ae6400aa18e942d8cffb7782

layers: Update Valid Usage enums (VL-61, partial)

Update Valid Usage enums in a subsection of core_validation.cpp by
adding a parameter to helper functions insideRenderPass() and
outsideRenderPass().  Update that VU status in the database.

Fix a couple other typos.

This partially implements Jira task VL-61.

Change-Id: I8d1fe99a237c4cd22216a1236f47f72fffd4fd00

scripts: Remove enum/struct_validate_helper funcs

From vk_helper.py.  These were not used in the codebase.

Change-Id: I3cc80e34eb831122ce8c54dbc004a003e02a47fe

layers: Remove unused header from threading/PV

vk_struct_validate_helper.h not used.

Change-Id: I19757e96900e0500f4db9355c38a497936706bba

layers: Remove redundant CreateRenderpass checks

From the image layer.  The check for valid attachment layouts
and load/storeOps already performed in the parameter validation
layer. Also removed now-dead support function and unneeded
header file.

Change-Id: I4f1ad1ba899a712ba21ae32e3b3e2d947b640f4d

layers: Remove enum_validate_helper from PV

A single call was used, replaced with an internal generated function
call and removed the old include. Added note with validation error
ID.

Change-Id: Ied88daf5bab003269c44ef2bef8b2eb81a9e9094

layers: Remove unused threadchecker header file

Change-Id: I7906b82c1c9c0397113508da74348ea28fe21add

layers: Remove cmds vector from GLOBAL_CB_NODE

Now that cmd buffer printing has been removed, the only use of this is
to distinguish between cmd buffers invalidated before and after a
complete recording. This should eventually be folded into the state
machine, but removing the vector of cmds is a first step.

Signed-off-by: Chris Forbes <chrisforbes@google.com>

layers: Fix size_t-int cast warning in descriptor_sets

Discovered in VS2015.

Change-Id: I2a42209a9ee9bd0e8f3da21c0f1a613315d5a6ce

layers:Clean up descriptor set layout on destroy

Erase destroyed descriptorSetLayout from map.

layers:Add validation flag for DestroyBuffer()

layers:Refactor DestroyBuffer()

Update DestroyBuffer() to use Pre/Post pattern. All of the state
updates are now only performed if call down chain is not skipped.

layers:Add GetQueryPoolResults() validation flag

layers:Refactor GetQueryPoolResults() to pre/post

Refactor GetQueryPoolResults() to use Pre/Post pattern. Pulled all of
the validation into PreCallValidate* function. There was only one
state update occuring so pulled that code into PostCallRecord function.

layers:Add validation flag for DestroyQueryPool()

layers:Refactor DestroyQueryPool

Refactor DestroyQueryPool() to use Pre/Post pattern. This moves the map
update to after the call down the chain which is where it should live
so that it only executes when validation passes.

layers:Add validation flag for DestroySemaphore()

layers:Add validation flag for DestroyFence()

layers:Refactor DestroySemaphore

Refactor DestroySemaphore() to use Pre/Post pattern.