Merge pull request #311 from jherico/debug_postfix

Build: Produce different names for files in debug builds on MSVC

Merge pull request #317 from iostrowsINTEL/double_matrix_fix

SPV: Fix double matrix creation

Fix double matrix creation

Front-end: Complete GL_ARB_compute_shader, previous commit was missing new barriers.

This amends the previous commit, which ommitted barriers in version 420 for compute shader.

Front-end: Complete GL_ARB_compute_shader implementation.

Core compute shaders were working, but the extension wasn't implemented.

Fix issue #313: Catch internal attempts to modify built-in symbols that don't exist.

Also beefed up support for running compute shaders is #version 420, but this
work is only partially done.

Merge pull request #230 from dekimir/incresconst

Make two more IncludeResult members const.

Merge pull request #309 from jekstrand/sampler-params

SPV: Don't shadow sampler parameters when performing function calls

Merge pull request #308 from thp/multiple-es-compilation-units

Link: Handle multiple ES profile compilation units for a single shader stage

SPV: Don't shadow sampler parameters when performing function calls

Fixes #179

Produce different names for files in debug builds on MSVC

Check for linking multiple ES shaders to the same stage

Fix test case for the "empty" linker test case

The test result should only give one error about linking
ES and non-ES shading language compilation units:

 - empty.frag: No version info, interpreted as 100, ES
 - empty2.frag: No version info, interpreted as 100, ES
 - empty3.frag: Version declared as 110, non-ES

Previously, because the new intermediate is always created
without version/profile information, there would be two
linker errors:

 1.) When merging the new intermediate with empty.frag
 2.) When merging (new intermediate + empty.frag + empty2.frag) with empty3.frag

Now, there is only one error; as the error with merging the
new intermediate with empty.frag has been removed.

Use version/profile from first compilation stage

When linking multiple compilation units per shader stage,
the code creates a new intermediate, but fails to set its
version and profile.

This change makes it so that the new intermediate inherits
the version and profile of the first compilation unit, so
that two ES SL compilation units can be combined.

Add test case: Link 2 ESSL fragment shader units

Merge pull request #306 from johnkslang/finish-spec-const-semantics

Vulkan: Finish semantics for what creates spec-const-semantics.

Vulkan: Finish semantics for what creates spec-const-semantics.

Note: This required adding a new test mode to see the AST for vulkan tests.
This also required reworking some deeper parts of type creation, regarding
when storage qualification and constness is deduced bottom-up or dictated
top-down.

Merge pull request #305 from steve-lunarg/intrinsics

HLSL: Add base class TParseables for intrinsic / builtin generation.

Add base class TParseables for intrinsic / builtin generation.
Add stubbed HLSL derivation.  GLSL derivation is still called TBuiltIns,
for historical compatibility.

Merge pull request #302 from amdrexu/bugfix

SPV: Fix an issue of interpolation decoration.

SPV: Fix an issue of interpolation decoration.

GLSL interpolation qualifiers and auxiliary storage qualifiers are not
mutually exclusive. So when they are translated to SPIR-V decorations, two
independent utility methods should be employed to do this job.

KHR_vulkan_glsl: name mangle distinguish pure textures.

Fixes issue #252.

SPV: Don't put locations on OpTypeStruct. Related to Issue #291.

Merge pull request #301 from dneto0/fix-android-build-atoi-in-cstddef

Build: atoi comes from stddef.h or cstddef

atoi comes from stddef.h or cstddef

This is required to fix the Android build for ARM.

Merge pull request #297 from antiagainst/config-tests

Test compiling shaders with given resource limits using GTest.

Merge pull request #300 from Qining/fix-slow-down-in-no-contraction-propagation

Fix the slow down in noContraction propagation

remove redundant 'return false'

HLSL: Add more matrix types to the grammar.

Full stack: distinguish between a scalar and a vector of size 1.

There have been GLSL extensions considering this, and HLSL does it.
This is a fully backward compatible change that allows this distinction.

Fix the slow down in noContraction propagation

SPV: Don't emit memory barrier for ESSL barrier(), but still do for GLSL barrier().

Addresses issue #205.
Current open Khronos bug for finalizing this.

Merge pull request #298 from thp/remove-unused-prototype

