Only apply volatile semantics to atomics when using Vulkan Memory Model

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.

Fix code style issue and remove setXfbBufferStride new paramte

1. Keep the curly braces style.
2. revert a Improper change

Merge pull request #1889 from Roy-AMD/code-refine

Code refine

code refine

Reduce the number of cycles.

Adjusting code interface

glslang/include/intermediate.h -> Add a new interface to set TIntermBranch's expression.

glslang/include/Types.h -> Add interface to set Type's basicType and add interface to get basicType form a TSampler.

glslang/MachineIndependent/intermediate.cpp -> Part of the code in createConversion been encapsulating as a new function called buildConvertOp

glslang/MachineIndependent/localintermediate.h -> Export createConversion and
buildConvertOp as a public function

glslang/Public/ShaderLang.h -> Add interface to get shader object and shader source.

Merge pull request #6 from Roy-AMD/sync

Sync code

Merge pull request #5 from KhronosGroup/master

Sync code from KhronosGroup/glslang master

Placeholder fix for part of #1870.

Also fixes, in practice, https://github.com/KhronosGroup/GLSL/issues/83.
When the specification language is correctly created, glslang can be
revisited for correctness.  In the meantime, this seems like the best
"bug" to have relative to the specification.

Memory qualifiers are only relevant to parameters when they apply
to what the argument points to, as otherwise the argument is copied.

This leaves the fix from #1870 in place, and then more correctly
ignores memory qualifiers when something will be passed by copy.

Non-functional: Make whitespace/braces consistent for a recent commit.

Merge pull request #1885 from zoddicus/fixUnInitializedVariableWarnings

Initialize variable to avoid uninitialized variable warnings in Clang

Change to initializing the variable

Return nullptr after assert to avoid uninitialized variables

In the current version of the code on non-debug builds these cases
will fallthrough, since assert is a no-op, and eventually make a call
passing in |op| which hasn't been initialized. clang is currently
throwing a warning about this behaviour when integrating downstream.

This patch changes the behaviour, so that in any branch that has an
assert now has a return nullptr, to indicate failure after it and
avoid the uninitialized variable usage.

Merge pull request #1876 from jeffbolznv/imma

GL_NV_integer_cooperative_matrix support

ESSL/SPV: Fix #1856: Allow ESSL shaders to compile to OpenGL SPIR-V.

Documentation: Provide more detail in setting up the environment.

Merge pull request #1883 from Kangz/fix_gn

BUILD.gn: Add missing HLSL files.

BUILD.gn: Add missing HLSL files.

Merge pull request #1881 from baldurk/buffer-array-bind-reflection-fix

Dereference any array type before expanding root-level SSBO members

Merge pull request #1882 from Kangz/fix-chrome

GN build (for Chromium): enable HLSL in dependents.

GN build (for Chromium): enable HLSL in dependents.

The previous fix for this only enabled HLSL internally in glslang which
means that dependent using HLSL, for example shaderc, failed
compilation.

Dereference any array type before expanding root-level SSBO members

If we don't do this then we get reflection output like so:

ArrayedBind[0].a.a: offset 0, type 1406, size 1, index 4, binding -1, stages 0
ArrayedBind[0].a.b: offset 4, type 1406, size 1, index 4, binding -1, stages 0
ArrayedBind[0].b.a: offset 4, type 1406, size 1, index 4, binding -1, stages 0
ArrayedBind[0].b.b: offset 8, type 1406, size 1, index 4, binding -1, stages 0
ArrayedBind[0].b: offset 4, type 1406, size 1, index 4, binding -1, stages 1

When the outer reflection loop that calls blowUpActiveAggregate incorrectly iterates over the struct members.

Merge pull request #1878 from dneto0/chrome-fix

GN build (for Chromium): enable HLSL

Add support for GL_ARB_explicit_uniform_location

GN build (for Chromium): enable HLSL

GL_NV_integer_cooperative_matrix support

Bump revision.

Merge pull request #1874 from KhronosGroup/inherit-memory-qualifiers

GLSL: Inherit memory qualifiers, both declaratively and in execution.

GLSL: Inherit memory qualifiers, both declaratively and in execution.

Fixes #1870, probably others.

Merge pull request #1875 from jonahryandavis/extra-tokens

Fix conformance with -Wextra-tokens

Fix conformance with -Wextra-tokens

Bump version.

Merge pull request #1855 from KhronosGroup/web

Web:  Create a very small-footprint glslang for web use.

web: Fix accidental additon of refract() prototypes and update README.

Web: Turn off includes, independent preprocessing path, fine tune all.

Saved about 21K, size down to 380K of MSVC x86 code.
Fixed one bug that needs to be looked at on the master branch:
The test for needing a Vulkan binding has a bug in it, "!layoutAttachment"
which does not mean "no layoutAttachment", because that is non-zero.
This is why some test and test results changed.

Web: Make switched methods all be non-virtual, more web-dependent code,

added a few more HLSL flag tests.  This was mostly focused on the SPV generator.
Saves about 17K.

Web: Optional error management and error tightening.

Saves about 6.5K