Revert "Update minimum SPIR-V requirement for GL_EXT_buffer_reference"

Merge pull request #2598 from greg-lunarg/noprop1

Do not propagate packing qualifiers to scalars or vectors

Do not propagate packing qualifiers to scalars or vectors

Packing qualifiers have no practical effect on scalars or vectors
so this is unnecessary and its confusing tools downstream that
consume the AST.

Merge pull request #2593 from jeremy-lunarg/hayes-update-min-spv-requirement

Update minimum SPIR-V requirement for GL_EXT_buffer_reference

Merge pull request #2597 from greg-lunarg/interp0

Fix SPIR-V for HLSL EvaluateAttribute* of interpolants in structs

Fix SPIR-V for HLSL EvaluateAttribute* of interpolants in structs

Generate load of interpolant for first operand to GLSLstd450
InterpolateAt* SPIR-V ops. This allows the interpolants to
propagate from the input struct in the wrapper around main
into the shader during HLSL legalization. A new pass has been
added to legalization which will remove the load and replace
with the pointer of the load to create valid external
interpolate op.

Fixes #2584

Merge pull request #2590 from pmistryNV/GL_ARB_sparse_texture2-bug-fix

For bug #2580: sparseTextureGatherOffsetsARB should only take constant offsets

For bug #2580: sparseTextureGatherOffsetsARB should only take constant offsets.

Update minimum SPIR-V requirement

Require SPIR-V 1.3 when using GL_EXT_buffer_reference.
Migrate tests to SPIR-V 1.3 fixture as necessary.
Fix extension table key.
Fix whitespace.

Merge pull request #2589 from ben-clayton/fix-warnings

Fix warnings about int comparisons of different signness

Fix warnings about int comparisons of different signness

Warnings in public headers can trigger warnings-as-errors.

Merge pull request #2588 from jeremy-lunarg/hayes-2530

Default to KHR extension

Merge pull request #2587 from mbechard/master

Don't check precision qualifiers on cross-stage checks

Default to KHR extension

Fix #2530.

Don't check precision qualifiers on cross-stage checks

Fixes #2586

Merge pull request #2583 from nullcatalyst/bazel-windows-bat

Add cmd_bat action to bazel export_spirv_headers target

Merge pull request #2582 from JustSid/master

Fixed OpGroupNonUniformQuadBroadcast Direction operand being marked as a literal

Add cmd_bat action to bazel export_spirv_headers target

Only having `cmd` rather than separate `cmd_bash` and `cmd_bat` (or
`cmd_ps`) requires bash to be installed on Windows. So adding support
for `cmd_bat` makes glslangValidator easier to compile on Windows.

Merge pull request #2581 from mbechard/InconsistentGLPerVertex

Partial fix for inconsistencies re: #2578

Fixed OpGroupNonUniformQuadBroadcast Direction operand being marked as a literal

Partial fix for inconsistencies re: #2578

gl_SecondaryPositionNV and gl_PositionPerViewNV are inconsistently
declared inside and outside of gl_PerVertex. This breaks interface block
matching. For now ignore these errors since it should be fixed with how
they are declared.

Merge pull request #2579 from mbechard/master

Fix issue with remapping global uniform blocks

Merge pull request #2574 from feifei14119/master

fix typo for u64vec2/3/4 type define value

Fix issue with remapping global uniform blocks

Avoid adding global uniform blocks to stages that don't already have it.
Otherwise multiple stages point to the same block object, and a
remapping that occurs later on will change the mapping on multiple
stages.

Merge pull request #2577 from mbechard/master

fix variable capitalization. Don't initialize it in the struct's constructor

fix variable capitilization. Don't initialize it in the struct's constructor

Merge pull request #2576 from KhronosGroup/revert-2569-revert-2242-GL_EXT_vulkan_glsl_relaxed

Revert "Revert "GL_ext_vulkan_glsl_relaxed extension support, and cross stage aware IO mapper""

Revert "Revert "GL_ext_vulkan_glsl_relaxed extension support, and cross stage aware IO mapper""

1. fix macro definition value for unsinged-int64-vector, according to kronos spec at https://khronos.org/registry/OpenGL/extensions/ARB/ARB_gpu_shader_int64.txt; 2. fix typo in reflection

Merge pull request #2569 from KhronosGroup/revert-2242-GL_EXT_vulkan_glsl_relaxed

