Merge pull request #1684 from baldurk/reflection-interface-improve

Opt-in improvements to reflection interface

Reflow for better readability

Bump version.

Build: Fix #1665: remove __fastcall

Merge pull request #1693 from sparmarNV/fix-NV_mesh_shader

Add missing support for gl_MeshViewCountNV/gl_MeshViewIndicesNV in task shaders

Add missing support for gl_MeshViewCountNV/gl_MeshViewIndicesNV in task shaders

Merge pull request #1692 from alan-baker/update-spvtools

Update SPIRV-Tools and tests expectations

Update expected test results

Update SPIRV-Tools version

Add option to reflect all block members, inactive or active.

* The stages mask is more fine-grained, and each variable or block's mask
  indicates which stages it's active in.

Add option to reflect buffer blocks & variables separately to uniforms

* Also note the uniform indices of atomic counter buffers

Reflect array stride, top-level array stride, and block member count

Reflect pipeline outputs as well as inputs, optionally from other stages

* We add an option to reflect inputs from other stages than vertex, if only a
  later subset of the stages is linked into the program.

Add an option to report array variables with trailing [0] suffix

* This is as expected by ARB_program_interface_query

Include array index in reflected uniform names more consistently

* This comes from the resolution of issues 4, 5 & 6 in
  ARB_program_interface_query, stating that uniform buffers should have their
  members expanded out as normal and arrays should have elements added.
* If a buffer block has a large array e.g. [10000] we don't want to iterate over
  every array element. Instead we should only expand out the first [0] element,
  then expand as normal from there.
* The array name should still be appended with [0] to indicate that it's an
  array.

Add options to control how reflection information is built

Move TObjectReflection into public interface to clean up reflection

* Forwarding functions are left to preserve source compatibility with code using
  the old queries.

Merge pull request #1687 from Igalia/apinheiro/no-more-struct-member-offsets

ParseHelper: don't assign xfb_offset for struct members

Merge pull request #1667 from amdrexu/bugfix

Add 8-bit/16-bit transform feedback support for future use

Add 8-bit/16-bit transform feedback support for future use

Merge pull request #1690 from BinaryRK/relative-include-fix

Use relative include paths in StandAlone/ResourceLimits.h

Switch to relative include path for ResourceLimits

Merge pull request #1688 from mattparks/patch-1

Switched to std::string for TReflection and TObjectReflection

Fix #1682: flesh out more vulkan/SPV ModuleProcessed

Merge pull request #1686 from mspang/glslang-fuchsia-build

Port Unix/ossource.cpp to Fuchsia

Merge pull request #1685 from baldurk/warning-fix

Fix a couple of warnings clang found

Merge pull request #1683 from null77/disable-extra-semi

Disable Clang extra semicolon warning.

Switched to std::string for TReflection and TObjectReflection

In MSVC 2017 Debug mode when a reflection is deleted (when a shader goes out of scope) xutility will fail to clear the children due to the TString allocator implementation. By switching to std::string xutility no longer throws the error.

ParseHelper: don't assign xfb_offset for struct members

This commit undoes the code on the commit "ParseHelper: assign
xfb_offset for struct members too" (commit
af8c1bdb161e4b2ddf019b61bdfca61f84cb1ca8), except the method renaming,
and updates the tests.

During the review of the implementation of Vulkan extension
VK_EXT_transform_feedback for the Linux Intel driver, it was pointed
that the Vulkan environment spec for SPIR-V only ever refers to block
or block members being decorated. It would be strange to not do the
same for OpenGL (ARB_gl_spirv). That would also fit better to what
GLSL does, where setting explicit xfb offsets for struct members is
not allowed.

FWIW, the original patch was proposed based on the fact that
ARB_gl_spirv is relying on OpenGL for how xfb offsets should be
assigned to members, and it was not clear (at least to me) which is
the responsible of such.

Port Unix/ossource.cpp to Fuchsia

