Build: Fix linux

I believe this is a repeat of 12d6936.

HLSL: Fix #919: for-init-statement is arbitrary declaration or expression.

Unlike "if (XXX)" and "while (XXX)", with "for (YYY...", the YYY can be
more kinds of statements than the XXX.

GLSL: replace general missing functionality with specific messages.

Merge pull request #912 from amdrexu/feature

Implement the extension GL_AMD_texture_gather_bias_lod

Implement the extension GL_AMD_texture_gather_bias_lod

GLSL: Fix bug setting component=0 for an auto-location assignment.

Setting component=0 is later taken to mean the shader did so, which
is not always legal. It should instead set the component as not set
by the shader.

GLSL: Error when using location on an arrayed block.

This implements a recent change to the GLSL specification to enforce
this ill-defined situation.

Merge pull request #899 from antiagainst/rm-empty-cpps

Remove empty cpp files

Merge pull request #915 from LoopDawg/subvec4-intrinsic

HLSL: add test coverage for sub-vec4 texture intrinsics

Merge pull request #905 from KhronosGroup/flatten-opaque-structs

SPV: When passing structs of opaque types, flatten and pass the membe…

HLSL: Convert run-time sampler assignments to compile-time aliases.

For "s.m = t", a sampler member assigned a sampler, make t an alias
for s.m, and when s.m is flattened, it will flatten to the alias t.
Normally, assignments to samplers are disallowed.

SPV: When passing structs of opaque types, flatten and pass the members instead.

This avoids either A) needing uniformConstant struct, or
B) initializing a struct with opaque members, as writing them is not
allowed.

HLSL: iomapper: Fix #914. Tolerate user aliasing of bindings.

Because it is valid in HLSL to alias bindings:
A) remove validation that aliasing is not done
B) make the algorithms tolerate aliasing

HLSL: add test coverage for sub-vec4 texture intrinsics

This changes no functional code.  There was a bit of a testing hole
in that textures templatized on sub-vec4 types were not being exercised
with any intrinsics.  This adds some basic sanity coverage of that case.

Merge pull request #907 from KhronosGroup/include

HLSL: Add an Includer to handle #include for local HLSL paths.

Merge pull request #913 from amdrexu/bugfix

Parser: Add missing codes for float16

HLSL: Add an Includer to handle #include.

Parser: Add missing codes for float16

Merge pull request #911 from KhronosGroup/debug-info

SPV: Add OpSource shader source code and file name.

SPV: Debug output: Include OpLine information for execution path.

Note that declaratives are not handled, only procedurals.

SPV: Add OpSource shader source code and file name.

Merge pull request #856 from TiemoJung/texture_upgrade

Pure Texture to Sampled Texture Transform

Pure Texture to Sampled Texture Transform

Adds a transformation step to the post processing step.
Two modes are available:
1) keep
- Keeps samplers, textures and sampled textures as is
2) transform pure texture into sampled texture and remove pure samplers
- removes all pure samplers
- transforms all pure textures into its sampled counter part

Change-Id: If54972e8052961db66c23f4b7e719d363cf6edbd

Merge pull request #901 from LoopDawg/imat-construct

HLSL: Add imat, umat, and bmat constructors

SPV: Fix #904: Correctly check for built-in block redeclations for location check.

HLSL: Fix #903: Don't short-circuit && or ||.

HLSL: Fix #902: Incorrect protection against zero arguments.

HLSL: Add imat, umat, and bmat constructors

Fixes #894

Fix #373: Implicitly make gl_FragColor a location=0 output.

SPV: Correctly enforce 'location' presence on in/out blocks.

Blocks have this on members, not the object.

Merge pull request #900 from LoopDawg/tx-overloads

HLSL: allow name mangling based on texture template type

More non-determinism fixed.

Fix C++ portability, non-deterministic order of argument evaluation.

Fix #857: Convert uniform int to local bool for struct alias assignment.

This was done for one direction, but not both directions, so this commit
picks up the other direction.

HLSL: Don't do logical short-circuits when the operands are bool-vectors.

This seems a bit ill-defined, and was generating code that made OpPhi of two
operands that were Boolean vectors result in a scalar bool.

HLSL: allow name mangling based on texture template type

Name mangling did not account for the vector size in the template type of a texture.
This adds that.  The mangle is as it ever was for the vec4 case, which leaves
all GLSL behavior and most HLSL behavior uneffected.  For vec1-3 the size is added
to the mangle.

Current limitation: textures cannot presently be templatized on structured types,
so this works only for vectors of basic types.

Fixes #895.

Remove empty cpp files

This solves ranlib warnings on MacOS.

Merge pull request #896 from KhronosGroup/spv-location

SPV: Give error on not assigning locations to I/O when generating SPIR-V.

Merge pull request #897 from LoopDawg/remap-specconstop-fix.2

Remapper: handle embedded opcode in OpSpecConstantOp

Remapper: handle embedded opcode in OpSpecConstantOp