Revert "GL_ext_vulkan_glsl_relaxed extension support, and cross stage aware IO mapper"

Revert "GL_ext_vulkan_glsl_relaxed extension support, and cross stage aware IO mapper"

Merge pull request #2565 from greg-lunarg/i2564

Pass correct proxy type for atomicStore

Merge pull request #2242 from mbechard/GL_EXT_vulkan_glsl_relaxed

GL_ext_vulkan_glsl_relaxed extension support, and cross stage aware IO mapper

Pass correct proxy type for atomicStore

Fixes #2564

Implement GL_EXT_vulkan_glsl_relaxed option

Merge pull request #2560 from amdrexu/bugfix

SPV: The capability string for RayTracingProvisionalKHR is missing

SPV: The capability string for RayTracingProvisionalKHR is missing

Add it to GLSLang disassembler.

Merge pull request #2559 from greg-lunarg/i2555

Allow DepthUnchanged and DepthReplaced Modes in same SPIR-V module.

Allow DepthUnchanged and DepthReplaced Modes in same SPIR-V module.

Fixes #2555

Merge pull request #2558 from greg-lunarg/i2551

Allow grad texture ops in all shaders

Allow grad texture ops in all shaders

Fixes #2551

Merge pull request #2557 from greg-lunarg/i2552b

Fix off-by-1 bug in gl_MaxCombinedTextureImageUnits check

Merge pull request #2554 from proydakov/Try-to-find-python-interpreter-from-host-first

Returned using find_host_package for PythonInterp 3 instead of find_pachage.

Merge pull request #2553 from amdrexu/bugfix

SPV: Change the key of extBuiltinMap to std::string

Fix off-by-1 bug in gl_MaxCombinedTextureImageUnits check

The problem was only with arrays of samplers.

Fixed #2552

Merge pull request #2556 from greg-lunarg/i2541

Fix precision propagation around shifts

Fix precision propagation around shifts

Fixes #2541

Returned using find_host_package for PythonInterp 3 instead of find_package.

I changed this logic here: https://github.com/KhronosGroup/glslang/pull/2526/commits/05798c17fb17d339d66e064a407e75ceae4c0316
It was originally fixed here: https://github.com/KhronosGroup/glslang/commit/967fa92d14acea305267574faf63ebe753de98c4

SPV: Change the key of extBuiltinMap to std::string

Using const char* is not a good choice. We assume strings are always
from spv::E_SPV_XXX definitions. However, it is not the case. We can
store the name strings of extended intruction set in local variables.

Merge pull request #2544 from ZhiqianXia/master

Replace log2 and exp2 math func.

fix a typo

Merge pull request #2549 from jeremy-lunarg/hayes-2479

Require fixed workgroup size declaration

Require fixed workgroup size declaration

Fix 2479.

Merge pull request #2546 from proydakov/msvc-noexcept-fix

Fixed msvc 2019 nmake noexcept build.

Fixed msvc 2019 nmake noexcept build.

By default cmake generates cxx_flags with /EHsc parameter.
I updated CMAKE_CXX_FLAGS string and removed /EHsc, also I added compile defenitions _HAS_EXCEPTIONS=0, it is mandatory for noexcept build with MSVC STL implementation.
Output files became smaller.

How to reproduce:

Visual Studio 2019 x64 command port

mkdir build-msvc2019
cd build-msvc2019
cmake -G"NMake Makefiles" -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_LIBDIR=install ..
nmake

CXX FLAGS BEFORE:

-- CMAKE_C_FLAGS:   /DWIN32 /D_WINDOWS /W3
-- CMAKE_CXX_FLAGS: /DWIN32 /D_WINDOWS /W3 /GR- /EHsc
-- CMAKE_CXX_FLAGS_DEBUG:   /MDd /Zi /Ob0 /Od /RTC1
-- CMAKE_CXX_FLAGS_RELEASE: /MD /O2 /Ob2 /DNDEBUG
-- ENABLE_RTTI:       OFF
-- ENABLE_EXCEPTIONS: OFF

OUTPUT SIZE BEFORE:

Build folder size: 61,8 MB (64 808 580 bytes)

GLSLANG SIZE BEFORE:

glslang.lib 22,7 MB (23 887 150 bytes)

CXX FLAGS AFTER:

-- CMAKE_C_FLAGS:   /DWIN32 /D_WINDOWS /W3
-- CMAKE_CXX_FLAGS: /DWIN32 /D_WINDOWS /W3 /GR-
-- CMAKE_CXX_FLAGS_DEBUG:   /MDd /Zi /Ob0 /Od /RTC1
-- CMAKE_CXX_FLAGS_RELEASE: /MD /O2 /Ob2 /DNDEBUG
-- ENABLE_RTTI:       OFF
-- ENABLE_EXCEPTIONS: OFF

OUTPUT SIZE AFTER:

Build folder size: 58,4 MB (61 331 179 bytes)

GLSLANG SIZE AFTER:

glslang.lib 21,6 MB (22 655 252 bytes)

Merge pull request #2537 from proydakov/fixed-cmake-warning

Fixed cmake generation warning for regeneration project build files.

Code review. Thx `greg-lunarg` for comments.

Replace the old version math func with C++11 to
prevent some presicion issue.

reference link:
https://www.cplusplus.com/reference/cmath/

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

Merge pull request #2525 from ZhiqianXia/master

Cannot specify atomic counter with location.

Merge pull request #2540 from dgkoch/dgkoch_build_fix

Fix warning in build

Merge pull request #2543 from greg-lunarg/chg11.2.0

Update CHANGES.md to 11.2.0 to sync with current tagging.

Update CHANGES.md to 11.2.0 to sync with current tagging.

Fix warning in build

glslang/MachineIndependent/SymbolTable.h:892:41: error: comparison of integers of
different signs: 'int' and 'const uint32_t' (aka 'const unsigned int') [-Werror,-Wsign-compare]
            uint64_t level = currentLevel() > MaxLevelInUniqueID ? MaxLevelInUniqueID : currentLevel();
                             ~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~
1 error generated.

Fixed cmake generation warning for regeneration project build files.

[BEFORE]:

% cmake ..
CMake Warning (dev) at CMakeLists.txt:35 (project):
  Policy CMP0048 is not set: project() command manages VERSION variables.
  Run "cmake --help-policy CMP0048" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.

  The following variable(s) would be set to empty:

    CMAKE_PROJECT_VERSION
    CMAKE_PROJECT_VERSION_MAJOR
    CMAKE_PROJECT_VERSION_MINOR
    CMAKE_PROJECT_VERSION_PATCH
This warning is for project developers.  Use -Wno-dev to suppress it.

-- No build type selected, default to Debug
-- Found PythonInterp: /usr/local/bin/python3 (found version "3.9")
-- Found PythonInterp: /usr/local/bin/python3 (found suitable version "3.9", minimum required is "3")
-- optimizer enabled
-- Google Mock found - building tests
-- Configuring done

[AFTER]:

% cmake ..
-- No build type selected, default to Debug
-- Google Mock was not found - tests based on that will not build
-- Configuring done
-- Generating done

Merge pull request #2536 from greg-lunarg/chg0

Update CHANGES for 11.1.0

Update CHANGES for 11.1.0

Merge pull request #2532 from juliusikkala/master

Pass environment through PreprocessDeferred

Merge pull request #2458 from ShchchowAMD/unique_id-fix

Fix issue for new unique id system.

Fix issue for new unique id system. Add level bits to help verifying symbols and split symbol tables.
For intermediates rebuilding, now need manually amending level bits for redeclaring built-ins.

Pass environment through PreprocessDeferred

Merge pull request #2531 from greg-lunarg/kg116

Update spirv-tools and spirv-header known good.

Update spirv-tools and spirv-header known good.

This is being done to allow Vulkan validation layers to pick up
the new differentiated error code support in spirv-tools for
GPU-AV.

Merge pull request #2526 from proydakov/regression-python3-is-required-even-without-using-SPIRV-Tools

Fixed issues 2496. Used option 4: parse_version.cmake to avoid python3 usage.

Code review. Thx ben-clayton for comments.

Fixed issues 2496. Used option 4: parse_version.cmake to avoid python3 usage.

https://github.com/KhronosGroup/glslang/issues/2496

Cannot specify atomic counter with location.

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

Cannot specify atomic counter with location.

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

Cannot specify atomic counter with location.

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

Merge pull request #2523 from dneto0/remove-stray-semi

Remove useless semicolon

Remove useless semicolon

The extra semicolon causes a build failure if warnings are turned up high, and
warnings-as-errors is on. (-Werror=extra-semi)