Fuchsia has pthreads but doesn't have sys/resource.h and doesn't
implement pthread_cleanup_push().

ANGLE bug: 2475

Add missing override specifiers in virtual class

Fix mismatched integer comparison (size_t/unsigned int vs int)

Disable Clang extra semicolon warning.

Usually this warning is disabled by default. But when turned on Clang
complains about extra semicolons in Glslang headers. Turn this off for
now. See http://crbug.com/926235

Merge pull request #1675 from nicebyte/ext_yuv_target

Add support for GL_EXT_YUV_target

Build: Remove extra commas in calling INSTANTIATE_TEST_SUITE_P.

Hopefully, this fixes the sudden new breakage with the bots.

Merge pull request #1681 from baldurk/reflection-fixes

Reflection consistency changes & bugfixes

Add support for GL_EXT_YUV_target

Update reflection offset consistently for structure members

* Previously the offset was only updated with EOpIndexDirectStruct, but we also
  need to update it for the other index types into arrays, and when expanding an
  aggregate reached after dereferencing.

Add missing initialisation of counterIndex and stages in badReflection()

HLSL: Fix #1679: Add HLSL earlydepthstencil -> SPV EarlyFragmentTests.

Merge pull request #1677 from greg-lunarg/kg100

Update spirv-tools known-good.

Update spirv-tools known-good.

Merge pull request #1676 from nsubtil/nsubtil/do-not-cache-lib-suffix

build: Do not cache debug library suffix

build: Do not cache debug library suffix

For nested project builds, writing CMAKE_DEBUG_POSTFIX into the cache
ends up affecting other projects. Caching this value doesn't seem to be
required in practice, so this change removes the cache tag.

Merge pull request #1674 from null77/fix-msvc-sign-warning

Suppress a MSVC cast warning in GN.

Suppress a MSVC cast warning in GN.

Also came up when using the BUILD.gn file with ANGLE.

ANGLE bug: 3088

Merge pull request #1672 from sparmarNV/NV-extension-checks

Add extension checks for NV block member builtins

Merge pull request #1671 from null77/fix-chrome-clang-warning

Suppress inconsistent override warning in GN.

Add extension checks for NV block member builtins

Suppress inconsistent override warning in GN.

This came up when trying to roll the new glslang into Chromium.

ANGLE bug: 3088

Merge pull request #1626 from KhronosGroup/extension-support

Infrastructure: Generalize and broaden per-variable extension checking.

Merge pull request #1664 from null77/support-angle

Fix a few GN build issues.

HLSL: Protect against some crashes

Merge pull request #1663 from jeffbolznv/reference_vs_memorymodel

More precisely check for the qualifiers that enable the vulkan memory model for buffer references.

More precisely check for the qualifiers that enable the vulkan memory model for buffer references.

Not all memory qualifiers necessarily need the vulkan memory model, e.g. volatile/restrict that correspond to core SPIR-V features do not.

Fix a few GN build issues.

 * adds a source set for default resource limits to mirror CMake
 * adds a target executable for the standalone validator
 * fixes a missing warning

Allows ANGLE to use the integrated BUILD.gn instead of a custom one.

ANGLE bug: 3088

Merge pull request #1656 from amdrexu/bugfix

