Merge pull request #3043 from gnl21/helper-invocation

Change Volatile generation for HelperInvocation

Test for spv1.6 + memory model HelperInvocation

This should generate a Volatile tag on the access, not on the variable
itself.

Change Volatile generation for HelperInvocation

For SPIR-V 1.6 HelperInvocation accesses need to be volatile to avoid
undefined values when shaders execute 'demote'. Previously this was
always decorated on the gl_HelperInvocation variable, but this is not
valid when the Vulkan memory model is in use.

When the memory model is enabled, stop decorating the variable
declaration and apply the memory semantic to access chain loads instead.

Fixes #3042

Merge pull request #3049 from jeremy-lunarg/hayes-update-changes

Update CHANGES for release 11.12.0

Update CHANGES for release 11.12.0

Merge pull request #3048 from jeremy-lunarg/hayes-update-known-good

Update known_good.json

Update known_good.json

Merge pull request #3037 from mbechard/master

only use dead input elimination on vertex shaders

Merge pull request #3045 from James2022-rgb/feature/c_interface_preamble_support

Add preamble support to the C interface.

CInterface: Add preamble support.

Add interface for `TShader::setPreamble`.

Merge pull request #3038 from ewerness-nv/extmicromapongithubmaster

GL_EXT_opacity_micromap

Merge pull request #3028 from rhabacker/fix-shared-build-mode

cmake: Do not install static libraries in shared build mode

GL_EXT_opacity_micromap

Merge pull request #3027 from rhabacker/fix-cmake-files-install-location

cmake: Use common installation directory for cmake support files.

only use dead input elimination on vertex shaders

It can't be safely used unilaterally on other stages, since that will
result in output/input mismatches between stages. They arn't
having their output variables eliminated accordingly.

Merge pull request #3012 from JohannesKauffmann/fix-wundef

Fix -Wundef warnings for MINGW_HAS_SECURE_API

Merge pull request #3036 from pmistryNV/bug3019

Fix for bug #3019 : That incorrectly removed perVertexEXT qualifier as…

Fix for bug #3019 : That incorrectly removed perVertexEXT qualifier as arrayed IO

This bug got introduced as part of EXT_mesh_shader changes 228c672
Also updated barycentric test cases to add coverage for multiple pervertexEXT array inputs

Merge pull request #3034 from jeremy-lunarg/hayes-fix-debugdeclare-debugvalue

Use DebugDeclare for local variables

Use DebugDeclare for local variables

Previously we had decided to issue DebugValue directly in glslang.
However, this was incorrect and causing issues with RenderDoc.

Merge pull request #3021 from qingyuanzNV/fix_opline_prepending_opfunction_with_pp

Fix OpLine prepending OpFunction reports wrong file when #line is pre…

Merge pull request #3031 from rg3igalia/sub-group-size-arb-flat-fix

Make gl_SubGroupARB a flat int in Vulkan

Make gl_SubGroupARB a flat int in Vulkan

gl_SubGroupARB was being correctly declared as an Input variable in
Vulkan but it was missing the Flat decoration, which made spirv-val emit
the VUID-StandaloneSpirv-Flat-04744 validation error with shaders using
that built-in.

Merge pull request #3032 from mbechard/master

Improve naming in link validation

further updates to 67384dd18b1715

since we are changing the type before the visitBinary now, we need to be
comparing against the new type.
Previous test cases worked since the 'unitType' was a shallow copy and
ended up getting updated in some cases to match the new type, but not all.

cmake: Do not install static libraries in shared build mode.

The glslang cmake build system installs static libraries in addition to
the dynamic glslang library, which are required when used in a package
that uses glslang when calling find_package().

Distributions such as openSUSE (and perhaps others) use a "shared only"
strategy, which conflicts with the current state of the glslang build
system.
The static libraries mentioned are already all included in glslang and
thus not needed. With this commit, these will no longer install the
associated cmake support files when glslang is built shared.

cmake: Use common installation directory for cmake support files.

At least on openSUSE, the usual installation location for cmake support
files is ${MAKE_INSTALL_LIBDIR}/cmake/<name>

   $ find /usr/lib64 -name '*.cmake' | grep /cmake/ | wc -l
   834
   $ find /usr/lib64 -name '*.cmake' | grep -v /cmake/ | wc -l
   8

which is also used by glslang with these changes.

Merge pull request #3025 from pmistryNV/GL_EXT_mesh_shader

