Merge pull request #1500 from KhronosGroup/turing-known-good

Update known-good to tools/headers for the Nvidia Turing extensions.

Update known-good to tools/headers for the Nvidia Turing extensions.

Bump minor version.

Merge pull request #1499 from chaoc/nvidia_turing

Add support for multiple Nvidia extensions

update glslang test results for temporary validation fail

Add-support-for-SPV_NV_shading_rate

Add-support-for-SPV_NVX_raytracing

Add-support-for-SPV_NV_mesh_shader

0003-Add-support-for-SPV_NV_shader_image_footprint

Add-support-for-SPV_NV_compute_shader_derivatives

Add-support-for-SPV_NV_fragment_shader_barycentric

Update to latest spirv.hpp header (which fixed a typo).

SPV: Move to latest SPIR-V header (latest NV extensions support).

Build: Fix some warnings.

Bump revision.

Merge pull request #1492 from NorbertGarnysAMD/shader_atomic_int64

Add GL_EXT_shader_atomic_int64

Add GL_EXT_shader_atomic_int64

Merge pull request #1491 from DennisOSRM/patch-1

Fix compiler warning emitted from GCC8

Fix compiler warning emitted from GCC8

the following warning gets emitted:

```
In file included from ./Vulkan/glslang/SPIRV/hex_float.h:39,
                 from ./Vulkan/glslang/SPIRV/SpvBuilder.cpp:49:
./Vulkan/glslang/SPIRV/bitutils.h: In instantiation of ‘Dest spvutils::BitwiseCast(Src) [with Dest = spvutils::Float16; Src = short unsigned int]’:
./Vulkan/glslang/SPIRV/hex_float.h:138:47:   required from ‘T spvutils::FloatProxy<T>::getAsFloat() const [with T = spvutils::Float16]’
./Vulkan/glslang/SPIRV/hex_float.h:821:52:   required from here
./Vulkan/glslang/SPIRV/bitutils.h:29:14: warning: ‘void* memcpy(void*, const void*, size_t)’ writing to an object of non-trivially copyable type ‘class spvutils::Float16’; use copy-assignment or copy-initialization instead [-Wclass-memaccess]
   std::memcpy(&dest, &source, sizeof(dest));
   ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ./Vulkan/glslang/SPIRV/SpvBuilder.cpp:49:
./Vulkan/glslang/SPIRV/hex_float.h:43:7: note: ‘class spvutils::Float16’ declared here
 class Float16 {
       ^~~~~~~
In file included from ./Vulkan/glslang/SPIRV/hex_float.h:39,
                 from ./Vulkan/glslang/SPIRV/SpvBuilder.cpp:49:
./Vulkan/glslang/SPIRV/bitutils.h: In instantiation of ‘Dest spvutils::BitwiseCast(Src) [with Dest = spvutils::FloatProxy<spvutils::Float16>; Src = short unsigned int]’:
./Vulkan/glslang/SPIRV/hex_float.h:431:28:   required from ‘void spvutils::HexFloat<T, Traits>::setFromSignUnbiasedExponentAndNormalizedSignificand(bool, spvutils::HexFloat<T, Traits>::int_type, spvutils::HexFloat<T, Traits>::uint_type, bool) [with T = spvutils::FloatProxy<spvutils::Float16>; Traits = spvutils::HexFloatTraits<spvutils::FloatProxy<spvutils::Float16> >; spvutils::HexFloat<T, Traits>::int_type = short int; spvutils::HexFloat<T, Traits>::uint_type = short unsigned int]’
./Vulkan/glslang/SPIRV/hex_float.h:633:5:   required from ‘void spvutils::HexFloat<T, Traits>::castTo(other_T&, spvutils::round_direction) [with other_T = spvutils::HexFloat<spvutils::FloatProxy<spvutils::Float16>, spvutils::HexFloatTraits<spvutils::FloatProxy<spvutils::Float16> > >; T = spvutils::FloatProxy<float>; Traits = spvutils::HexFloatTraits<spvutils::FloatProxy<float> >]’
./Vulkan/glslang/SPIRV/hex_float.h:817:39:   required from here
./Vulkan/glslang/SPIRV/bitutils.h:29:14: warning: ‘void* memcpy(void*, const void*, size_t)’ copying an object of non-trivial type ‘class spvutils::FloatProxy<spvutils::Float16>’ from an array of ‘short unsigned int’ [-Wclass-memaccess]
   std::memcpy(&dest, &source, sizeof(dest));
   ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ./Vulkan/glslang/SPIRV/SpvBuilder.cpp:49:
./Vulkan/glslang/SPIRV/hex_float.h:115:7: note: ‘class spvutils::FloatProxy<spvutils::Float16>’ declared here
 class FloatProxy {
       ^~~~~~~~~~
```