OpSpecConstantOp contains an embedded opcode which is given as a literal
argument to the OpSpecConstantOp.  The subsequent arguments are as the
embedded op would expect, which may be a mixture of IDs and literals.  This
adds support for that to the remapper binary parser.  Upon seeing such an
embedded op, the parser flips over to parsing the argument list as
appropriate for that opcode.

Fixes #882.

SPV: Give error on not assigning locations to I/O.

Also, provides an option to auto-assign locations.
Existing tests use this option, to avoid the error message,
however, it is not fully implemented yet.

Merge pull request #860 from steve-lunarg/sb-counter-args.2

HLSL: add ability to pass struct buffers with counters to fns

Merge pull request #893 from KhronosGroup/member-call

HLSL: Implement member functions calling member functions.

Fixes #883.

WIP: HLSL: add ability to pass struct buffers with counters to fns

This modifies function parameter passing to pass the counter
buffer associated with a struct buffer to a function as a
hidden parameter.  Similarly function declarations will have
hidden parameters added to accept the associated counter buffers.

There is a limitation: if a SB type may or may not have an associated
counter, passing it as a function parameter will assume that it does, and
the counter will appear in the linkage whether or not there is a counter
method used on the object.

HLSL: Implement member functions calling member functions.

Merge pull request #852 from steve-lunarg/declared-builtin

track declared builtin type

WIP: track declared builtin type [proposal]

Marking as WIP since it might deserve discussion or at least explicit consideration.

During type sanitization, the TQualifier's TBuiltInVariable type is lost.  However,
sometimes it's needed downstream.  There were already two methods in use to track
it through to places it was needed: one in the TParameter, and one in a map in the
HlslParseContext used for structured buffers.

The latter was going to be insufficient when SB types with counters are passed to
user functions, and it's proving awkward to track the data to where it's needed.
This PR holds a proposal: track the original declared builtin type in the TType,
so it's trivially available where needed.

This lets the other two mechanisms be removed (and they are in this PR).  There's a
side benefit of not losing certain types of information before the reflection interface.

This PR is only that proposal, so it changes no test results.  If it's acceptable,
I'll use it for the last piece of SB counter functionality.

Merge pull request #890 from LoopDawg/mip-operator

HLSL: add .mips[][] operator for texture types

HLSL: add .mips[][] operator for texture types

This implements mytex.mips[mip][coord] for texture types.  There is
some error testing, but not comprehensive.  The constructs can be
nested, e.g in this case the inner .mips is parsed before the completion
of the outer [][] operator.

   tx.mips[tx.mips[a][b].x][c]

Merge pull request #889 from antiagainst/travis-android

Check Android build on Travis CI

Check Android build on Travis CI.

Also stop requiring sudo privilege so that we can use containers
for testing.

Merge pull request #888 from LoopDawg/geometry-stage-mix

HLSL: allow GS-specific methods in non-GS stages

HLSL: allow GS-specific methods in other stages

Using GS methods such as Append() in non-GS stages should be ignored, but was
creating errors due to the lack of a stream output symbol for the non-GS stage.

HLSL: Fix #884: Use promoted children, not pre-promoted, in completing binary nodes.

Merge pull request #886 from antiagainst/unordered-set

Use unordered_set in ReadableOrderTraverser.

Use unordered_set in ReadableOrderTraverser.

Previously we are using unordered_map to map to bool, which can
be simplified with unordered_set.

Merge pull request #880 from aras-p/master

Fix for lumped/unity/batch builds

Merge pull request #874 from xlpiao/DescriptorSet-and-Binding

HLSL: Manually configure descriptor set and binding number for resources