Make a spirv builder utility function for termination instructions that take input …

Make a utility function for termination instructions that take input operands.
Use the new utility function for generating OpEmitMeshTasksEXT.

Merge pull request #3023 from pmistryNV/GL_EXT_mesh_shader

Make OpEmitMeshTasksEXT a terminal instruction

Merge pull request #2985 from jeremy-lunarg/hayes-nonsemantic-shader-debuginfo-rebase

Implement NonSemantic.Shader.DebugInfo.100 debug instruction generation.

These instructions will be generated under the -gV and -gVS  command line options. These instructions enable source-level shader debugging with Renderdoc.

This is an alpha release of this capability. Additional improvements are forthcoming. Use and feedback are welcome.

Fix for issue #3020

Make OpEmitTMeshasksEXT a terminal instructions

Fix OpLine prepending OpFunction reports wrong file when #line is present

Merge pull request #3016 from ichordev/patch-1

Fixed a small grammatical error

Merge pull request #3015 from spnda/remove_nv_c_interface

Fix: Remove NV suffix from glslang C interface

Fix: Remove NV suffix from C interface

Merge pull request #3014 from pmistryNV/GL_EXT_mesh_shader

GL_EXT_mesh_shader/SPV_EXT_mesh_shader implementation

Fixed a small grammatical error

GL_EXT_mesh_shader/SPV_EXT_mesh_shader implementation
Added following updates to GL_EXT_mesh_shader implementation:

1. Added SPIRV and GLSL test cases
2. Added checks to ensure NV and EXT mesh shader builtins cannot be used interchangeably.
3. Updated the language name by removing the postfix "NV" to MeshShader and TaskShader.
4. Added checks for grammar checking to comply with the spec.

5. Added gl_NumWorkGroups builtin to Mesh shader
6. Fixed data type of gl_PrimitiveLineIndicesEXT and gl_PrimitiveTriangleIndicesEXT
7. Added new constants to the resources table
8. Updates to handle new storage qualifier "taskPayloadSharedEXT"
9. Updated test cases by replacing "taskEXT" with storage qualifier "taskPayloadSharedEXT"

Addressed  Review comments
1. Fixed instruction description used by glslang disassembly.
2. Updated OpEmitMeshTasksEXT as per spec update
3. Fixed implementation that errors out if there are more then one taskPayloadSharedEXT varjables.
4. Fixed miscellaneous error logs and removed unwanted code.

SPIRV 1.6 related build failure fixes
- Update SPIRV header to 1.6
- Fix conflict wiht SPIRV 1.6 change, where localSizeId is used for execution mode for mesh/task shaders

Enable SPIRV generated for EXT_mesh_shader to be version 1.4

GL_EXT_mesh_shader: Add checks for atomic support and corresponding test cases

Merge pull request #2997 from RandomShaper/remove_unused

Remove the unused `OS_CleanupThreadData`

Merge pull request #3011 from hbatagelo/fix_type_punning_ub

Fix strict aliasing violation

Fix -Wundef warnings for MINGW_HAS_SECURE_API

Since 12e27e17deb3102f1f6f08ec19caf5e32becb3f8, it was assumed that the
MINGW_HAS_SECURE_API macro was unconditionally defined. This is not
always the case, and GCC produces -Wundef warnings when that happens.

Fix this, by first checking if MINGW_HAS_SECURE_API is defined, and if
so, also check that it does not evaluate to zero.

As a drive-by, place parentheses around _MSC_VER for consistency.

Implement NonSemantic.Shader.DebugInfo.100

See https://github.com/KhronosGroup/SPIRV-Registry.

Remove the unused OS_CleanupThreadData

Fix strict aliasing violation

Merge pull request #3009 from Tachi107/cmake-config-file-libdir

build: install glslang-config.cmake to libdir

build: install glslang-config.cmake to libdir

As glslang ships architecture dependant files, the Config file should be
installed to libdir, not datadir. See
https://github.com/KhronosGroup/glslang/pull/2989#discussion_r955367103
for more details.

