Merge branch 'glsl-ext' of https://github.com/nicebyte/glslang into nicebyte-glsl-ext

Bump revision.

Merge pull request #1345 from amdrexu/bugfix

Add AMD float16/int16 SPV extension string when GLSL specifies them

Specify SPV_AMD_gpu_shader_int16 extension string when frexp() takes 16-bit
exp operand

Bump revision.

HLSL/SPV: Declare DepthReplacing for shaders declaring SV_Depth* outputs.

Merge pull request #1348 from greg-lunarg/kg16

Update spirv-opt known good

Merge pull request #1351 from danginsburg/issue_1350

Fix issue #1350 - set the node type to match the atomic TIntermAggreg…

Update gold results

Make FindLanguage use the entire name if parseStageName is false

Fix build error

Make glslang validator support files ending in .glsl

This patch makes  the validator accept *.<stage name>.[g/h]lsl pattern
for file names.

This patch preserves previous behavior (i.e. *.vert/*.frag/etc. in file
names still work).

Fix issue #1350 - set the node type to match the atomic TIntermAggregate node type so that AtomicSMin/Max is not generated with a result type of void.

GLSL/Vulkan: Enforce no location aliasing in vertex inputs.

Update spirv-opt known good

Changes include:
    Fix SSA rewrite for nested loops.
    Add support for two new extensions:
    - SPV_NV_shader_subgroup_partitioned
    - SPV_EXT_descriptor_indexing
    Legalize OpImageTexelPointer
    Copy propagate arrays

Build: Fix #1347: Compile with NV_EXTENSIONS disabled.

Merge pull request #1326 from mattparks/patch1

Allowed for LIB_TYPE to be set to switch between static and shared libs for glslang, SPIRV, and HLSL

Non-functional: Fix tabs.

GLSL: Fix #1330: default outputs for GL_NV_geometry_shader_passthrough

Bump revision.

Merge pull request #1343 from KhronosGroup/uint-counter-bufs

HLSL: Fix #1332: consistently use uint for counter buf typing.

Merge pull request #1341 from St0fF-NPL-ToM/master

fix VS2012 compile issues

HLSL: Fix #1332: consistently use uint for counter buf typing.

Revert "add ability to record accessed and declared "named defines""

This reverts commit e7350423067442f616431c1cdf4f18dd7990a491.

Merge pull request #1338 from cgmb/minor-shaderlang-fixups

Minor cleanup in ShaderLang.cpp

GLSL/SPV: no location on atomic_uint; Fix #1339.

Minor cleanup in ShaderLang.cpp

Use unique_ptr to simplify memory management in ProcessDeferred.

fix further MSVC 2012 build issues

MSVC 2012 does not allow initializer lists - replaced with corrresponding CTor calls.

add ability to record accessed and declared "named defines"

Added the needed 2 sets to TIntermediate, added accessor-functions, inserter functions.  Implemented recording of such named defines inside the preprocessor parser.

fix VS2012 compile issues

small fixes needed to compile under Win8.1/VS2012/static_lib

GLSL/SPV: Fix #1310: don't create unnecessary integer matrices.

README: Expand on the C++11 compiler requirement for MSVS.

Update revision.

Fix #1328: ENABLE_HLSL=OFF breaking build.

Fix #1331: Emit SPV_GOOGLE_hlsl_functionality1 for counters.

It was missing when there were no semantics.

Fix #1333: Protect against -g for non-generating code.

Merge branch 'nonuniform-dynindex'

Update revision.

Revision/Version update.

GLSL/SPV: Implement SPV_EXT_descriptor_indexing and GL_EXT_nonuniform_qualifier

Update revision.

GLSL: Implement correct semantic checking for run-time sized arrays.

Linker: Fix #1329: correctly merge unsized arrays, and fix link tests.

Merge pull request #1327 from mattparks/patch2

Added getter for uniform block bindings

Added getter for uniform block bindings

Allowed for LIB_TYPE to be set to switch between static and shared libs for glslang, SPIRV, and HLSL

Merge pull request #1325 from KhronosGroup/simplify-doc

Non-Functional: Remove unused parts of the disassembler.

Non-Functional: Remove unused parts of the disassembler.

Merge pull request #1322 from jeffbolznv/master

Implement GL_NV_shader_subgroup_partitioned

Implement GL_NV_shader_subgroup_partitioned

Merge pull request #1320 from greg-lunarg/addopt21

Only print legalization warning if optimizer not enabled

Only print legalization warning if optimizer disabled

Merge pull request #1319 from KhronosGroup/array-rationalization

Array rationalization

Merge pull request #1316 from greg-lunarg/addopt18

Disable DCE by Remapper through glslangValidator...

Merge pull request #1318 from amdrexu/feature

Add int16 support to AMD_trinary_minmax and AMD_shader_ballot

Non-functional. Rationalizations enabling future generalizations:

- Use much simpler method to update implicit array sizes.
  The previous overly complicated method was error prone.
- Rationalize all use of unsized arrays.
- Combine decorations when generating SPIR-V, to simplify
  adding extensions.

Non-Functional: Remove merge-replicated code and fix white space.

Add int16 support to AMD_trinary_minmax and AMD_shader_ballot

Disable DCE by Remapper through glslangValidator...

...now that spirv-opt is capable enough.

Merge pull request #1313 from greg-lunarg/kg15

Update SPIRV-Tools known good.

Merge pull request #1312 from KhronosGroup/fix-mixed-array-styles

Fix mixed array styles

Merge branch 'master' into kg15

Update SPIRV-Tools known good.

Also update spirv-opt recipes.

Types: Fix #1290: Rationalize and correct "mixed" style array dimensioning.

There a couple functional problems, which when reduced down also led to
some good simplifications and rationalization.  So, this commit:
 - corrects "mixed" functionality: int[A] f[B] -> f[B][A]
 - correct multi-identifier decls: int[A] f[B], g[C] -> f and g are independently sized.
 - increases symmetry between different places in the code that do this
 - makes fewer ways to do the same thing; several methods are just gone now
 - makes more clear when something is copied or shared

Test Results Only: Include size in implicitly sized arrays (change to unsized).

This effects 60 tests, in a trivial way, but is critical to factor out this
noise from future planned changes.

Non functional: add EshTargetClientVersion alias for EShTargetClientVersion.

Fixes #1304.

Non-functional: always return value from makeFpConstant().

Should fix #1308.

Merge pull request #1306 from Igalia/uniform-aml-v2

Improve --auto-map-locations for uniforms (v2)

Merge pull request #1305 from amdrexu/bugfix

Add additional error check for fragment shader outputs

Take into account the number of locations taken up by a uniform

When assigning uniform locations it now takes into account the number
of locations occupied by the type. For uniforms, all types except
arrays and structs take up one location. For arrays the base location
count is multiplied by the array dimensions and for structs it is the
sum of the locations of each member.

Add locations to opaque types for OpenGL

Add additional error check for fragment shader outputs

Int64 and uint64 are disallowed as fragment shader outputs, similar to
double.

Bump patch version.

Merge pull request #1302 from KhronosGroup/cpp-relax-skiptokens-errors

PP: don't give errors on some tokens under #if 0 (or similar).

PP: don't give errors on some tokens under #if 0 (or similar).

Fixes #1295.

Tokens that are accepted by any version of HLSL or GLSL should be
allowed when #ifdef'd off, such that errors are not reported.

Merge pull request #1282 from KhronosGroup/hlslFunctionality1

WIP: Implement SPV_GOOGLE_hlsl_functionality1.

Implement SPV_GOOGLE_hlsl_functionality1.

Enabled via -fhlsl_functionality1

Merge pull request #1291 from ben-clayton/add-nv-nopersp

Add support for GL_NV_shader_noperspective_interpolation

Bump patch version.

Merge pull request #1299 from KhronosGroup/fix-conversions

Fix conversions

Add tests for GL_NV_shader_noperspective_interpolation

Merge pull request #1289 from Igalia/igalia/dmat-scalar-division

Use the correct type for the constant for matrix/scalar division

AST: Fix shift conversions, which don't require matching types.

The base and shift amount need to be integers, but not of the same type.
This fixes #1296 and replaces #1297.

Non-functional: Rationalize operators handled by the split addConversion() functions.

Historically, addConversion() was split to handle binary node <-> node conversions
from non-binary node -> type conversions. However, the split wasn't entirely clean
WRT HLSL design and left duplication of case statements, which are misleading, and
this commit cleans up.

AST: Remove convertToBasicType() which was obfuscating the conversion design.

Reinforces that conversion rules are operation-specific.
Side effect is that HLSL logical-operator conversions are more direct
(e.g. float -> bool, rather than float -> int -> bool).

Merge pull request #1294 from dgkoch/dgkoch_build_fixes4

Fix build several build errors

Fix build several build errors

glslang/SPIRV/SpvBuilder.cpp:2533:27: error: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long') [-Werror,-Wsign-compare]
        for (int c = 0; c < accessChain.swizzle.size(); ++c)
                        ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

glslang/hlsl/hlslParseHelper.cpp:69:5: error: field 'cullDistanceInput' will be initialized after field 'clipDistanceOutput' [-Werror,-Wreorder]
    cullDistanceInput(nullptr),
    ^
1 error generated.

glslang/glslang/MachineIndependent/attribute.cpp:85:16: error: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long') [-Werror,-Wsign-compare]
    if (argNum >= args->getSequence().size())
        ~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Merge pull request #1293 from baldurk/remove-en-dash

Replace en-dash with simple ASCII hyphen

Replace en-dash with simple ASCII hyphen

* This means the source file is ASCII instead of (assumed) UTF-8 and
  doesn't cause warnings when compiling on non-western codepages.

Add support for GL_NV_shader_noperspective_interpolation

Use the correct type for the constant for matrix/scalar division

When a matrix is divided by a scalar it tries to take the reciprocal
of the scalar to convert the operation into a multiply. However it was
always doing this by making a 32-bit constant. If the scalar is a
double then this would end up making an FDiv instruction with
different types in the operands.

This patch adds a helper method called makeFpConstant which makes a
floating-point constant of the given type. The code to take the
reciprocal now uses it to make the same type as the result.

Fixes https://github.com/KhronosGroup/glslang/issues/1278

Merge pull request #1286 from greg-lunarg/kg14

Update known_good to SPIR-V 1.3 support

Merge pull request #1288 from amdrexu/bugfix

HLSL: Correct some mistakes for min16 types

Merge pull request #1287 from eintw1ck/master

include/Common.h: reorder includes to avoid redefinition on some platforms

HLSL: Correct some mistakes for min16 types

- Add missing constructor ops to support float16/int16/uint16 types
- Allow half float literals
- Correct two errors of double literal parse in HLSL: extension check and
  postfix

include/Common.h: reorder includes to avoid redefinition on some platforms

On some systems sstream defines snprintf, this leads to the a redefinition of sprintf_s if snprintf is defined before checking if it exists in sstream.

Increment minor version to 4

Update known_good to SPIR-V 1.3 support

Merge pull request #1283 from siavashserver/fix-memory-leak

Fix memory leak upon ProcessDeferred failure.

Merge pull request #1284 from amdrexu/bugfix

Allow --hlsl-enable-16bit-types to be used without AMD_EXTENSIONS