Nonfunctional: Remove declaration for ShGetPhysicalAttributeBindings

Remove declaration for ShGetPhysicalAttributeBindings

The function ShGetPhysicalAttributeBindings() is declared in the
header, but not defined (or referenced) anywhere in the codebase.

Test compiling shaders with given resource limits using GTest.

Rename DefaultResourceLimits.* to ResourceLimits.*.

Merge pull request #288 from thp/attribute-reflection

Reflection: Add support for querying vertex attributes in reflection API

Update test cases for vertex attribute reflection

Add support for querying vertex attributes in reflection API

Merge pull request #295 from amdrexu/bugfix

Parser: Redeclaration of gl_CullDistance is disallowed mistakenly.

Merge pull request #293 from antiagainst/link-tests

Testing: Add link tests in the GTest framework

Parser: Redeclaration of gl_CullDistance is disallowed mistakenly.

Run link tests in the GTest framework.

Merge pull request #294 from scyganINTEL/scygan-fix-io_storage_structs-location

SPV: Fix missing location decoration for structures put directly on input/output interfaces

Fix missing location decoration for structures put directly on input/output interfaces

Spec for decorating the OpVariable:
"The remaining variables listed by OpEntryPoint with the Input or Output storage class form the user-defined variable interface. These variables must be identified with a Location decoration"

Spec for decorating struct type:
"The layout of a structure type used as an Input or Output depends on whether it is also a Block (i.e. has a Block decoration).
If it is a not a Block, then the structure type must have a Location decoration"

Fix several comments in GTest fixture.

Non-functional: Fix some comments English and brace formatting in recent merges.

Merge branch 'scyganINTEL-struct_member_decorations'

Merge branch 'struct_member_decorations' of https://github.com/scyganINTEL/glslang into scyganINTEL-struct_member_decorations

Merge pull request #286 from amdrexu/bugfix

SPV: Correct an issue in createUnaryMatrixOperation().

SPV: Correct an issue in createUnaryMatrixOperation().

Type of the source is not necessarily the same as that of the destination.

Merge pull request #285 from baldurk/vs2010-fixes

VS2010 compilation fixes

SPV: Don't add clip/cull distance capabilities unless used.

These capabalities were added on declaration of the members, but
that is considered too aggressive, as those members are automatically
declared in some shaders that don't use them.  Now, actual access
is needed to make the capabalities be declared.

Tests: Add a set of gtest-file-based HLSL tests.

Compile fix - if _MSC_VER is undefined, _MSC_VER < 1700 is true!

[VS2010] Define strtoll() and atoll() functions

[VS2010] Locally define std::to_string on VS2010 as well as android

[VS2010] Remove use of ranged-for loops

[VS2010] Remove use of "= default" and "= delete"

[VS2010] Change "using X = Y;" to "typedef Y X;"

Merge pull request #284 from antiagainst/gtest-hlsl

Tests: Add support for testing file-based HLSL source code in GTest.

Merge pull request #253 from amdrexu/bugfix2

SPV: Handle matrix's OpFConvert vector by vector.

Merge pull request #281 from amdrexu/bugfix

Parser: Add missing vector relational functions for double type.

Merge pull request #282 from baldurk/android-warning-fix

Build: Minor android compile warning fix

Add support for testing file-based HLSL source code in GTest.

Use DetachThreadLinux wrapper on android

* Fixes a warning - static function being defined but not used.
* Just in case any more code is added to DetachThreadLinux, this will go
  through the same path on both platforms.

Parser: Add missing vector relational functions for double type.

SPV: Handle matrix's OpFConvert vector by vector.

HLSL: Add function call syntax and AST building.

Merge pull request #278 from greg-lunarg/greg-mr3

Build: fix for -Werror=non-virtual-dtor for Android system builds

Merge pull request #276 from amdrexu/bugfix

Parser: Redeclare gl_ClipDistance/gl_CullDistance should update their array sizes.

Merge pull request #277 from amdrexu/bugfix3

SPV: Add capability ImageGatherExtended if necessary.

fix for -Werror=non-virtual-dtor for Android system builds

SPV: Add capability ImageGatherExtended if necessary.

Parser: Redeclare gl_ClipDistance/gl_CullDistance should update their array sizes.