Here's the diff between the install tree before and after this patch:

    $ diff <(tree install-datadir) <(tree install)
    1c1
    < install-datadir
    ---
    > install
    74,99c74,98
    <         ├── lib
    <         │   ├── cmake
    <         │   │   ├── glslang-default-resource-limitsTargets.cmake
    <         │   │   ├── glslangTargets.cmake
    <         │   │   ├── glslangValidatorTargets.cmake
    <         │   │   ├── HLSLTargets.cmake
    <         │   │   ├── OGLCompilerTargets.cmake
    <         │   │   ├── OSDependentTargets.cmake
    <         │   │   ├── spirv-remapTargets.cmake
    <         │   │   ├── SPIRVTargets.cmake
    <         │   │   └── SPVRemapperTargets.cmake
    <         │   ├── libGenericCodeGen.a
    <         │   ├── libglslang.a
    <         │   ├── libglslang-default-resource-limits.a
    <         │   ├── libHLSL.a
    <         │   ├── libMachineIndependent.a
    <         │   ├── libOGLCompiler.a
    <         │   ├── libOSDependent.a
    <         │   ├── libSPIRV.a
    <         │   └── libSPVRemapper.a
    <         └── share
    <             └── glslang
    <                 ├── glslang-config.cmake
    <                 ├── glslang-config-version.cmake
    <                 ├── glslang-targets.cmake
    <                 └── glslang-targets-debug.cmake
    ---
    >         └── lib
    >             ├── cmake
    >             │   ├── glslang-default-resource-limitsTargets.cmake
    >             │   ├── glslangTargets.cmake
    >             │   ├── glslangValidatorTargets.cmake
    >             │   ├── HLSLTargets.cmake
    >             │   ├── OGLCompilerTargets.cmake
    >             │   ├── OSDependentTargets.cmake
    >             │   ├── spirv-remapTargets.cmake
    >             │   ├── SPIRVTargets.cmake
    >             │   └── SPVRemapperTargets.cmake
    >             ├── glslang
    >             │   ├── glslang-config.cmake
    >             │   ├── glslang-config-version.cmake
    >             │   ├── glslang-targets.cmake
    >             │   └── glslang-targets-debug.cmake
    >             ├── libGenericCodeGen.a
    >             ├── libglslang.a
    >             ├── libglslang-default-resource-limits.a
    >             ├── libHLSL.a
    >             ├── libMachineIndependent.a
    >             ├── libOGLCompiler.a
    >             ├── libOSDependent.a
    >             ├── libSPIRV.a
    >             └── libSPVRemapper.a
    101c100
    < 15 directories, 83 files
    ---
    > 14 directories, 83 files

Merge pull request #3000 from jeremy-lunarg/hayes-update-cmake-minimum

Update cmake minimum required version

Update cmake minimum required version

Merge pull request #2998 from jeremy-lunarg/hayes-update-changes

Update CHANGES for release 11.11.0

Update CHANGES for release 11.11.0

Merge pull request #2995 from jeremy-lunarg/hayes-update-known-good

Update known_good.json

Update known_good.json

Merge pull request #2991 from KhronosGroup/revert-2988-build_arm64_binaries_mac

Revert "Fix: Build arm64 binaries for macOS"

Revert "Fix: Build arm64 binaries for macOS"

Merge pull request #2988 from spnda/build_arm64_binaries_mac

Fix: Build arm64 binaries for macOS