Fix xfb_stride incorrectness(#1654)

Merge pull request #1662 from dneto0/fix-leak2

HLSL: Avoid leaking a shadow sampler struct

Merge pull request #1661 from dneto0/fix-leak

Avoid leaking name for counter buffer

HLSL: Avoid leaking a shadow sampler struct

Avoid leaking name for counter buffer

Merge pull request #1658 from KhronosGroup/fix-name-loc-str

HLSL: Fix #1655; use "" for nullptr file names

HLSL: Fix #1655; use "" for nullptr file names. Needs test cases.

Build: fix warnings. Fixes #1653.

Fix xfb_stride incorrectness(#1654)

Add int64 support in XFB. Change containsDouble to contains64BitType. Make
it more general.

Bump revision.

Merge pull request #1652 from gnl21/cmake-fix

Work around a CMake cross-compilation issue

Merge pull request #1647 from dneto0/update-known-good

Update SPIRV-Tools known good: relax Uconvert check

Update SPIRV-Tools known good: relax Uconvert check

Permit UConvert as an OpSpecConstantOp operation when
SPV_AMD_gpu_shader_int_16 is used.

Infrastructure: Generalize and broaden per-variable extension checking.

Merge pull request #1646 from dneto0/parens-warning

Parenthesize to avoid GCC warning

Parenthesize to avoid GCC warning

Merge pull request #1645 from KhronosGroup/dupe-accelstruct

Don't output duplicate OpTypeAccelerationStructureNV declarations

Merge pull request #1644 from jeffbolznv/buffer_reference

GL_EXT_buffer_reference

Don't output duplicate TypeAccelerationStructureNV declarations

GL_EXT_buffer_reference

SPV: Move to the latest header.

Make sure source strings are terminated (#1641)

* Make sure source strings are terminated

The source strings may or may not have a null terminator. We need to
make sure we add one before outputting the source strings as we iterate
over the c-str looking for the null terminator.

* Review feedback

Merge pull request #1643 from dneto0/update-issue-1642

Mark spv.int16.amd.frag.out as failing validation

Mark spv.int16.amd.frag.out as failing validation

See #1642

Also updates SPIRV-Tools and SPIRV-Headers known_good.

Work around a CMake cross-compilation issue

CMake can't correctly configure the latest code because of an issue with
cross-compilation of pthreads:
https://gitlab.kitware.com/cmake/cmake/issues/16920

Merge pull request #1634 from greg-lunarg/lines10

Create separate OpSource for each included file when compiling -g

Create separate OpSource for each included file

Merge pull request #1627 from greg-lunarg/lines8

Add passes to spirv-opt passes to preserve source line info when compiling -g.

Merge pull request #1633 from akien-mga/cmake-2.8-pthread

CMake: Fix linking pthread of CMake < 3.1

CMake: Fix linking pthread of CMake < 3.1

As reported in #1624, Ubuntu 14.04 LTS still uses CMake 2.8.12 which
does not support the Threads::Threads target (added in CMake 3.1).

This could be reverted once the required CMake version is bumped to 3.1+.

Merge pull request #1632 from alan-baker/mem-model-1610

Move memory model scope tests to Vulkan 1.1 batch

Move memory model scope tests to Vulkan 1.1 batch

Fixes #1610

* Updated expected result due to version change

Build: Make calling argument types match declared parameter types.

Fixes #1630 and fixes #1631.

Add passes to legalization to preserve source line info when generated.

Merge pull request #1628 from jeffbolznv/storagebuffer_sbt

Fix bug where shaderRecordNV buffers were using StorageBuffer storage class

Fix bug where shaderRecordNV buffers were using StorageBuffer storage class

Licensing. Fixes #958. Add licenes file and update copyrights.

Build: Reduce warnings.

GLSL/SPV: XFB: No streams on types, but support them on built-in blocks.

From internal Khronos discussions, work, and testing.

Merge pull request #1625 from jeffbolznv/update_spirv_tools

Update to newest SPIRV-Tools, emit default Binding decoration.

Update to newest SPIRV-Tools. It checks the Vulkan rule that all resources must have a set and binding decoration, which many tests fail. So add code to set a default value of zero. Also disable PCH for Ninja builds.

Infrastructure: Fix .gitattributes typo eof -> eol. Also update README

Add to README: git config --global core.fileMode false

Merge pull request #1620 from akien-mga/unix-executable-bit

Remove Unix executable permission from text files

Merge pull request #1624 from akien-mga/osdependent-link-threads

CMake: Link OSDependent with Threads::Threads