Merge pull request #2759 from mbechard/vkRelaxed-RetargetID

GL_EXT_vulkan_glsl_relaxed - retarget gl_VertexID to gl_VertexIndex

Merge pull request #2756 from mbechard/vkRelaxed-AvoidDuplicateUniform

GL_EXT_vulkan_glsl_relaxed - avoid growing the global uniform block with duplicates

GL_EXT_vulkan_glsl_relaxed - retarget gl_VertexID to gl_VertexIndex

instead of allowing for multiple declarations of the variable in the
resulting SPIR-V, instead use a retargeted mechanism to cause references
to gl_VertexID and gl_InstanceID to use the gl_VertexIndex and
gl_InstanceIndex symbol.

avoid growing the global uniform block with duplicates

When using GL_EXT_vulkan_glsl_relaxed, check to see if a
uniform already exists in the block uniform block, and if so,
ensure they are the same type. Otherwise, avoid growing the block
with a duplicate.

Merge pull request #2745 from jeremy-lunarg/hayes-fix-warnings

Fix GCC warnings

Merge pull request #2752 from InsertAReallyCreativeNameHere/master

Change MINGW_HAS_SECURE_API checks.

Merge pull request #2755 from greg-lunarg/2748

Run update_precision() on array and matrix constructors.

Run update_precision() on array and matrix constructors.

Change MINGW_HAS_SECURE_API checks.

MINGW_HAS_SECURE_API can be defined as 0, but this will be ignored here without this change.
Without these *_s "safe" functions, this code will also build on Windows XP.

Merge pull request #2747 from greg-lunarg/i2740

Perform update_precision on constructors and converts

Perform update_precision on constructors and converts

Fixes #2740

Fix GCC warnings

Fix -Wsign-compare warnings.
Fix -Wunused-parameter warnings.

Merge pull request #2738 from greg-lunarg/kg119

Update known goods and CHANGES for 11.6.0

Update known goods and CHANGES for 11.6.0

Merge pull request #2718 from ZhiqianXia/AtomicOpsARB

Support the atomic_counter_xxxARB  built-in functons at #extension GL_ARB_shader_atomic_counter_ops.

Merge pull request #2731 from alelenv/rt_motion

Add support for GL_NV_ray_tracing_motion_blur.

Update known_good for SPV_NV_ray_tracing_motion_blur.

Merge branch 'KhronosGroup:master' into rt_motion

Add support for GL_NV_ray_tracing_motion_blur.

Merge pull request #2726 from TheMindVirus/master

Update CMakeLists.txt

Merge pull request #2729 from greg-lunarg/2725

Fix seperate stores to swizzled lvalue.

Fix seperate stores to swizzled lvalue.

Seen with += and ++ ops on swizzled lvalues. Was using stale access chain.

Fixes 2725.

Update CMakeLists.txt

Checked REGEX REPLACE for consistency failure

Support Extension GL_ARB_shader_atomic_counter_ops and relative intrisic functions.

refs:
https://www.khronos.org/registry/OpenGL/extensions/ARB/ARB_shader_atomic_counter_ops.txt

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

Merge pull request #2716 from greg-lunarg/i2689

Add support for pragma STDGL invariant(all)

Add support for pragma STDGL invariant(all)

Fixes #2689

Merge pull request #2715 from greg-lunarg/i2714

Allow layout(std430) uniform with GL_EXT_scalar_block_layout

Merge pull request #2690 from AndreyMlashkin/make_msvc_runtime_override_optional

add possibility to not override runtime of MSVC

Allow layout(std430) uniform with GL_EXT_scalar_block_layout

correct ident

Merge pull request #2691 from ZhiqianXia/BugPoint

Atomic memory function can only be used for shader storage block member or shared variable.

Merge branch 'master' into BugPoint

Merge pull request #2700 from ZhiqianXia/u2i

Support Uint to Int implicit conversions at #extension GL_ARB_gpu_shader5.

Merge pull request #2702 from jeremy-lunarg/hayes-2518

Generate separate stores for partially swizzled memory stores

Generate separate stores for partially swizzled memory stores

Full vector and fully specified vector swizzle stores are not affected by this change, only partial swizzles ie swizzles with fewer components than the vector being stored to.

