toolchain: Switch to using github for glslang

We don't have any tags to pull from, but we can pull specific
commits that are tested and known to work.

The current commit matches up with the previous subversion
version of glslang, so there is no functional change.

toolchain: Add script to pull and build on Windows

Summary:

Add Windows version of update_external_sources, and relocate
glslang and LunarGLASS target revisions to external files shared
with Linux.

Details:

On Linux, we've successfully been using update_external_sources.sh
to get on the target version of glslang (which generates SPIR-V) and
LunarGLASS (which consumes SPIR-V).  Both of these components are
peers to the typical Vulkan directory structure:

   /LoaderAndTools
       /layers
       /loader
       /demos
       /...
   /glslang
   /LunarGLASS

There hasn't been an equivalent on Windows, so tedious manual steps
and GUIs have been used on individual systems.

This commit introduces a Windows batch script that automates the
process similar to Linux.  It detects if you have the required
dependendcies and early-outs for any problems.

If you are a Windows developer that only wants spirv headers, run:

  update_external_sources.bat --sync-glslang

If you want to build the Windows version of glslangValidator.exe, run:

  update_external_sources.bat --build-glslang

If you want to run LunarGOO with SPIR-V support on Windows, run:

  update_external_sources.bat --build-LunarGLASS

The script builds Debug and Release x64 versions, and should be run from:

  Developer Command Prompt for VS2013

This commit also relocates the target version of glslang and LunarGLASS
to an external shared file that only contains the version number of
each.  Both scripts read the value from the file, therefore they
cannot get out of sync.

tests: Fix some failing layer validation tests

MemTracker was changing an API return value during an error case which caused ParamChecker to complain, flagging unexpected error.
ParamChecker was no allowing NULL fence on Queue Submit or NULL pDynamicOffsets on when binding Descriptor Sets. Both of these values can be NULL.

loader: Rename loader_platform.h -> vk_loader_platform.h

layers: Rename all layer headers to begin with "vk_layer_" prefix

include: Rename vkLayer.h -> vk_layer.h and vkIcd.h -> vk_icd.h