[lumped builds] Add include guards (#pragma once) to header files that did not have any.

The codebase seems to use both “#pragma once“ approach, and “#ifndef / #define” approach,
so I picked pragma once as that one is less typing & less brittle.

When glslang is built with some other build system and lumped/unity builds are used,
without the include guards some headers would get included multiple times, leading to duplicate
declaration errors.

[lumped builds] Only define _CRT_SECURE_NO_WARNINGS if it’s not defined yet.
When glslang is built with some other build system and lumped/unity builds are used,
without the checks this would get “macro is being redefined” warnings/errors.

HLSL: Manually configure descriptor set and binding number for resources

Build fix: Make string assignment simpler to avoid some compiler issues.

Included changing related parameters to const.
Should fix complaints with previous commit. See issue #879.

Build: Fix portability in previous commit.

SPV: Support texelFetch() on a textureBuffer (no sampler).

Fixes Vulkan Khronos-internal issue 770.

SPV: Emit extension for using StorageClassStorageBuffer.

Merge pull request #878 from steve-lunarg/unroll-hint

HLSL: implement [unroll] and [loop] attributes

HLSL: implement [unroll] and [loop] attributes

This adds infrastructure suitable for any front end to create SPIR-V loop
control flags.  The only current front end doing so is HLSL.

[unroll] turns into spv::LoopControlUnrollMask
[loop] turns into spv::LoopControlDontUnrollMask
no specification means spv::LoopControlMaskNone

Merge pull request #877 from steve-lunarg/warn-fix1

Fix iomapper build warnings

Fix iomapper build warnings

Merge pull request #873 from steve-lunarg/gathercmp

HLSL: Implement missing GatherCmp method

HLSL: Implement missing GatherCmp

Merge pull request #870 from SoapGentoo/cmake-fixes

Modernise CMake #2

Modernise CMake #2

This reverts commit cfc69d95afed34e1ba1371df0ddb56f2f491a5cb.
* Change CMAKE_INSTALL_PREFIX default on Windows in order
  to prevent permission denied errors when trying to install
  to "Program Files".

Merge pull request #869 from KhronosGroup/revert-828-fixes

Revert "Modernise CMake"

Revert "Modernise CMake"

Merge pull request #828 from SoapGentoo/fixes

Modernise CMake

Make test suite optional by using CTest

Modernise CMake

* Use `GNUInstallDirs` in order to respect GNU conventions.
  This is especially important for multi-arch/multi-lib setups.
* Specify position independent mode building properly, without
  using the historic hack of adding `-fPIC` as a definition.
  This makes the build system more portable.
* Only detect C++ (and not C) to slightly speed up configuring.
* Specify C++11 mode using modern CMake idioms.
* Fix some whitespace issues.

Merge pull request #868 from TiemoJung/two_pass_io_remap

Notification phase for io remapper

Merge pull request #867 from steve-lunarg/emptyinit

HLSL: Allow empty struct initializers

Notification phase for io remapper

Adds a notification phase to the io remapper.
The idea behind this is to give the user a
chance to group uniforms and/or in/out variables
for a better pipeline layout sharing for vulkan.

Change-Id: I7492421085a4156ed3534f01d906ab390d73a623

HLSL: Allow empty struct initializers

Review request: does this harm GLSL in any way?

HLSL: Turn on hlsl-offsets by default for HLSL-source shaders.

Merge pull request #859 from steve-lunarg/sampleposition

HLSL: add standard sample position return form of GetSamplePosition m…

HLSL: add standard sample position return form of GetSamplePosition method

Multisample textures support a GetSamplePosition() method intended to query
positions given a sample index.  This cannot be truly implemented in SPIR-V,
but #753 requested returning standard positions for the 1..16 cases, which
this PR adds.  Anything besides that returns (0,0).  If the standard positions
are not used, this will be wrong.

This should be revisited when there is a real query available.

Merge pull request #855 from steve-lunarg/constantbuffer

HLSL: Add ConstantBuffer<T> syntax

HLSL: Add ConstantBuffer<T> syntax

Note: multi-dimension arrays of ConstantBuffer objects will go through uniform flattening.

Merge pull request #851 from dgkoch/dgkoch_build_fixes3

Fix Android build errors

Fix Android build errors

glslang/MachineIndependent/iomapper.cpp:207:9: error: field 'resolver' will be initialized after field 'stage' [-Werror,-Wreorder]
      : resolver(r)
        ^
glslang/MachineIndependent/iomapper.cpp:263:9: error: field 'resolver' will be initialized after field 'stage' [-Werror,-Wreorder]
      : resolver(r)
        ^
hlsl/hlslParseHelper.cpp:70:5: error: field 'gsStreamOutput' will be initialized after field 'inputPatch' [-Werror,-Wreorder]
    gsStreamOutput(nullptr),
    ^

Merge pull request #849 from steve-lunarg/samplercmp-errmsg

HLSL: add error for expected comparison sampler in SampleCmp* ops

HLSL: add error for expected comparison sampler in SampleCmp* ops

This adds an error message if a non-comparison sampler is used with
comparison sampling methods.  There's no functional change for correct shaders.

HLSL: Fix #846: support mixed ternary types.

Vector conditions properly convert the true/false expression types to same
width vector as the condition.

Scalar conditions make the true/false expressions convert to each other.

Merge pull request #847 from steve-lunarg/sb-param-fix

HLSL: fix for byte address buffers in fn parmeters

Merge pull request #842 from steve-lunarg/sb-cast

HLSL: cast non-int types to uint on Load/Store indexes

Merge pull request #840 from steve-lunarg/iomap-hlsl

HLSL: hlsl register class iomapping

HLSL: fix for byte address buffers in fn parmeters

Byte address buffers were failing to detect that they were byte address
buffers when used as fn parameters.

Note: this detection is a little awkward, and could be simplified if
it was easy to obtain the declared builtin type for an object.

HLSL: add readonly qualifier to tbuffer, so they end up as SRV

WIP: HLSL: hlsl register class iomapping

Adds --hlsl-iomap option to perform IO mapping in HLSL register space.

--shift-cbuffer-binding is now a synonym for --shift-ubo-binding.

The idea way to do this seems to be passing in a dedicated IO resolver, but
that would require more intrusive restructuring, so maybe best for its
own PR.

The TDefaultHlslIoResolver class and the former TDefaultIoResolver class
share quite a bit of mechanism in a common base class.

TODO: tbuffers are landing in the wrong register class, which needs some
investigation.  They're either wrong upstream, or the detection in the
resolver is wrong.

GLSL: Fix #822: Improve information given for syntax errors.

Also, fixed one test file that didn't have its syntax error as the last line.

HLSL: Fix #96: Support do-while loop substatements with no curly braces.