Merge pull request #1961 from zoddicus/fixChromiumRoll

Remove std::move that is breaking chromium roll

Remove std::move that is breaking chromium roll

This is causing the following error:

moving a temporary object prevents copy elision
[-Werror,-Wpessimizing-move]

Merge pull request #1953 from null77/gn-add-missing-headers

Add a couple missing headers to the GN build.

Merge pull request #1954 from corporateshark/master

Fix warning: size_t->int implicit cast

Tests: Add more test results for the previous generator version bump.

Bump up the generator version, which is exposed in SPV test results.

Fix warning: size_t->int implicit cast

Add a couple missing headers to the GN build.

Detected with ANGLE's export_targets script.

Merge branch 'dneto0-dead-code'

Merge branch 'dead-code' of https://github.com/dneto0/glslang into dneto0-dead-code

Bump version.

Merge pull request #1949 from corporateshark/master

Add option ENABLE_CTEST to skip testing

SPIR-V: Aggressively prune unreachable merge, continue target

More aggressively prune unreachable code as follows.
When no control flow edges reach a merge block or continue target:
- delete their contents so that:
  - a merge block becomes OpLabel, then OpUnreachable
  - a continue target becomes OpLabel, then an OpBranch back to the
    loop header
- any basic block which is dominated by such a merge block or continue
  target is removed as well.
- decorations targeting the removed instructions are removed.

Enables the SPIR-V builder post-processing step the GLSLANG_WEB case.

Add option ENABLE_CTEST to skip testing

Merge pull request #1948 from greg-lunarg/kg108

Update spirv-tools known good.

Update spirv-tools known good.

Merge pull request #1947 from ehsannas/fix_licenses

Make  buildbot licenses consistent with the rest of the code

Make  buildbot licenses consistent with the rest of the code

Merge pull request #1946 from KhronosGroup/extend-web

Web: Extend with compute stage, SSBOs, and atomics on SSBO memory.

