Merge pull request #748 from dgkoch/dgkoch_build_fixes2

More build fixes

More build fixes

Fix another build warning on some platforms

Use an explicit cast from size_t to int to avoid warning.

HLSL: Fix #747: accept 'struct' in front of previously user-defined type name.

SPV: Fix #739: OpSelect can only operate on scalars and vectors.

Merge pull request #744 from realitix/sanitize_sh_link

Sanitize ShaderLang.h

Merge pull request #746 from dgkoch/dgkoch_build_fixes

Fix build warnings on some platforms

Build: Another fix for issue #718: implement the hidden methods of tFinalize.

Fix build warnings on some platforms

Use an explicit cast from size_t to int to avoid errors like the following:
glslang\glslang\MachineIndependent\preprocessor\Pp.cpp(1053) : error C2220: warning treated as error - no 'object' file generated
glslang\glslang\MachineIndependent\preprocessor\Pp.cpp(1053) : warning C4267: '=' : conversion from 'size_t' to 'int', possible loss of data

affects Pp.cpp, hlslParseHelper.cpp.

Initialize local variable to get rid of warningsa about potentially
uninitialized variables:
glslang\hlsl\hlslparsehelper.cpp(3667) : error C2220: warning treated as error - no 'object' file generated
glslang\hlsl\hlslparsehelper.cpp(3667) : warning C4701: potentially uninitialized local variable 'builtIn' used

affects hlslParseHelper.cpp

Sanitize ShaderLang.h

ShaderLang.h contains declaration for `ShLink` function which is never
defined.

See this grep output:

$grep -Hrn 'ShLink'
glslang/Public/ShaderLang.h:206:SH_IMPORT_EXPORT int ShLink(
glslang/Public/ShaderLang.h:214:SH_IMPORT_EXPORT int ShLinkExt(
glslang/MachineIndependent/ShaderLang.cpp:1268:int ShLinkExt(

PP: Fix issue #738: don't assert on characters within a string.

PP: Partially address issue #738: handle premature end of argument when token pasting.

Merge pull request #742 from steve-lunarg/f16tof32-typefix

HLSL: small fix for index type in f16tof32 opcode

HLSL: small fix for index type in f16tof32 opcode

The f16tof32 opcode was indexing a vector with a float 0, rather
than an int 0.  It may have made no functional difference due to the
identical bit pattern, but code looking at the type could be
confused.

GLSL: Fix #741: rationalize per-block vs. per-member offset checking.

Merge pull request #736 from steve-lunarg/structbuffer-params

HLSL: add structuredbuffer pass by reference in fn params

PP: Address #737: accept 'h'/'H' floating-point suffix more broadly.

Merge pull request #740 from steve-lunarg/f16tof32

HLSL: add f16tof32 and f32tof16 decompositions.

WIP: HLSL: add f16tof32 and f32tof16 decompositions.

SPV: Update SPV header to official Rev. 10 of 1.0, and fix the consequences of doing so.

SPV: Implement new extensions GL_KHX_device_group and GL_KHX_multiview.

These correspond to SPV_KHR_device_group and SPV_KHR_multiview.
Also, bring tests up to date with Khronos internals, and some misc. related changes.

WIP: HLSL: add structuredbuffer pass by reference in fn params

This PR adds the ability to pass structuredbuffer types by reference
as function parameters.

It also changes the representation of structuredbuffers from anonymous
blocks with named members, to named blocks with pseudonymous members.
That should not be an externally visible change.

SPV: Handle nested opaque types as function parameters.

Merge pull request #734 from steve-lunarg/ssbo-reflection

Add reflection queries for thread local size.  Add ssbo auto-binding.

Merge pull request #733 from amdrexu/bugfix

SPV: Unexpected declarations of capabilities from NV extensions.

Add reflection queries for thread local size and ssbo auto-binding

New command line option --shift-ssbo-binding mirrors --shift-ubo-binding, etc.

New reflection query getLocalSize(int dim) queries local size, e.g, CS threads.

SPV: Unexpected declarations of capabilities from NV extensions.

Merge pull request #725 from steve-lunarg/structurebuffer

HLSL: implement 4 (of 6) structuredbuffer types

HLSL: implement 4 (of 6) structuredbuffer types

This is a partial implemention of structurebuffers supporting:

* structured buffer types of:
*   StructuredBuffer
*   RWStructuredBuffer
*   ByteAddressBuffer
*   RWByteAddressBuffer

* Atomic operations on RWByteAddressBuffer

* Load/Load[234], Store/Store[234], GetDimensions methods (where allowed by type)

* globallycoherent flag

But NOT yet supporting:

* AppendStructuredBuffer / ConsumeStructuredBuffer types
* IncrementCounter/DecrementCounter methods

Please note: the stride returned by GetDimensions is as calculated by glslang for std430,
and may not match other environments in all cases.

Merge pull request #732 from dneto0/fix-overrides

Add override where needed by clang++

Add override where needed by clang++

Otherwise we get an error when we have both
-Werror and -Winconsistent-missing-override.
This is required for building with NDK r14 betas

Fix issue #676: emit error message on failure to open spv file.

SPV: Fix #723: construct vectors from matrices.

Merge pull request #727 from jekstrand/image-load-formatted

Implement GL_EXT_shader_image_load_formatted

Merge pull request #729 from gwihlidal/master

HLSL - Support for SV_GroupIndex

Implement GL_EXT_shader_image_load_formatted

Warning fixes

HLSL - Support for SV_GroupIndex

Merge pull request #1 from KhronosGroup/master

Update from master

Merge pull request #720 from flaviobortot/master

Added --vn option to generate a C header file containing a variable a…

Merge pull request #728 from MaciejJesionowskiAMD/fix-mismatched-subgroup-vote-string

Fix mismatched doc string for SPV_KHR_subgroup_vote

Fix mismatched doc string for SPV_KHR_subgroup_vote

Text for opcodes OpSubgroupAllKHR and OpSubgroupAnyKHR was swapped.

Merge pull request #724 from chaoc/master

Implement NVX_multiview_per_view_attributes

Removed tabs and replaced with spaces. Changed layout for "else if"

Implement NVX_multiview_per_view_attributes

Support align/offset for all versions GLSL/ESSL targeting SPIR-V.

Merge pull request #722 from steve-lunarg/tessellation

Add basic HS/DS implementation.

PP, nonfunctional: Properly encapsulate a TokenStream.

Add basic HS/DS implementation.

This obsoletes WIP PR #704, which was built on the pre entry point wrapping master.  New version
here uses entry point wrapping.

This is a limited implementation of tessellation shaders.  In particular, the following are not functional,
and will be added as separate stages to reduce the size of each PR.

* patchconstantfunctions accepting per-control-point input values, such as
  const OutputPatch <hs_out_t, 3> cpv are not implemented.

* patchconstantfunctions whose signature requires an aggregate input type such as
  a structure containing builtin variables.  Code to synthesize such calls is not
  yet present.

These restrictions will be relaxed as soon as possible.  Simple cases can compile now: see for example
Test/hulsl.hull.1.tesc - e.g, writing to inner and outer tessellation factors.

PCF invocation is synthesized as an entry point epilogue protected behind a barrier and a test on
invocation ID == 0.  If there is an existing invocation ID variable it will be used, otherwise one is
added to the linkage.  The PCF and the shader EP interfaces are unioned and builtins appearing in
the PCF but not the EP are also added to the linkage and synthesized as shader inputs.
Parameter matching to (eventually arbitrary) PCF signatures is by builtin variable type.  Any user
variables in the PCF signature will result in an error.  Overloaded PCF functions will also result in
an error.

[domain()], [partitioning()], [outputtopology()], [outputcontrolpoints()], and [patchconstantfunction()]
attributes to the shader entry point are in place, with the exception of the Pow2 partitioning mode.

PP, nonfunctional: Remove crufty bit-twiddling of tokens.

Fix issue #708: token pasting within macro argument expansion.

Merge pull request #707 from KhronosGroup/emit-opSelect

SPV: Emit op select

Address issue #718. Should change which warning is generated, hopefully to a better one.

SPV: Implement specialization constants for ?:.

SPV: Emit OpSelect when a selection node is simple enough.

Also, ensures it has a type, no disallowed side effects,
or performance trade offs.

Merge pull request #685 from KhronosGroup/wrap-entry-point

HLSL: Wrap entry-point; needed to write 'in' args, and support 'inout' args

HLSL: don't do a deepCopy() for typedef, as we still want to share the type graph.

This enables the IO type mapping to work transparently for typedefs.

HLSL: Add tests and refine what decorations are passed through per stage/in/out.

HLSL: Move to fine-grained control for defining input/output/uniform IO types.

HLSL: Build IO types bottom up, as parsed, and cache the original (IO).

Previously, this was done recursively, per object, and the nonIO version
was cached. This reverses both those approaches.

HLSL: Have loose uniforms also go through the makeTypeNonIo() path.

HLSL: remove pervertex output blocks

This removes pervertex output blocks, in favor of using only
loose variables.  The pervertex blocks are not required and were
only partly implemented, and were adding some complication.

This change goes with wrap-entry-point.

Build: Fix Linux for older compiler: Use TMap instead of TUndorderedMap.

HLSL: fix copies between arrays of structs of builtins, and arrayed builtins.

Structs are split to remove builtin members to create valid SPIR-V.  In this
process, an outer structure array dimension may be propegated onto the
now-removed builtin variables.  For example, a mystruct[3].position ->
position[3].  The copy between the split and unsplit forms would handle
this in some cases, but not if the array dimension was at different levels
of aggregate.

It now does this, but may not handle arbitrary composite types.  Unclear if
that has any semantic meaning for builtins though.

HLSL: Make the entry-point shadow function have non-IO params and return.

This also removes an no longer needed makeTemporary() and rationalizes
makeTypeNonIo()'s interface.

HLSL: Type sanitization: create non-IO types for var decl and fn param/ret

This introduces parallel types for IO-type containing aggregates used as
non-entry point function parameters or return types, or declared as variables.
Further uses of the same original type will share the same sanitized deep
structure.

This is intended to be used with the wrap-entry-point branch.

AST: Have type deepCopy() preserve type graphs as graphs.

Previously, a type graph would turn into a type tree. That is,
a deep node that is shared would have multiple copies made.

This is important when creating IO and non-IO versions of deep types.

HLSL: Wrap the entry-point; need to write 'in' args, and support 'inout' args.

This needs some render testing, but is destined to be part of master.

This also leads to a variety of other simplifications.
 - IO are global symbols, so only need one list of linkage nodes (deferred)
 - no longer need parse-context-wide 'inEntryPoint' state, entry-point is localized
 - several parts of splitting/flattening are now localized

Added --vn option to generate a C header file containing a variable assigned to the hex representation of the shader. This is a standard feature on Microsoft's HLSL compiler and it allows developers to include pre-compiled shaders directly into the code. This option enables "Hex output", so it is NOT required to specify -x as well. The output file name is preserved, so no ".h" extension is added. If you want the output file to have ".h" extension then you have to specify it on the output file name. The generated header file uses the "#pragma once" pragma to avoid multiple inclusions.

Resolve issue #700: allow initializers on struct members.

Memory/constructor/warning clean-up. Addresses issue #705.

Merge pull request #711 from eoma/fix-friendly-specialization-name

SPV: Emit names of specialization constants, fixes #710

SPV: Emit names of specialization constants

Merge pull request #706 from Ralith/install-headers

Install headers

Install headers

This enables the vulkan loader to be built against an installed glslang.

Fix bug from PR #678: NV_EXTENSIONS deleted implicit array sizing.

Fix issue #693. Ternary operator on void type.

Merge pull request #690 from ligfx/proper_dependencies

CMake: add target_link_libraries internally

README: point to HLSL status issues

Merge pull request #702 from resetnow/master

SpvBuilder: add const specifier to vector reference arguments

SpvBuilder: add const specifier to vector reference arguments

Merge pull request #694 from mtavenrath/fix_origin_link

Use correct OriginUpperLeft when linking more than one shader.

Use correct OriginUpperLeft when linking more than one shader.

Merge pull request #696 from amdrexu/bugfix2

Semantics: Texture clamp functions are available for fragment shader only

Merge pull request #695 from amdrexu/bugfix

Fix a CMakeLists issue

Semantics: Texture clamp functions are available for fragment shader only.

Fix a CMakeLists issue

Merge pull request #692 from hrydgard/no-nvextensions-build-fix

Fix build if NV_EXTENSIONS is not set

Fix build if NV_EXTENSIONS is not set

Merge pull request #678 from chaoc/stereo_view_rendering

support SPV_NV_viewport_array2 and SPV_NV_stereo_view_rendering

CMake: add target_link_libraries internally

Makes it easier to include glslang in a larger CMake project---instead
of having to call `target_link_libraries(glslang OSDependent OGLCompiler
HLSL)`, for example, you only need to call
`target_link_libraries(glslang)` and it will pull in the helpers it
needs.

This is also better in terms of cleaning up the "public interface",
of sorts, for building glslang: end-users probably shouldn't need to
know or be explicitly dependent on internal targets.

support SPV_NV_viewport_array2 and SPV_NV_stereo_view_rendering

Merge pull request #689 from mikew-lunarg/master

Fix for not-handled-in-switch warnings

Fix for not-handled-in-switch warnings

Added default to the switch() for AMD_EXTENSIONS to avoid a spew
of warning messages.

Merge pull request #683 from rdb/master

Fix compilation with MSVC 2010

Fix compilation with MSVC 2010

Merge pull request #682 from steve-lunarg/split-copy-fix

HLSL: fix dereferencing when copying split structures with arrays

Merge pull request #679 from ashwinkolhe/akolhe_spv_khr_subgroup_vote

Implement SPV_KHR_subgroup_vote

SPV: Merge pull request #675 from amdrexu/feature

SPV: Implement new revision of extension GL_AMD_shader_ballot

Infrastructure: remove potential memory leaks.

HLSL: fix dereferencing when copying split structures with arrays

When copying split types with mixtures of user variables and buitins,
where the builtins are extracted, there is a parallel structures traversal.
The traversal was not obtaining the derefenced types in the array case.