Add unified `glslang` CMake config collecting `glslang-targets` targets (#2989)

Avoid double-free in functions cloned for vulkan relaxed mode (#2987)

* Avoid double-free in functions cloned for vulkan relaxed mode

When rewriting function calls atomicCounterIncrement and
atoicCounterDecrement, clone the parameters so that the TParameter
'type' field is cloned.  This avoids double-free when both the original
and transformed functions are deleted by the parser.

Fixes a ubsan failure.

Fix: Build arm64 binaries for macOS

Merge pull request #2986 from jeremy-lunarg/hayes-update-macos-runner

Update MacOS runner

Update MacOS runner

Per https://github.com/actions/virtual-environments/issues/5583

Fix #2984

Merge pull request #2977 from AMD-dwang/AMD_shader_early_and_late_fragment_tests

Add SPV_AMD_shader_early_and_late_fragment_tests

Merge pull request #2982 from jeremy-lunarg/hayes-remove-tabs

Replace tabs with spaces

Merge pull request #2981 from jeremy-lunarg/hayes-fix-2978

Update release description

Replace tabs with spaces

This file was accidentally using mixed tabs and spaces.

Update release description

Fix #2978.

Merge pull request #2979 from spnda/master

Fix #2658: Properly include all headers in deployments

Fix: Properly include all headers in deployments

Merge pull request #2974 from thoave-arm/EOpConstructAccStruct

Make GL_KHR_ray_query provide EOpConstructAccStruct

Make GL_KHR_ray_query provide EOpConstructAccStruct

Previously, GL_KHR_ray_tracing was a required extension to generate
OpConvertUToAccelerationStructureKHR conversion instructions from uint64
and uvec2. However, both GL_KHR_ray_tracing and GL_KHR_ray_query should
provide this construction.

Change-Id: I6564c127fd28d9b527d334958a5adc168f5cdd9a

Add SPV_AMD_shader_early_and_late_fragment_tests

Merge pull request #2976 from jeremy-lunarg/hayes-fix-2975

Emit Int64Atomics for imageAtomicStore

Emit Int64Atomics for imageAtomicStore

This covers a corner case wherein imageAtomicStore is used exclusively.
The proxy type for imageAtomicStore is inferred from the image type.

Fix #2975.

Merge pull request #2973 from haasn/version_macros

Fix version check macros

Fix version check macros

These were defined backwards to the usual convention.

 #if GLSLANG_VERSION_GREATER_THAN(11, 10, 0)

This reads as "if glslang version is greater than 11.10.0" to any
reasonable sane programmer, and should therefore expand to
"glslang_version > macro_argument".

Yet the check it references was actually written as "macro_argument >
glslang_version", thus expressing the completely opposite condition of
"if glslang version is *less than* 11.10.0". This is definitely
backwards and extremely, dangerously surprising behavior to any
programmer familiar with such version macros.

I'm not sure if anybody actually ever used them. I certainly didn't, on
account of them being backwards. I could not find a single reference to
them on GitHub (other than in copies of this header) - every project I
found just used the GLSLANG_VERSION_MAJOR etc. macros directly.

Merge pull request #2970 from greg-lunarg/i2969

Fix getEnhancedMsgs to work when HLSL not enabled

Fix getEnhancedMsgs to work when HLSL not enabled

Fixes #2969

Merge pull request #2963 from greg-lunarg/i2956

Do not generate samplerBuffer for spirv1.6 and beyond

Do not generate samplerBuffer for spirv1.6 and beyond

This type was removed from spirv1.6. If samplerBuffer is specified in
GLSL, generate textureBuffer. If samplerBuffer type is constructed,
just return the buffer.

Fixes #2956

Merge pull request #2962 from James2022-rgb/feature/c_interface_opsource_support

Add OpSource support to the C interface.

CInterface: Add OpSource support.

Add interface for `TIntermediate::addSourceText` and `TIntermediate::setSourceFile`.

Merge pull request #2961 from jeremy-lunarg/hayes-update-changes

Release 11.10.0

Release 11.10.0

Merge pull request #2960 from jeremy-lunarg/hayes-update-known-good

Update known_good.json

Update known_good.json

Merge pull request #2957 from greg-lunarg/p2933

Restore legacy interface for remap()

Restore legacy interface for remap()

Fixes ABI breakage caused by #2933

Merge pull request #2955 from andfau-arm/duplicate-builtin-clash

Avoid duplicate BuiltIn variables for ray tracing matrices (fix #2921)

Avoid duplicate BuiltIn variables for ray tracing matrices (fix #2921)

Fixes an issue where invalid SPIR-V was generated when
gl_ObjectToWorldEXT and gl_ObjectToWorld3x4EXT, or
gl_WorldToObjectEXT and gl_WorldToObject3x4EXT, were used in the same
shader. The SPIR-V specification requires that there be at most one
OpVariable decorated with a given BuiltIn value.

Merge pull request #2950 from qingyuanzNV/insert_opline_before_opfunction2

Generate OpLine Before OpFunction

avoid using make_unique for c++11 compatibility

use unique_ptr to avoid calling deleted move ctor

Merge pull request #2952 from stu-s/EXT_fragment_shader_barycentric

Add support for VK_EXT_fragment_shader_barycentric

Merge pull request #2953 from greg-lunarg/mingw0

Fix build for clang + mingw32-make

Fix build for clang + mingw32-make

Fixes #2951

Add support for VK_EXT_fragment_shader_barycentric

Merge pull request #2947 from corporateshark/master

Add CInterface files and documentation

Merge pull request #2948 from Gabriele91/fix/HLSL-Instance-attribute-as-spirv-invocation-attribute

HLSL Instance attribute translated as spirv invocations attribute

Add hlsl.instance.geom output

Add test for the instance param (geometry shader)