vulkan.h: v129 -- rework clear colors, remove raw clear [#14020]

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

tri: Use render pass load ops for color and depth clears

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

cube: Use render pass load op for depth clear

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

vulkan.h: Split attachment and image clears (#13914, v126)

- Add bit flags for image aspects.
- Replace VkRect with VkRect2D and VkRect3D.
- Rename vkCmdClearDepthStencil to vkCmdClearDepthStencilImage
- Add vkCmdClearColorAttachment and vkCmdClearDepthStencilAttachment

Remaining to be done:
- Actually implement vkCmdClearColorAttachment,
  vkCmdClearDepthStencilAttachment in the Intel ICD
- Enforce renderpass interactions: CmdClear*Attachment may only be
  called within a renderpass; CmdClear*Image may only be called outside
  a renderpass.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

vulkan.h: Update features/limits structures (#12827, v125)

- Add infrastructure in vulkaninfo to dump features/limits. To be filled
  out to dump everything interesting.
- Move existing limits from VkPhysicalDeviceInfo to
  VkPhysicalDeviceLimits, and drop those that no longer exist.
- Adjust the drivers to fill out the above in their new location.
  Other new features/limits still to be exposed by nulldrv and the intel
  ICD.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

update vulkan.h for multi-level command buffers (V121)

With minimal changes to keep everything building and functioning.

vulkan.h: Added #define for MAX_DESCRIPTION

Part of header revision 135 changes.

vulkan.h: V120 -- Tighten spec around VkMemoryRequirements

Moved the HOST_LOCAL memory property.

vulkan.h: V119 -- Remove unsigned byte indices

Took out VK_INDEX_TYPE_UINT8.

vulkan.h: V118 --  Add array  types to VkImageViewType

vulkan.h: V117 -- Remove Atomic Counters, Bug #14081

vulkan.h: V116 -- Remove VK_IMAGE_LAYOUT_CLEAR_OPTIMAL

Usage was declared redundant.

vulkan.h: V115 -- Reintroduce memory heaps & types.  Bug #14082.

Reworked memory properties and types and added support for multiple
heaps.

layers: Update ObjectTracker layer to check first-level struct parameters for valid objects

layers: Fix ObjectTracker to add Instance when it's created

demos: Set pName for cube shaders to "main" to keep ShaderChecker happy

loader: Use _alloca instead of alloca on Windows

bug 14014: Fill out support for VkShaderModule

Updated tests and demos to properly use VkShaderModule.
Add support to shader_checker for shader module.
Doesn't do anything with the pName parameter.

bug-14014: Allow multiple entrypoints in modules

Continue implementation of shader module.

loader: Move CreateDevice to device table

Discovered an issue where a layer was doing cleanup
in it's DestroyDevice function but the CreateDevice
was never called.
This happened because the extension was only enabled
on the device chain and the device chain doesn't actually
call CreateDevice. That happens on the Instance chain.
Making it so that we can call down the device chain -
which is terminated by the ICD.
We need to know the real device object to construct the
device chain heiarchy and when calling down the device
chain it should end with the ICD doing the actual device
object creation.

This patch fixes the issue by using the
same process as CreateInstance. The loader will call
the ICD's CreateDevice and pass that in the *pDevice
argument. The layers then ignore the PhysicalDevice parameter
and use the *pDevice to access the device chain.
To prevent the ICD from being called twice needed to
stub in a special loader_GetDeviceChainProcAddr to provide
a stub for only CreateDevice as the end of the chain.

loader: Remove locks from vkGet routines

If the driver needs thread safety to answer "get" requests, it is
assumed that it will lock for itself

vulkan.h: Bug 14181 - Barriers need to be more precise

Demos: use vcproj .user file to configure location of vulkan.dll in msvs

loader: Remove uneeded ICD and layers scanned boolean

Artifact of when layers and ICDs weren't scanned initially.

loader: Support gettting JSON ICD manifest files from registry

Windows now supports the JSON manifest file for ICDs

loader: Fix compilation warnings for debug_report on Windows

loader: Add math library to the build for linux  cJSON files

loader: Add parsing of JSON file to find location of ICD libraries

Works on Linux not yet on Windows.

cube: fix validation and clean up errors

Layers: Fix screenshot seg fault, changed accesses to dispatch tbl

demos: Ensure VertexInputState is NULL if unused

Was passing in pointer to a null VertexInputState structure, but
driver needs null pointer if structure unused.

Revert "loader: Remove CreateDevice from loader's instance"

This reverts commit b81161d0fa45c3b1333d8954220c992c5eb8aac4.

Change did not match commit message.

loader: Need to use ICD's GetDeviceProcAddr

The loader_GetDeviceProcAddr is the terminator
for the CreateDevice chain. It needs to call the ICD's
GetDeviceProcAddr to get the correct function pointer
not the object's current dispatch table that points
at the beginning of the chain.
Saw issue when running cube with only DRAW_STATE layer
enabled.

loader: Remove CreateDevice from loader's instance

Need to remove the CreateDevice function pointer from the
loader's instance table.

loader: Move CreateDevice to device table

Discovered an issue where a layer was doing cleanup
in it's DestroyDevice function but the CreateDevice
was never called.
This happened because the extension was only enabled
on the device chain and the device chain doesn't actually
call CreateDevice. That happens on the Instance chain.
Making it so that we can call down the device chain -
which is terminated by the ICD.
We need to know the real device object to construct the
device chain heiarchy and when calling down the device
chain it should end with the ICD doing the actual device
object creation.

This patch fixes the issue by using the
same process as CreateInstance. The loader will call
the ICD's CreateDevice and pass that in the *pDevice
argument. The layers then ignore the PhysicalDevice parameter
and use the *pDevice to access the device chain.
To prevent the ICD from being called twice needed to
stub in a special loader_GetDeviceChainProcAddr to provide
a stub for only CreateDevice as the end of the chain.

integrate review feedback.

layers: Do not advertise support for WSI

Layers that do not implement WSI features yet can 'trace'
WSI calls do not need to advertise WSI as an extension.
Any WSI calls made will land in the layer if it hooks
those calls. Otherwise should not care about WSI.

layers: Abort logging if null debug_report ptr

If a layer were to only be enabled on the device chain
it would not have had a chance to allocate a debug_report
data structure. Just treat such cases as having debug_report
disabled.

layers: Remove unneeded asserts

We'll soon break the rule being asserted, so remove
them.

generator: Add more debug statements

The location comments in the generated code help
a lot in determining what part of the generator generates
what code.

generator: Clear dispatch table

Finding entries that are not filled in properly is easier
if the table is initialized to zeros.

loader: Need to search device extension cache

Code was searching global extension cache for
extensions enabled at CreateDevice time. List are similar
but not identical and cause vulkaninfo to hit a segfault.

loader: Add debug info

vulkaninfo: Use app's extension count, not system

Was telling the driver to enable more extensions than
intended.

vulkaninfo: handy debug output

vulkaninfo: Need WSI extension for some queries

vulkan.h: V110 #13864 -- Border Color Values Underspecified

layers: ObjectTracker cleanup

Removed some unused error codes.
Updated INVALID_OBJECT error to correctly be flagged.

vulkan.h: Bug 14171 - remove unused structure type enums

vulkan.h: Bug 14153 - remove maxInlineMemoryUpdateSize

vulkan.h: Bug 14136 - reorder logicop enums

vulkan.h: Bug 13900 - Remove initialCount from semaphores

layers: Use new param checker layer.

vulkan.h: Bug 13616 Clarification of types of sample counts

vulkan.h: Bug 13939 Remove references to depth/stencil resolve

layers: Flag DrawState INVALID_RENDERPASS error if NULL RenderPass used to Begin/End/Continue a RenderPass

layers: Warn in DrawState if DescriptorSet bound that was never updated.

layers: In DrawState, validate correct use of RenderPass

This change validates certain API calls are made inside of or outside of a RenderPass as appropriate. Exact restrictions included in this changelist are included below.

Also fixed cube and tri demos to Bind Pipeline & DescriptorSets within RenderPass and updated some layer validation tests.

The following calls must be within RenderPass:
vkCmdBindPipeline() w/ GFX pipelineBindPoint
vkCmdBindDescriptorSets() w/ GFX pipelineBindPoint
vkCmdBindDynamicStateObject()
vkCmdBindIndexBuffer()
vkCmdBindVertexBuffers()
vkCmdEndRenderPass()

The following calls must be outside of RenderPass:
vkCmdBindPipeline() w/ COMPUTE pipelineBindPoint
vkCmdBindDescriptorSets() w/ COMPUTE pipelineBindPoint
vkCmdBlitImage()
vkCmdResolveImage()
vkCmdBeginRenderPass()

layers: Add DrawState check to verify Draw* cmds only issued within RenderPass

layers: Add DrawState check to verify that Pipeline layout from vkCmdBindDescriptorSets() matches layout from PSO at Draw time.

Also fix a bug where PSO status flags were not getting set into Cmd Buffer.

layers: Flag DrawState INVALID_RENDERPASS error if NULL RenderPass used to Begin/End/Continue a RenderPass

vulkan.h: V109 -- Remove minLod from VkImageViewCreateInfo

vulkan.h: V108 -- Static Types, Bug# 13919

layers: Correctly set Object Type for Dynamic State when flagging error

layers: Report error in DrawState if vkCmdBlitImage() called during RenderPass.

layers: Added DrawState check to flag error if compute pipeline bound during RenderPass

Bug 14014: Allow multiple entrypoints in modules

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14014
Header and minimal code changes to include support for
multiple entrypoints in a shader module.
Driver implementation TBD.

bug 14016: Make vkResetFences take const pFences

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=14016

screenshot: remove unneeded initialization code

The device dispatch is setup in GetDeviceProcAddr, don't
need to do it in layer's CreateDevice.

screenshot: Need to wrap more functions

Need to include CreateInstance, DestroyInstance
and CreateDevice in screenshot's GetInstanceProcAddr
otherwise cannot process setup correctly.

screenshot: Need a DestroyInstance

Need way to clean up resources allocated at CreateInstance.

loader: Use loader_GetInstanceProcAddr

By using the vkGetInstanceProcAddr function in the loader's
instance dispatch table we will trace that function
during initialization of layers. Don't want to do that so
call loader internal function.

image: Need to get layer_data from correct place

vulkan: remove unused error code

vulkan.h: Remove vkGetFormatInfo, add features/limits (#12827, v125)

NOTES:

1/ Some layers impact from vkGetFormatInfo ->
vkGetPhysicalDeviceFormatInfo; some checks are currently disabled in
ParamChecker pending discussion on the best way to do this. Similar
checks in Image layer implemented via additional layer_data member
to link back from VkDevice -> VkPhysicalDevice.

2/ VkPhysicalDeviceFeatures, VkPhysicalDeviceLimits members all zero
for now; also some further churn to be done to the contents of these
structures.

Signed-off-by: Chris Forbes <chrisf@ijw.co.nz>

bug 13038: Remove support for multiple GPUs

https://cvs.khronos.org/bugzilla/show_bug.cgi?id=13038

loader: Add {}'s so that loader compilers on Windows

layers: Add DrawState check to verify that Pipeline layout from vkCmdBindDescriptorSets() matches layout from PSO at Draw time.

Also fix a bug where PSO status flags were not getting set into Cmd Buffer.

vulkan.h: V104 -- Unify compute/graphics pipeline shader passing semantics, Bug# 13800

layers: ObjectTracker per-instance storage, updated logging

o  Moved non-generated code into object_track.h header file
o  Added support for per-instance storage
o  Updated for new debug logging mechanism/callbacks

layers: Fix DrawState compile fail

Missed adding in layout when resolving merge conflict.

layers: Add DrawState check to verify Idx buffer bound before an Indexed Draw

Also some minor formatting clean-up to reduce lines of code

layers: Add DrawState check to verify BeginCommandBuffer was called before updating or ending a Cmd Buffer

Also migrated code to set Dynamic State back into CmdBindDynamicState function since that's the only call to set Dynamic State.

tri/cube: Convert GPA usage/error-checking/reporting to a macro.

tri: Change to vkGetDeviceProcAddr() for WSI functions.

vulkan.h: V103 -- Add VkPipelineLayout to vkCmdBindDescriptorSets, Bug# 14047

layers: Addition of source_line_info.py script to simplify codegen debugging

vk-layer-generate.py and vk_helper.py now use added source_line_info.py script to insert comments throughout generated code indicating where the code was generated from.

Layers: Correctly handle WSI_LUNARG extension.

Other code cleanup.

MemTracker: Advertise WSI_LUNARG extension in GetPhysDevExtInfo

layers: Erase proper key in draw state.

layers: Fix iterator comparison.

layers: Move image validation to separate layer.

layers: Comment for newbies.

MemTracker: make sure get_mem_binding_from_object uses dispatchable object

layers: Add DrawState check for Invalid PSO CreateInfo state

Report any violations in PSO restrictions as DRAWSTATE_INVALID_PIPELINE_CREATE_STATE error and don't call into driver.
Also fix a couple of broken error calls that had NULL dispatchable object parameter.