Previously the vector being stored to loaded and any components not specified in the swizzle were used to create a full store to the vector.

While this change generates more SPIR-V instructions, it is necessary for correctness.

Fixes #2518.

Fix OpImageRead result type when compiling HLSL (#2706)

Fix OpImageRead result type when compiling HLSL

- Per the Vulkan spec, OpImageRead must return a 4-component vector always. When compiling HLSL, loads from a RWTexture of a template type with < 4 components would incorrectly generate an OpImageRead with a < 4 component result, resulting in validation errors.
- This was previously fixed for OpImageFetch in commit 4425f24; this commit does the same thing for OpImageRead.
- Added associated tests and expanded existing image fetch tests to check all the different types of textures, in both float and int incarnations, for completeness.
- Update other HLSL tests involving OpImageRead

Merge pull request #2705 from pkasting/master

Fix some instances of -Wunused-but-set-variable.

Merge pull request #2704 from YunHsiao/master

Fix per-set argument parsing in standalone

Fix some instances of -Wunused-but-set-variable.

Bug: chromium:1203071

Fix per-set argument parsing in standalone

Support Uint to Int implicit conversions at #extension GL_ARB_gpu_shader5.

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

Merge pull request #2695 from ShabbyX/fix-bool-array-in-uniform-load

Fix loading bool arrays from interface blocks

Fix loading bool arrays from interface blocks

SPIR-V disallows bool in interface blocks, which is emulated with uint.
When loading a bool variable (through accessChainLoad()), it's converted
from uint to bool if it came from an interface block.

This was handled for bool and bvecN, but not for bool arrays.  This
change implements the conversion for bool arrays.

Closes #2694

make AND to upper case

add possibility to not override runtime of MSVC

Atomic memory function can only be used for shader storage block member or shared variable.

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

Merge pull request #2688 from KhronosGroup/revert-2643-master

Revert "Atomic memory function can only be used for shader storage block member or shared variable."

Revert "Atomic memory function can only be used for shader storage block member or shared variable."

Merge pull request #2684 from ZhiqianXia/Dev

Add support for gl_MaxVaryingVectors for ogl.

Merge pull request #2643 from ZhiqianXia/master

Atomic memory function can only be used for shader storage block member or shared variable.

Add support for gl_MaxVaryingVectors for ogl.

Merge branch 'KhronosGroup:master' into master

Merge pull request #2671 from greg-lunarg/kg118

Update known goods and CHANGES for 11.5.0

Update known goods and CHANGES for 11.5.0

Merge pull request #2670 from dneto0/fix-android-mk

Android.mk: Add SpirvIntrinsics.cpp

Android.mk: Add SpirvIntrinsics.cpp

It was missing, and was breaking the build for Android.mk.

Merge pull request #2669 from sudonatalie/remove-unused-var

Remove unused variable Num_string_literal_EXTs

Merge pull request #2664 from mbechard/master

only declare compatibility gl_ variables in compatibility mode

Remove unused variable Num_string_literal_EXTs

Remove an unused variable that causes build failures in downstream
repository.

Merge pull request #2668 from cmarcelo/GLSL_EXT_shader_atomic_float2

Implement GLSL_EXT_shader_atomic_float2

Merge pull request #2625 from amdrexu/feature

Implement the extension GL_EXT_spirv_intrinsics

Merge pull request #2667 from greg-lunarg/i2666

Remove output variables from compute regression tests

Implement GLSL_EXT_shader_atomic_float2

Remove output variables from compute regression tests

Output variables in GLCompute shaders is not supported in Vulkan.
Recent upgrade of spirv-tools revealed this problem.

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

Update SPIRV-Tools and SPIRV-Headers

Update test expectations

Update known good SPIRV-Tools and SPIRV-Headers

Merge pull request #2665 from JustSid/master

Fix SPIR-V remapper not remapping OpExtInst instruction set IDs

Fixed SPIR-V remapper not remapping OpExtInst instruction set IDs

only declare compatibility gl_ variables in compatibility mode

avoid declaring them in GLSL 1.50+ if core profile is chosen by the
version statement

fixes #2663

Merge pull request #2662 from greg-lunarg/spec1

Add support for float spec const vector initialization

Add support for float spec const vector initialization

Fixes #2025

Implement the extension GL_EXT_spirv_intrinsics

- Add support of SPIR-V execution mode qualifiers.
- Add support of SPIR-V storage class qualifier.
- Add support of SPIR-V decorate qualifiers.
- Add support of SPIR-V type specifier.
- Add support of SPIR-V intruction qualifiers.
- Add support of spirv_by_reference/spirv_literal parameter qualifier.
- Add shader stage macros introduced by this extension.

Merge pull request #2661 from jaebaek/update_spirv_tools_and_unit_tests

Update spirv tools and unit tests

Fix unit test failures

Update known_good.json for SPIRV-Tools

Merge pull request #2659 from alan-baker/subgroup-uniform-control-flow

Implement GL_EXT_subgroup_uniform_control_flow.

Implement GL_EXT_subgroup_uniform_control_flow.

Merge pull request #2653 from syntheticmagus/selectionBlockSetters

Add setters for true and false blocks of selection nodes, copying the pattern already used for the condition.

Merge pull request #2656 from ShabbyX/fix-oob

Fix OOB write in matrix constructor

Merge pull request #2655 from ShabbyX/fix-ubsan

Fix UBSAN error with negating 0x8000'0000

Fix UBSAN error with negating 0x8000'0000

Signed-off-by: Shahbaz Youssefi <ShabbyX@gmail.com>

Fix OOB write in matrix constructor

In a matrix constructor that takes a number of components, as many
components as necessary must be taken, with the rest discarded, as GLSL
allows more components than necessary to be specified.  For example, the
following:

    mat4 m4 = mat4(v4, v4.yzwx, v4.zwx, v4.zwxy, v4.wxyz);

is equivalent to:

    mat4 m4 = mat4(v4, v4.yzwx, v4.zwx, v4.zwxy, v4.w);

glslang takes the components from the constructor and builds the single
components of the matrix in a 2D array before constructing the matrix
itself.  It however did not check for extra parameters and was thus
writing OOB to said 2D array.  This is fixed in this change

Signed-off-by: Shahbaz Youssefi <shabbyx@gmail.com>

Add setters for true and false blocks of selection nodes, copying the pattern already used for the condition.

Merge pull request #2651 from greg-lunarg/rel0

Add missing libs to release

Add missing libs to release

GenericCodeGen, MachineIndependent and glslang-default-resource-limits

Fixes #2648

Merge pull request #2647 from greg-lunarg/arg2

Do true SPV type check for function array arg linkage

Do true SPV type check for function array arg linkage

Previous check was missing type difference between uniform array
actual arg with stride decoration and the formal arg without. Now
does logical or component-wise copy where needed.

Fixes #2637

Merge pull request #2646 from johnstiles-google/parseconst

Fix mat4x2(scalar) constructor.

Fix mat4x2(scalar) constructor.

Atomic memory function can only be used for shader storage block member or shared variable.

The glsl spec says:A shader will fail to compile if the value passed to the mem argument of an atomic memory function does not
correspond to a buffer or shared variable.

Merge pull request #2641 from tgfrerer/fix_hlsl_function_error_msg

fix error message for hlslGrammar::acceptConstructor

Merge pull request #2634 from xantares/patch-1

Dont export inline TObjectReflection::getType()

fix error message for hlslGrammar::acceptConstructor

Fix the error message for when an erroneous HLSL constructor statement
is detected.

Prior to this change, such error messages would not show correct file
path and line number information.

Additionally, update test data to account for updated error messages.

Dont export inline TObjectReflection::getType()

Fixes mingw build
Closes #2460

Merge pull request #2632 from greg-lunarg/karl0

Fix arrays dimensioned with spec constant sized gl_WorkGroupSize

Fix arrays dimensioned with spec constant sized gl_WorkGroupSize

Merge pull request #2631 from godlikepanos/master

Add support for 64bit integer types and 64bit integer vector types to bitCount() builtin

Merge pull request #2629 from okuoku/silence-cygwin-warning

Set CMake policy CMP0054 to NEW to silence warning

Add support for 64bit integer types and 64bit integer vector types to bitCount() builtin.

Fixes #2630