Merge pull request #2800 from mhillenbrand/string_literal_endianness

Fix encoding/decoding of string literals for big-endian systems

Merge pull request #2849 from jeremy-lunarg/hayes-fix-issue-2848

Fix seg fault

Fix seg fault

Check types before accessing typeName.

Fix #2848.

Merge pull request #2845 from greg-lunarg/sup1.6

Add support for spirv1.6

Add support for spirv1.6

Add command line support which enables the following behavior:

- Remap discard

  Map discard to DemoteToHelperInvocation for HLSL shaders. Map to
  OpTerminateInvocation for GLSL shaders.

- Decorate HelperInvocation with Volatile

- Use localSizeId for execution mode

  WorkGroupSize is deprecated in spirv1.6

Also update known goods to SPIRV 1.6

Merge pull request #2844 from greg-lunarg/hdr1.6

Pickup header for SPIR-V 1.6

Pickup header for SPIR-V 1.6

Merge pull request #2840 from alelenv/no_location2

Do not output location decoration for certain variables in ray tracing storage classes.

Merge pull request #2836 from chaoticbob/vulkan_rules_relaxed

Vulkan rules relaxed for glslang C interface

Merge pull request #2841 from greg-lunarg/i2244a

Fix --hlsl-dx-position-w to work for SV_Position embedded in struct

Fix --hlsl-dx-position-w to work for SV_Position embedded in struct

Fixes #2244

Do not output location decoration for certain variables in ray tracing storage
classes.

Merge pull request #2838 from jeremy-lunarg/hayes-remove-badge

Remove unnecessary badge

Remove unnecessary badge

I had copy/pasted this badge without really thinking about it's
usefulness. It is unnecessary.

Fixed copy/paste error

Changed Vulkan rules relaxed to use shader options

- Removed vulkan_rules_relaxed from glslang_input_s
- Added GLSLANG_SHADER_VULKAN_RULES_RELAXED to glslang_shader_options_t
- Modified glslang_shader_set_options to handle new enum value
- Corrected enum value for GLSLANG_SHADER_AUTO_MAP_LOCATIONS

Merge pull request #2830 from jeremy-lunarg/hayes-migrate-travis-to-github-actions

Migrate travis to github actions

Vulkan rules relaxed for glslang C interface

- Added vulkan_rules_relaxed flag to glslang_input_t to turn on Vulkan
  rules relaxed during shader creation.