Merge pull request #274 from antiagainst/stoul-strtoul

Use strtoul instead of stoul for Android.

Use strtoul instead of stoul for Android.

Merge pull request #273 from antiagainst/appveyor-readme-status

Add badge status for Appveyor build in README.

Add badge status for Appveyor build in README.

Merge pull request #272 from antiagainst/appveyor

Configuration for Windows buildbot & fix a bug

Use std::string for TBD/missing functionality reporting.

We can have multiple instances of the same string, so comparing
const char* is not guaranteed working.

Fixed the failure on VS 2013 with Debug build.

Add test configuration for Appveyor.

Merge pull request #222 from Qining/support-precise

Full stack: Support *precise* qualifier

Merge pull request #270 from scyganINTEL/bitfield_buildins

Use proper precision qualifiers for bitfield build-in functions on ES

Merge pull request #271 from greg-lunarg/greg-mr2

fix check for non-positive array size

remove double underscore

Rebase to d3d3ce7160001846ede74978954e67f4f9fd84e4,update comments, rename a member variable name, fix format issue

fix check for non-positive array size

Use proper precision qualifiers for bitfield build-in functions on ES

This change causes ES shaders to precision qualifiers for build-in functions as defined in ESSL spec. It especially mattersfor functions that are defined as highp or taking a highp.

Fixes vulkanCTS dEQP-VK.glsl.builtin.function.integer.bitfieldreverse.*, where bitfieldReverse() retval was wrongly marked as RelaxedPrecision.

Note: floatBitsToInt/floatBitsToUInt precision is also broken, but in different way - so it is not addressed here.

integer type operation should not be noContraction

simplified the symbol definition collector code, add assert()

add a test for matrix, fix a variable name

Resolve comments

1. Sink adding noContraction decoration to createBinaryOperation() and
createUnaryOperation().

2. Fix comments.

3. Remove the #define of my delimiter, use global constant char.

Removed the redundant functionalities in the assignee checking traverser

Fix the test of precise output function parameter

Rebase to 5cc344d8ce09a6d35ff88b9ea9f351062d779081 and update the
expected test result.

Precise and noContraction propagation

    Reimplement the whole workflow to make that: precise'ness of struct
    members won't spread to other non-precise members of the same struct
    instance.

    Approach:
    1. Build the map from symbols to their defining nodes. And for each
    object node (StructIndex, DirectIndex, Symbol nodes, etc), generates an
    accesschain path. Different AST nodes that indicating a same object
    should have the same accesschain path.

    2. Along the building phase in step 1, collect the initial set of
    'precise' (AST qualifier: 'noContraction') objects' accesschain paths.

    3. Start with the initial set of 'precise' accesschain paths, use it as
    a worklist, do as the following steps until the worklist is empty:

        1) Pop an accesschain path from worklist.
        2) Get the symbol part from the accesschain path.
        3) Find the defining nodes of that symbol.
        4) For each defining node, check whether it is defining a 'precise'
        object, or its assignee has nested 'precise' object. Get the
        incremental path from assignee to its nested 'precise' object (if
        any).
        5) Traverse the right side of the defining node, obtain the
        accesschain paths of the corresponding involved 'precise' objects.
        Update the worklist with those new objects' accesschain paths.
        Label involved operations with 'noContraction'.

    In each step, whenever we find the parent object of an nested object is
    'precise' (has 'noContraction' qualifier), we let the nested object
    inherit the 'precise'ness from its parent object.

Comments only.

SPIR-V do not decorate: members of plain structs with location; non-top level members with interp.

This fixes some vulkanCTS tests that use struct arrays as a member of in/out interface blocks.

From Vulkan spec:
"If it is a not a Block, then the structure type must have a Location decoration. Its members are assigned consecutive locations in their declaration order, with the first member assigned to the location specified for the structure type. >>>>> The members, and their nested types, must not themselves have Location decorations <<<<"

From SPIR-V spec:
"When applied to structure-type members, the Decorations Noperspective, Flat, Patch, Centroid, and Sample can only be applied to the top-level members of the structure type. (Nested objects' types cannot be structures whose members are decorated with these decorations.)"

README: Editorial update.

SPV: Non-functional: modularize createInvocationsOperation() from the previous commit.