BUILD.gn: Fix fuchsia build (#1944)

The Fuchsia platform build and the Chromium one use a completely
different set of configs to specify default warnings. The previous
version of BUILD.gn worked with Chromium, but failed to build
with Fuchsia.

This CL fixes the issue by identifying said configs and reusing
them consistently.

Change-Id: I7de526a57d2f14eb93c03e06401d2c3059d35e9a

Web: Reclaim more space and make all work w/wo GLSLANG_WEB.

Web: Add basic atomics for SSBOs.

Web: Add SSBOs and a few other missing compute features.

Web: Add compute stage.

Web: Add separate texture/sampler, exclude *CubeArray*.

Looks like will add about 1K compressed size to executable footprint.

Enable generation of compile_commands.json (#1938)

Fix the exports of glslang-default-resource-limits (#1942)

Merge pull request #1941 from ehsannas/bazel_build

Add Bazel build files and bots for them

Run tests on Windows too.

Fix build scripts.

More cleanups for Windows.

Add Kokoro bots for building using Bazel.

Make it work on Windows.

Merge pull request #1937 from jeffbolznv/volatile_atomics

Only apply volatile semantics to atomics when using Vulkan Memory Model

Only apply volatile semantics to atomics when using Vulkan Memory Model

Add Bazel build configuration files.

In order to avoid potential build failures that could arise from
SPIRV-Tools/Headers, and to avoid reading known_good.json, this build
configuration does not build the SPIRV-Tools dependency and hence cannot
perform HLSL compilation to SPIR-V.

Merge pull request #1933 from dj2/runtest

Make runtests configurable.

Use commandline options instead of ENV variables

Merge pull request #1932 from zoddicus/anotherSemiColonFix

Fix config issues and revert previous fixes for semi-colon issues

Keep code style improvements

Merge pull request #1934 from durswd/master

Added an option to make pch enabled in Cmake

Indenting

Update CMakeLists.txt

Move TARGETDIR as well

Make runtests configurable.

This CL allows setting the location of glslangValidator and spirv-remap
when calling the runtests script. A test target is added to CMake to
execute runtests and sets the location to the build folder for the two
applications.

Fix config issues and revert previous fixes for semi-colon issues

This reverts commit 41261d95a417bb3a121c162154d441433705a237.
This reverts commit 7c9accb6534d4cde050555b8091b434dc7e4d218.

Instead of trying to work around all of the potential semicolon issues
in glslang, making it conform to Chromium's style, mark the code
explicitly as non-chromium in the BUILD.gn, so chromium doesn't
attempt to enforce its style rules on glslang.

Fixes #1931

Added an option to make pch enabled in Cmake

Merge pull request #1866 from rumblehhh/master

Export glslang targets on installation

Merge pull request #1930 from ux3d/fix/CMakeOptionalExternalBuild

find Python and add External subdir only if BUILD_EXTERNAL option was set

find Python and add External subdir only if BUILD_EXTERNAL option was set (on by default) and /External dir exists

GLSL/SPV: Pre SPIR-V 1.5, subgroupQuadBroadcast index must be constant

Merge pull request #1880 from georgeouzou/master

Add support for GL_ARB_explicit_uniform_location

Merge pull request #1928 from Xottab-DUTY/patch-1

Fixed typo in README.md

remove redundant check

Fixed typo in README.md

Merge pull request #1926 from zoddicus/fixChromiumRoll

Remove unnecessary semi-colons and add warning about them

Forgot that GCC doesn't recognize this flag

Remove unnecessary semi-colons and add warning about them

These are causing integration issues with Chromium down stream since
it is more strict about these.

Merge pull request #1888 from Roy-AMD/Adjusting-code-interface

Adjusting code interface

Merge pull request #1925 from greg-lunarg/kg107

Update spirv-tools known good.

Fix #1924: Promote SPV_EXT_physical_storage_buffer to KHR when required.

If the semantics that require the KHR form over the EXT form are seen
(OpBitcast between a vector and a pointer) promote the requested extension
from the EXT to the KHR.

Update spirv-tools known good.

Fix #1924: Emit SPV_EXT_physical_storage_buffer, not the KHR form.

The KHR form comes along as part of 1.5, and otherwise should require
another form of request to emit it.

Merge pull request #1919 from dj2/install_spirv

Move install directory for SPIRV/ folder.

single line

Update appveyor and travis files

Move install directory for SPIRV/ folder.

Currently the SPIRV/ folder will get installed into the include
directory. This folder is part of GLSLang, so it makes more sense under
glslang/SPIRV.

Currently, GLSLang will install a SPIRV/ folder while spirv-headers will
install a spirv/ folder. This is confusing and will cause issues on a
case sensitive filesystem if both are installed at the same time.

Merge pull request #1918 from KhronosGroup/format-attributes

HLSL: Add attributes for image formats and nonreadable/nonwritable

HLSL: Fix #1912: add attribute syntax for nonreadable/nonwritable

HLSL: Fix #1912: add attribute syntax for overriding image formats.

Merge pull request #1906 from ShchchowAMD/master

Reflection will crash when the VS input symbol defines the same name with FS output symbol

Merge pull request #1910 from KhronosGroup/fix-1900-const-nonuniform

GLSL/SPV: Fix #1900: Drop const on literal when doing an object copy.

GLSL/SPV: Fix #1900: Drop const on literal when doing an object copy.

Merge pull request #1907 from kainino0x/separate-web-and-emscripten

Separate GLSLANG_WEB (min-size build) and Emscripten options

Merge pull request #1905 from KhronosGroup/GL_EXT_buffer_reference_uvec2

SPV_KHR_physical_storage_buffer/SPV: Add GL_EXT_buffer_reference_uvec2

SPV_KHR_physical_storage_buffer/SPV: Add GL_EXT_buffer_reference_uvec2

Adds uvec2 <-> reference constructor support.
Switches from EXT to KHR for physical_storage_buffer.

Merge pull request #1908 from digit-google/fuchsia-fixes

Fix Fuchsia build.

Fix Fuchsia build.

The Fuchsia build is very picky about newlines at the end of files
and will complain loudly about them. Removing the -Wnewline-eof
warning solves the issue.

Merge branch 'master' into separate-web-and-emscripten

use custom es6 modularization instead of MODULARIZE=1

Separate GLSLANG_WEB (min-size build) and Emscripten options

Allows building non-min-size builds with Emscripten.
Adds ENABLE_GLSLANG_WEB_DEVEL.
Moves the glslang.js files to OSDependent/Web.
Small cleanups and docs update.

README: Fix WASM typos.

HLSL: Fix #1903 Catch 0-argument case to constructors.

Reflection will crash when the VS input symbol defines the same name with FS output symbol

[PURPOSE]:
The current process design for Uniform / Block / Pipe IO symbols reflection (during program linking) is as following :

1.1 using a global mapper called 'TNameToIndex' to store all the relationship of name (of symbols) to their indexes (in their own MapIndexToReflection vectors).

1.2 TNameToIndex mapper will be used during program linking and helps to check and merge duplicate symbols within each stage ( Uniform, Block and Pipe IO)

1.3 Different types of symbols will have their own index mapping storage. All those symbols will share TNameToIndex as a general searching mapper.

1.4 Only IN in first stage and OUT in last stage will be dealed within traversing functions.

Now, here we meet those problems:

2.1 In and Out variables for pipelines are mapping to different MapIndexToReflection vector (ioItems), but they may still have same names within the general symbol search mapper : TNameToIndex.

2.2 Then, when there are same symbols of IN in VS and OUT in FS, TNameToIndex could not tell the difference because it only stores one local index for one symbol (1:1) as a pair of KeyValue.

[What fixed]:

Seperate I/O from other symbols like Uniform and Block (it is wrong to keep them all in TNameToIndex), and save in new searching mappers called pipeInNameToIndex and pipeOutNameToIndex.

Expose new top-level functions defined as getReflectionPipeIOIndex and getPipeIOIndex for users who need to query Pipe I/O information (As they may reach those things through getUniformIndex and getReflectionIndex now, which is a confused way.)

As there are 2 mappers for above symbols, users needs to input second argument when they wanna reach those pipe I/O parameters, that's also why we need to modify GET functions either.

[Test Case]:

The shader is as following:

######### VS ############
layout(location = 0) in vec4 g_position;
layout(location = 1) in vec4 g_color;
out StageData {
vec4 color;
} g_vs_out;
void main() {
gl_Position = g_position;
g_vs_out.color = g_color;
}

########### FS #############
in StageData {
vec4 color;
} g_fs_in;
layout(location = 0) out vec4 g_color;
void main() {
g_color = g_fs_in.color;
}

Merge pull request #1902 from jeffbolznv/extended_types

Add GL_EXT_shader_subgroup_extended_types support

Add GL_EXT_shader_subgroup_extended_types support

Merge pull request #1899 from KhronosGroup/SPIR-V_1.5

Branch to support SPIR-V 1.5 changes

GLSL: Only require constant for subgroupBroadcast when SPV < 1.5.

SPV: Support SPIR-V 1.5; five extensions no longer need OpExtension.

The generalization to addIncorporatedExtension() also fixed a 1.3
corner case with SPV_KHR_16bit_storage.

Merge pull request #1894 from ShchchowAMD/master

Add flags for local size values ( compute shader )

Add flags for local size values ( compute shader )

Purpose :

According to GLSL SPEC 4.6 ( 4.4.1.4 Compute Shader Inputs), for compute shader input qualifiers, we should declare such qualifiers with same values in the same shader (local_size_x, y and z).
"If such a layout qualifier is declared more than once in the same shader, all those declarations must set the same set of local work-group sizes and set them to the same values; otherwise a compile-time error results."

Why this fix:

If we manually set "local_size_x = 1" and directly following a declaration like "local_size_x = 2", this would not be detected. That is because currently we treat all the '1' as default value and could not restrictly detect whether those are default values.

Test case:
......
layout(local_size_x=1) in;
layout(local_size_x=2) in;
......

So I add test cases for this fix:
1. set local_size_y = 1 => success
2. set local_size_y = 2 => error
3. set local_size_y = 1 => success

SPV 1.5: Switch to the 1.5 header, for SPIR-V 1.5.

Merge pull request #1897 from dj2/unused

Comment out unused params; Fix type warning.

Merge pull request #1895 from amdrexu/bugfix

Fix incorrect function prototypes of 64-bit findLSB/findMSB

Comment out params instead of removing

Remove unused params

Fix incorrect function prototypes of 64-bit findLSB/findMSB

According to the 32-bit counterparts, their forms should be
genI64Type findLSB(genI64Type value)
genI64Type findLSB(genU64Type value)
genI64Type findMSB(genI64Type value)
genI64Type findMSB(genU64Type value)

Fix #1879: Check for valid variable before checking for unsized arrays.

The order of error checking was not quite being correct (maybe there is no correct
ordering, when many checks must be done and they affect each other).
So, check for block-name reuse twice.

Merge pull request #1892 from greg-lunarg/kg106

Update spirv-tools and spriv-headers known good.

Update spirv-tools and spriv-headers known good.