Support for automap options and resource binding shift (#2834)

* Support for automap options

- Added glslang_shader_options_t with enums for auto map bindings and
  auto map locations.
- Added options param to glslang_shader_parse.
- Modified glslang_shader_parse to call appropriate auto map function if
  auto map bits are set.

* Refactored auto map for C interface

- Added glslang_shader_set_options to independently set options instead
  of being a param on an existing function.
- Added glslang_program_map_io to call mapIO so auto map location
  actually works.

* Added support for shifting resource bindings

- Added resource binding shift functions to match C++ interface

* Uncommented preprocessor call that was commented out for debugging

Merge pull request #2832 from amdrexu/bugfix

Fix validation failures of test cases regarding GL_EXT_spirv_intrinsics

Fix validation failures of test cases regarding GL_EXT_spirv_intrinsics

1. The test cases of OpReadClockKHR is invalid. The return type is
   unsigned integer rather than signed integer.
2. When SPIR-V decorate or SPIR-V type is specified, we should avoid
   auto location mapping because the semantics are totally decided by
   SPIR-V tokens.

Merge pull request #2720 from ZhiqianXia/GL_ARB_fragment_coord_conventions

Support the #extension GL_ARB_fragment_coord_conventions.

Merge branch 'master' into GL_ARB_fragment_coord_conventions

Merge pull request #2831 from amdrexu/bugfix

Fix an issue of spirv_by_reference

Fix an issue of spirv_by_reference

When using this qualifier for a parameter, we make it as a pointer.
However, the function TranslateStorageClass() is therefore called and
the storage class should only be set to Function when it is invoked
to translate parameter types rather than actual argument types.

Migrate travis to github actions

Merge pull request #2723 from ZhiqianXia/textureQueryLOD

Support the textureQueryLOD at #extension GL_ARB_texture_query_lod.

Merge branch 'master' into textureQueryLOD

Merge pull request #2829 from amdrexu/bugfix2

Remove the test spv.intrinsicsSpecConst

Remove the test spv.intrinsicsSpecConst

According to current SPIR-V spec, we couldn't write a valid case that
references OpExecutionModeId and OpDecorateId.

Merge pull request #2782 from kevin-mccullough/FixIomapperSeperableShaderValidation

Fix issue with separable shader validation in iomapper

Merge pull request #2826 from amdrexu/bugfix

Fix an issue of spirv_type used in local variable definitions

Merge pull request #2811 from mbechard/master

remove unneeded extra constructions now that pool allocation is fixed

Fix an issue of spirv_type used in local variable definitions

We previously use createOp() in SPV builder to create type declaration.
However, all type declarations should be placed in const-type-variable
declaration section. And duplicated type defintions ought to be avoided.
We now make a method in SPV builder to perform this operation with a
more general solution: makeGenericType().

remove unneeded extra constructions now that pool allocation is fixed

Fix issue with separable shader validation in iomapper

Merge pull request #2824 from alan-baker/update-tools

Update SPIRV-Tools

Merge pull request #2825 from greg-lunarg/i2244

Add --hlsl-dx-position-w option

Add --hlsl-dx-position-w option

This reciprocates the w component of SV_Position in HLSL fragment shaders
to provide DirectX compatibility for HLSL shaders in Vulkan.

Fixes #2244

Update SPIRV-Tools

* Fix test expectations

Merge pull request #2823 from kda/master

Initialize member TSymbol::uniqueId.

Initialize member TSymbol::uniqueId.

Merge pull request #2821 from greg-lunarg/conv0

[RFC] add more negative float conversions to test constantUnaryConversion.comp and remove the undefined ones

remove undefined conversions from test constantUnaryConversion.comp

Remove remaining conversions from negative float64_t to unsigned
integers, which are undefined behavior.

As a result, this test will also succeed on platforms that implement
those conversions differently than x86. That addresses one of the issues
in #2815.

add negative float conversions to test constantUnaryConversion.comp

Add conversions from negative float16_t and float32_t to bool, all
signed integer types (i.e., including those in
GL_EXT_shader_explicit_arithmetic_types), and all float types (from the
same extension) to extend coverage.

Note that converting negative float values to unsigned integers is
undefined behavior. Thus, we exclude them.

Merge pull request #2806 from ZhiqianXia/bestFunctionMatching

#extension GL_ARB_gpu_shader5 support the implicit conversion ,

Merge pull request #2818 from mhillenbrand/promoteConstantUnion_signed_char

TIntermediate::promoteConstantUnion(): fix conversion to int8

Merge pull request #2819 from greg-lunarg/i2812

Fix struct type sharing

Merge pull request #2814 from mhillenbrand/isinf_isnan2

Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding (updated)

Fix struct type sharing

Fixes #2812

Merge pull request #2817 from greg-lunarg/sdk0

Revert port GL_EXT_shader_realtime_clock to GL_EXT_spirv_intrinsics

TIntermediate::promoteConstantUnion(): fix conversion to int8

The signedness of type char is implementation-defined in C++. The
conversion to (signed) int8 currently uses a cast to char, which is
undefined for negative values when the type char is implemented as
unsigned. Thus, fix to cast to "signed char", which has the intended
semantic on all implementations.

Fixes #2807

Revert port of GL_EXT_shader_realtime_clock to GL_EXT_spirv_intrinsics

Could not be found by glslang library interface. Blocking SDK build.

Merge pull request #2816 from greg-lunarg/v11.7.0

Release 11.7.0

Merge pull request #2813 from greg-lunarg/kg120

Update to latest spirv-headers

Release 11.7.0

Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding (updated)

There were two implementations of isInf() and isNan(), in Constant.cpp
and in intermOut.cpp. The former only works on little-endian systems,
the latter is a wrapper for library functions and works regardless of
endianness. Move the second version into Common.h and adopt it in both
places. Thereby avoid the duplication and fix for big-endian systems.

A previous commit with the same intent and purpose had missed a required
header for builds on Windows.

On s390x, this fixes the test case
Glsl/CompileToAstTest.FromFile/constFold_frag.

Fixes #2802

Update to latest spirv-headers

Merge pull request #2810 from greg-lunarg/push1

Generate error for binding on push_constant

Merge pull request #2809 from KhronosGroup/revert-2803-isinf_isnan

Revert "Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding"

Generate error for binding on push_constant

Revert "Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding"

Merge pull request #2805 from greg-lunarg/push0

Fix test spv.1.4.OpEntryPoint.frag

Support the #extension GL_ARB_fragment_coord_conventions.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>

#extension GL_ARB_gpu_shader5 support the implicit conversion ,
So the best function matching algorithm should be actived.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>

Fix test spv.1.4.OpEntryPoint.frag

Cannot apply binding qualifier to push_constant

Merge pull request #2781 from kevin-mccullough/FixLinkTimeValidationForGl_PerVertex

Fix incorrect link time validation for unused gl_PerVertex members

Merge pull request #2801 from haasn/thread_safety

Thread safety fixes

Merge pull request #2803 from mhillenbrand/isinf_isnan

Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding

Use intermOut.cpp's IsNan and IsInfinity for parse-time constant folding

There were two implementations of isInf() and isNan(), in Constant.cpp
and in intermOut.cpp. The former only works on little-endian systems,
the latter is a wrapper for library functions and works regardless of
endianness. Move the second version into Common.h and adopt it in both
places. Thereby avoid the duplication and fix for big-endian systems.

On s390x, this fixes the test case
Glsl/CompileToAstTest.FromFile/constFold_frag.

Fixes #2802

Merge pull request #2799 from cyanreg/master

Add support for targeting Vulkan 1.2 in the C API

Merge pull request #2798 from alan-baker/update-tools

Update tools

Fix incorrect link time validation for unused gl_PerVertex members

Merge pull request #2722 from ZhiqianXia/GL_ARB_draw_instanced

Support the #extension GL_ARB_draw_instanced.

Don't release global mutex before initialize/finalize is done

Otherwise this can race with other threads for access to the fields it's
supposed to be initializing/finalizing.

For example, imagine another thread is calling ShInitialize() while the
first thread is calling ShFinalize() - the finalize function would
destroy the state at the same time as the initialize function is trying
to initialize it.

Holding on to the global lock for the entire function prevents all of
these failure modes.

Initialize global mutex in a thread-safe manner

Currently, ShInitialize() and friends call glslang::InitGlobalLock()
which *overwrites* the global mutex. As such, even though it ostensibly
takes a mutex, this function is actually completely thread-unsafe.

Fix it by using pthread_once to ensure the mutex is only initialized
once, and then never again.

Fix encoding/decoding of string literals for big-endian systems

Per SPIR-V spec, a string literal's UTF-8 octets are encoded packed into
words with little-endian convention. Explicitly perform that encoding
instead of assuming that the host system is little-endian.

Note that this change requires corresponding fixes in SPIRV-Tools.

Fixes #202

Add support for targeting Vulkan 1.2 in the C API

Support the #extension GL_ARB_draw_instanced.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>

Update test expectations

Update SPIRV-Tools and SPIRV-Headers

Merge pull request #2719 from ZhiqianXia/GL_EXT_shader_integer_mix

Add the preamble for GL_EXT_shader_integer_mix

Add the GL_EXT_shader_integer_mix Preamble for glsl.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>

Merge pull request #2712 from ZhiqianXia/switch_error

glslang should report a error for Feature: last case/default label not followed by statements'.

To be compatible with Feature: 'last case/default label not followed by statements'.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>

Merge pull request #2796 from greg-lunarg/i2793

Remove support for double trig, pow, exp and log

Remove support for double trig, pow, exp and log

These are not supported in core GLSL or under any extension

Fixes 2793

Merge pull request #2792 from jeremy-lunarg/hayes-2760

Return new allocator for copied containers

Return new allocator for copied containers

Fix #2760.

Implement the optional function select_on_container_copy_contruction to
return a default-contructed allocator for containers that are
copy-constructed. This gives copy-constructed containers a pool
allocator for the current thead.

There may be a similar problem with the copy contructor which takes
allocators of type "Other" but, in practice, there is only one place
where this is being used and the allocators are always the same. (i.e.
executing from the same thread)

Merge pull request #2780 from kevin-mccullough/FixCrossStageCheck

Fix cross-stage check to allow subsequent stage to use previous stage's binding

Merge pull request #2777 from amdrexu/bugfix

Make modifications of GL_EXT_spirv_intrinsics

Merge pull request #2787 from greg-lunarg/i2779

Allow 8/16-bit integer as array index

Merge pull request #2788 from alelenv/iomapper_sr_fix

Skip auto decorating shader record buffer blocks with 'set' and 'binding'.

Merge pull request #2789 from greg-lunarg/i2785

Accept gl_ViewportMask in version 430 and later

Allow 8/16-bit integer as array index

Also enable 8/16 bit int capability in SPIR-V in such cases.

Also enable 64 bit capabilities when used in operations.

Fixes #2779

Accept gl_ViewportMask in version 430 and later

Was previously accepted only in 450 or later.

Fixes #2785

Skip auto decorating shader record buffer blocks with 'set' and
'binding'.

Merge pull request #2786 from dneto0/current-source-dir

Fix Cmake rule for extension header generation

Fix Cmake rule for extension header generation

Allow glslang to be embedded in an enclosing project (such as Shaderc)

Merge pull request #2783 from chrisforbes/untangle

Untangle use of core glslang version enums in SpvPostProcess