Merge pull request #2517 from glebm/fix-ubsan-localintermediate

callGraph.push_front -> emplace_front to fix UBSAN

Merge pull request #2512 from cmarcelo/GL_EXT_shared_memory_block

Add support for GL_EXT_shared_memory_block

callGraph.push_front -> emplace_front to fix UBSAN

UBSAN rightly complains on `push_front` here:

    glslang/MachineIndependent/localintermediate.h:100:8: runtime error: load of value 160, which is not a valid value for type 'bool'
    #0 in glslang::TCall::TCall(glslang::TCall&&) glslang/MachineIndependent/localintermediate.h:100
    #1 in void __gnu_cxx::new_allocator<std::_List_node<glslang::TCall> >::construct<glslang::TCall, glslang::TCall>(glslang::TCall*, glslang::TCall&&) /usr/include/c++/10/ext/new_allocator.h:150
    #2 in void std::allocator_traits<std::allocator<std::_List_node<glslang::TCall> > >::construct<glslang::TCall, glslang::TCall>(std::allocator<std::_List_node<glslang::TCall> >&, glslang::TCall*, glslang::TCall&&) /usr/include/c++/10/bits/alloc_traits.h:512
    #3 in std::_List_node<glslang::TCall>* std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::_M_create_node<glslang::TCall>(glslang::TCall&&) (...)
    #4 in void std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::_M_insert<glslang::TCall>(std::_List_iterator<glslang::TCall>, glslang::TCall&&) /usr/include/c++/10/bits/stl_list.h:1911
    #5 in std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::push_front(glslang::TCall&&) /usr/include/c++/10/bits/stl_list.h:1167
    #6 in glslang::TIntermediate::addToCallGraph(TInfoSink&, std::__cxx11::basic_string<char, std::char_traits<char>, glslang::pool_allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, glslang::pool_allocator<char> > const&) glslang/MachineIndependent/Intermediate.cpp:2860

What happens here:

1. TCall's bool fields are not initialized on construction.
2. `push_front` move the `TCall` passed into it.
3. The move constructor copies unitialized bool, which may have an
   out-of-range value.

What this fix does:

Calls `emplace_back` to ensure no copy/move constructor is called.

Fixes #2222
Refs #2112

Add support for GL_EXT_shared_memory_block

Uses SPV_KHR_workgroup_memory_explicit_layout.  Note that if
GL_EXT_scalar_block_layout is enabled, Workgroup blocks can also use
scalar layout.

Update known_good.json to pick up SPV_KHR_workgroup_memory_explicit_layout

Also update the test expectations regarding validation accordingly and
the in-tree spirv.hpp copy.

Consider GL_EXT_scalar_block_layout when validating SPIR-V

If GL_EXT_scalar_block_layout is requested by the shader, set the
option to allow scalar blocks in the SPIR-V validator.

Fix the existing tests using scalar layout to not expect "Validation
failed".

Fixes #2400.

Merge pull request #2513 from jeffbolznv/missingcap

Add missing capability when QueueFamily scope is used

Add missing capability when QueueFamily scope is used

Also, if this capability is added and the memory model is not
otherwise enabled by pragma, enable it as part of postprocessing.

Merge pull request #2511 from alan-baker/GL_EXT_null_initializer

Implement GL_EXT_null_initializer

Implement GL_EXT_null_initializer

Adds null initializer syntax (empty braces)
Allows null initialization of shared variables

Merge pull request #2508 from corporateshark/master

Fix missing enum values in the C-interface #2507

Fix missing enum values in the C-interface #2507

Merge pull request #2505 from ben-clayton/update-licnese-checker-cfg

license-checker.cfg: Update rules

Merge pull request #2504 from cdotstout/fuchsia-cmake

Add CMake support for Fuchsia

license-checker.cfg: Update rules

The license classification rules in github.com/google/licensecheck have been updated.

Add CMake support for Fuchsia

Merge pull request #2498 from dezinjob/fix-option-replacement-vs

Fix option replacement when flags are empty

Merge pull request #2500 from jeremy-lunarg/hayes-2497

Prevent HLSL input under OpenGL semantics

Merge pull request #2499 from greg-lunarg/i2494

Fix debugInfo test to target vulkan1.1 as intended

Prevent HLSL input under OpenGL semantics

Fix #2497.