Bump revision.

SPV: Fix #1487: Only declare AMD int16/half_float extensions when needed.

Printing preprocessed shaders (not a supported path): Fix #1490: strings

Put quote marks around strings, due to a change in how tokenization works.

Bump version and revision.

Merge branch 'jeffbolznv-memory_scope_semantics'

GL_KHR_memory_scope_semantics

Merge pull request #1465 from otakuto/remove-execute-permissions

Remove execute permissions

Merge pull request #1485 from karl-lunarg/karl-lunarg-patch-1

tooling: Fix update sources script for Python 3

Merge pull request #1486 from mrtrizer/patch-1

Fixed a typo leading to compilation error.

Update SpvTools.cpp

Fix build.

bump revision

SPV: Turn off the default validation done by the SPIRV-Tools optimizer.

Also use the glslang namespace qualifier consistently and validate after
legalization, not before. (But most tests don't legalize.)

Merge pull request #1482 from KhronosGroup/validate

SPV: Add SPIRV-Tools validator.

GLSL: Remove use of __ in the implementation of keywords having __.

GLSL: Remove use of __ in the implementation of keywords having __.

tooling: Fix update sources script for Python 3

This one small change allows the update_glslang_sources.py script to operate correctly with Python 2 and Python 3.

Change the string literal type to "bytes" so that it matches the type returned by the subprocess calls.  Otherwise, under Python 3, the search for "known-good" in the list of remotes always fails.  This is OK for the first execution of update_glsang_sources, since the remote is not there on the first run.  But on subsequent runs, the search still fails to match and the script stops when trying to create a remote that already exists.

SPV: Add option for controling when the SPIRV-Tools validator is used.

SPV: Isolate SPIRV-tools glue to its own file.

SPV: Add SPIRV-Tools validator. This needs the latest SPIR-Tools.

Non-functional: Rationalize some existing use of SPIRV-Tools.

Merge pull request #1480 from dneto0/stringify

Use our own SPIRV-Tools message stringifier

Use our own SPIRV-Tools message stringifier

Stop including an internal SPIRV-Tools header. It has been
deleted.

Fixes #1477

Merge pull request #1474 from KhronosGroup/pure-8-16-bit-capability

SPV: only declare the pure 8/16-bit capabilities when needed.

Merge pull request #1476 from greg-lunarg/kg24

Update SPIRV-Tools known good.

Update SPIRV-Tools known good.

Includes:

    Update OpPhi instructions after splitting block. (#1783)
    Don't change decorations and names in merge return. (#1777)
    Transform to combine consecutive access chains
    Handle undef literal value in vector shuffle
    Fix block ordering in dead branch elim
    Fix finding constant with particular type. (#1724)
    Fix infinite loop while folding OpVectorShuffle (#1722)
    Fix size error when folding vector shuffle. (#1721)
    Layout validation: Permit {vec3; float} tight packing

SPV: only declare the pure 8/16-bit capabilities when needed.

Only when operations stray outside the 8/16-bit storage-capabilities
are the general (pure 8/16-bit) capabilities needed.

SPV: More corrections of <id> versus "immediate" operands.

SPV: Correct SPIR-V operands for <id> versus immediate.

Bump version patch level.

Fix #1471: Merge shift amounts for different sets: --stb ... --std ...

Fix #1393: rationalize usage, in the spirit of #1393.

Order is kept about the same.

Bump revision.

Fix #1468: Add command-line --entry-point support, same as existing -e.

Fix #1469: Make it easier to copy/paste build instructions.

Merge pull request #1466 from Kangz/warning_fixes

Fix -Wignored-qualifier and -Wunused-variable warnings

Merge pull request #1470 from antiagainst/patch-2

Appveyor CI: build artifacts for x64

Appveyor CI: build artifacts for x64

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

Fix -Wignored-qualifier and -Wunused-variable warnings

Add shebang

Remove execute permissions

Merge pull request #1460 from KhronosGroup/postprocess

Non-Functional: Add postprocess as more robust way to add capabilities

Non-Functional: Add postprocess as more robust way to add capabilities

When capabilities are needed for specific SPIR-V instructions, it is
fragile to do so based on GLSL/AST usage; it should be based on actual
instructions they got translated to.

Merge pull request #1459 from dgkoch/dgkoch_build_fixes4

Fix various build warnings/errors

Fix build warnings/errors:

1) On some old versions of MSVC:

glslang\MachineIndependent\Constant.cpp(187): warning C4056: overflow in floating-point constant arithmetic

On this platform the definition of INFINITY is as follows:

#ifndef _HUGE_ENUF
   #define _HUGE_ENUF  1e+300  // _HUGE_ENUF*_HUGE_ENUF must overflow
#endif
#define INFINITY   ((float)(_HUGE_ENUF * _HUGE_ENUF))

Moving the negation outside the cast seems to resolve that issue.

2) Some Linux compilers were unhappy with lines 226/227

glslang/MachineIndependent/Constant.cpp: In member function 'virtual glslang::TIntermTyped* glslang::TIntermConstantUnion::fold(glslang::TOperator, const glslang::TIntermTyped*) const':
glslang/MachineIndependent/Constant.cpp:226:99: error: integer overflow in expression [-Werror=overflow]
 else if (rightUnionArray[i].getIConst() == -1 && leftUnionArray[i].getIConst() == -(int)0x80000000)
   ^~~~~~~~~~~~~~~~
glslang/MachineIndependent/Constant.cpp:227:48: error: integer overflow in expression [-Werror=overflow]
     newConstArray[i].setIConst(-(int)0x80000000);
                                                ^~~~~~~~~~~~~~~~

Moving the negation to the right side of the cast made those happy, but then some Windows compilers were unhappy:

glslang\MachineIndependent\Constant.cpp(226): warning C4146: unary minus operator applied to unsigned type, result still unsigned
glslang\MachineIndependent\Constant.cpp(227): warning C4146: unary minus operator applied to unsigned type, result still unsigned

which required adding on the "ll" suffix.

3) Android builds where unhappy with line 242:

glslang/MachineIndependent/Constant.cpp:242:100: error: comparison of integers of different signs: 'long long' and 'unsigned long long' [-Werror,-Wsign-compare]
                 else if (rightUnionArray[i].getI64Const() == -1 && leftUnionArray[i].getI64Const() == -0x8000000000000000ll)
                                                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Adding an explicit (long long) cast resolved this.
And the negation needs to be on the right side of the cast otherwise linux
builds are unhappy as in (2).

4) Android builds are unhappy with out of order initializers:

glslang/MachineIndependent/reflection.h:60:84: error: field 'type' will be initialized after field 'stages' [-Werror,-Wreorder]
glDefineType(pGLDefineType), size(pSize), index(pIndex), counterIndex(-1), type(pType.clone()), stages(EShLanguageMask(0)) { }
   ^
1 error generated.

Change-Id: Ic9a05fa7912498284885113d8b051f93f822f62b

Merge pull request #1457 from KhronosGroup/parse-floats

Tests: Add more string -> float tests. Related to #1456.

PP: Address #1456: Strip float suffixes before using platform library.

Tests: Add more string -> float tests. Related to #1456.

PP: floating-point parsing: more stable handling of istringstream::fail

Possibly addresses #1456

Bump revision.

Merge pull request #1455 from TiemoJung/nullpointer_crash_fix

Fixes a crash when in/out varying variable had no semantic name and a…

GLSL: No more restrictions on (non)shadow sampler construction.

 Match https://github.com/KhronosGroup/GLSL/pull/22

Fixes a crash when in/out varying variable had no semantic name and an error was reported in ioremapper phase when the variable was rejected

Merge pull request #1451 from KhronosGroup/shadow-forces-depth

GLSL/SPV: If a texture is used with a shadow sampler, force 'shadow'.

HLSL: Fix #1445: distance() works on scalars.

Merge pull request #1453 from greg-lunarg/kg22

Update spirv-tools known good

GLSL: Construct shadow texture from non-shadow sampler.

Tracks https://github.com/KhronosGroup/GLSL/pull/22.

GLSL/SPV: If a texture is used with a shadow sampler, force 'shadow'.

Fixes #854. But, only good if we are not trying to use the same
texture for both shadow and non-shadow constructors.

Force the type of the texture to have 'shadow' set when it is
constructed with a samplerShadow.

Update spirv-tools known good

Includes the following:

Add Vulkan 1.1 capability sets
Don't merge types of resources
Remove stores of undef.
Make sure the constant folder get the correct type.

Merge branch 'zeux-master'

Merge branch 'master' of https://github.com/zeux/glslang into zeux-master

Merge pull request #1450 from KhronosGroup/link-merge-all

Link: Merge all downstream consumed intermediate modes.

Link: Merge all the settings in TIntermediate.

Fixes #1309.

Merge pull request #1436 from karl-lunarg/karl-ccache

build: Add ccache option to CMake config

Merge pull request #1446 from KhronosGroup/spv-link

Link/SPV: Correct symbol IDs on merging ASTs to a single coherent space

Merge pull request #1437 from mattparks/patch-2

CMake options for postfix and SPVRemapper.

Merge branch 'jeffbolznv-missing_descriptor_indexing_extension'

Merge branch 'missing_descriptor_indexing_extension' of https://github.com/jeffbolznv/glslang into jeffbolznv-missing_descriptor_indexing_extension

Link/SPV: Correct symbol IDs on merging ASTs to a single coherent space

This is one step in providing full linker functionality for creating
correct SPIR-V from multiple compilation units for the same stage.
(This was the only remaining "hard" part. The rest should be simple.)

Merge pull request #1442 from dneto0/use-forked-android-ndk-repo

NDK build: Use NDK r17b and its own CMake toolchain file

Add missing OpExtension SPV_EXT_descriptor_indexing for dynamic indexing capabilities

Errors and Build: Fix build warnings, which also improved error messages.

NDK build: Use NDK r17b and its own CMake toolchain file

More recent NDK releases have their own CMake toolchain file.
Use it.

Also, download the NDK from github.com:dneto0/android-ndk.
That is a fork of the repo we used to use, but we have more
control over how long it stays stable.

Travis CI: Fix comments disabling code.

Merge pull request #1440 from dneto0/later-android

WIP: Travis-CI: Use Android NDK r13b specifically

Travis-CI: Use Android NDK r13b specifically

The Travis-CI bot downloads a copy of the Android NDK.  The source
we get it from recently updated to Android NDK r17b.  However,
the android.toolchain.cmake file does not know how to parse the
Android native API level from that version of the NDK.  So check
out the NDK r13b version that we were using until yesterday.

Fixes #1439

Merge pull request #1438 from Think-Silicon/getUniformStages

Reflection exposes the Shader Stages where a Uniform is present

Non-functional: Retrigger bots; the previous failure looks suspicious.

HLSL: Fix #1432: Globally initialize local static variables.

Reflection exposes the Shader Stages where a Uniform is present

Allowed debug postfix to be changed.

Added the cmake option to disable building SPVRemapper.