Front-end atomics: Finish adding atomic built-in functions (non-image) into the AST.

Todo.txt: Added key, including notation for who's working on something.

Update Todo.txt.

Portability: Remove unneeded "= default" to work with older compilers.

Merge pull request #31 from google/explicit-extension-directive

Only output explicit extension directives in preprocessing.

Issue 32: Guard against constructors consuming unsized arrays.

Merge pull request #33 from google/spv-builder-loop-ctor-inits-all-members

spv::Builder::Loop constructor inits all members.

spv::Builder::Loop constructor inits all members.

After construction, the Loop is effectively const.

This perturbs the IDs in SPIR-V tests because the body block
is created before generating any of the loop code, rather than
only when the body is first referenced.

Issue #28: Correctly handle a single input argument conversion when the input argument is an aggregate.

Only output explicit extension directives in preprocessing.

Preprocessing output shouldn't contain extensions enabled as
dependencies.

Issue #26: Detect empty array constructors, require at least on operand.

g++: Fix compilation issue with templates in g++.

Perf: Put in a portable std::hash that works with the pool.  This turns on the 10-15% perf gain.

SPV: Make previous commit compile with MSVC 2012.

Merge pull request #22 from google/structured-do-while

Generate correctly structured do-while loops.

Preprocessor: Prevent (and give an error on) expression division by 0.

Tests: Increase testing around incorrect use of block names and instances.

Front end: Prevent use of a block name, which has no other use in a shader.

Fix g++ build break (portibility problems with stl hash).  This partly turns off the stl improvements.

It also removes some old code that ancient compilers used to need.
However, the main issue is getting access to hash functions for
unordered_map in portable way.

Preprocessor: 1) Rationalize the "fixed atom" scheme, 2) remove redundant lookups when the text is already available.

This simplification is a prelude to eliminating what I appear unnecessary
symbol inserts into tables when tokenizing in the preprecessor, which
show up as taking notable time. (Performance issue.)  It also simply makes
the preprocessor easier to understand, which it is badly in need of.

Performance: Simple upgrade/cleanup of stl c++11 containers (10% perf. increase).

Building: Fix a couple of new warnings.

Front-end: Add error-recovery code for a switch statement that ends with a case/default that has no statements.

Generate correctly structured do-while loops.

The loop test is always emitted before the loop body.

For do-while loops, use a phi node to track whether we're
on the first loop iteration, and only check the loop test
on the second and subsequent iterations.

For do-while loops, the loop test branch no longer occurs
at the top of the loop, so it must get its own selection
merge instruction.

A block can't be the target of more than one merge instruction.
So when the loop test executes after the body (as in do-while in GLSL)
we need to introduce a dummy block to be the target of the selection
merge just before the loop test conditional branch.

The other arm of the branch exits the loop and hence is the
"break block" exception in the structured control flow rules.

Merge pull request #21 from google/allow_non_null_strings

Added a setStringsWithLengths() method to TShader.

Merge pull request #20 from google/remove_exception

Removed the single usage of exceptions in all of glslang.

Added a setStringsWithLengths() method to TShader.

This allows us to pass through non null-terminated strings.

Removed the single usage of exceptions in all of glslang.

Front-end: Implement GL_OES_gpu_shader5.

Merge pull request #18 from srk-lunarg/v30_static_analysis_fixes

Comment out some dead assignments per suggestion from static analysis…

Comment out some dead assignments per suggestion from static analysis tools.
Code is left in as comments for clarity to humans.

Merge pull request #17 from google/non-existing-string

Protect location setting methods from writing to non-existing strings.

Protect location setting methods from writing to non-existing strings.

TInputScanner advances its internal indices to the next character at
the end of get(), which means, after reading in the last character
in the user-provided shader string, internal index (currentSource)
will point to the next shader string (currentSource == numSources),
which doesn't exist. Then if a location setting method is called,
we will write to some out-of-bound memory.

A test case for this is "#line 10000\n". The eval() method in CPPline()
will evaluate 10000, but at the same time it reads in the next
token, '\n', and the currentSource will be numSources in TInputScanner.
Then a parseContext.setCurrentLine() is called, we are writing to
out-of-bound memory. Another test case will be "#line 10000 0\n".

Include specification revision as part of the version string.

Merge pull request #16 from google/pp-directive

Preprocessing directive handling

Front-end: enforce qualifiers that cannot appear on block declarations.

Also seems to pick up some white-space (line-ending) test differences with a
prevoius checkin.

Fix compile errors: about 25 .PpError didn't compile, need to be .ppError.

Merge pull request #15 from google/preprocessing-error

Added error output to the preprocessor.

This patch distinguishes preprocessing errors with normal parsing
errors and gives glslangValidator the ability to output preprocessing
errors.

Added error output to the preprocessor.

This patch distinguishes preprocessing errors with normal parsing
errors and gives glslangValidator the ability to output preprocessing
errors.

Remove uncessary line break.

We output empty lines before processing non-whitespace tokens,
not after done processing them.

Fix empty line handling in line directive callback.

The current line number for the #line directive should be passed
in as parameter to the line directive callback. Without it, we
don't know how many empty lines we should output.

Fix line number handling in line directive callback.

The line argument passed into the lineCallback function is the
literal value of the first argument of the #line directive.
lastLine in DoPreprocessing() should be updated taking into
consideration the different definitions for #line between specs.

Add a test to reveal the bug.

Merge pull request #14 from google/extensionturnedon

Simplify function calls for extensionsTurnedOn().

Lots of places in the code use extensionsTurnedOn(1, ...). This
patch introduces a new method, extensionTurnedOn(), for testing
if a single extension is turned on.

Merge pull request #11 from baldurk/size-t-int-warning-fixes

Fix warnings compiling on x64 with size_t to int casts

Simplify function calls for extensionsTurnedOn().

Lots of places in the code use extensionsTurnedOn(1, ...). This
patch introduces a new method, extensionTurnedOn(), for testing
if a single extension is turned on.

Update readme to point to Khronos landing page.

Fix incorrect used of snprintf()

Updated command-line options, adding -o for saving binaries, -G for OpenGL SPIR-V validation, -v etc.

Old uses should still work as they did before.
Also encapsulated use of these flags during parsing, for the parse context.
Added SPIR-V version to -v.

on x64, size_t is 64-bit and int is 32-bit, so needs an explicit cast

Fix clang static analyzer issues, as reported by floooh.

Merge pull request #4 from baldurk/relative-include-paths

include by relative path, so root doesn't need to be in the search path

Merge pull request #5 from floooh/fix-clang-warnings

Fix for most Xcode/clang warnings on OSX

Add some missing 'const' and texture-op guards, for future change to enum-based texture calls.

The infrastructure is in place to not do text comparisons for "texture" ... for deducing type of
texture call.  But, it is not yet turned on, as it could break some consumers.  Am soliciting
any feedback on that.

See in Initialize.cpp: const bool PureOperatorBuiltins = false;  // could break backward compatibility; pending feedback

Fix most clang warnings

- member initializing order in some constructors
- missing default branches in switch-case
- uninitialized variable if switch-case default (uncritical because
  program would exit)
- && and || brace warnings in if()

include by relative path, so root doesn't need to be in the search path

Linux build: fix case problem with new file.

Clear up a C++ compile warning

Front-end: move to rational internal array-of-array interfaces and design.  (A-of-A is not yet implemented though.)

docs: Update missing features list

glslang -> SPIR-V: smear scalars for integer (scalar * vector).  OpVectorTimesScalar is only for floats.

Use E_ prefix instead of _ for distinguishing extension strings from glext.h

Merge pull request #2 from nwnk/glext-compatibility

Fix incompatibility with <GL/glext.h>

Merge pull request #3 from fkaa/include-fix

Fix SPV include directives depending on root folder name

Fix include directive depending on root folder name

Fix incompatibility with <GL/glext.h>

New extensions in glext.h follow the pattern:

    #ifndef GL_ARB_texture_rectangle
    #define GL_ARB_texture_rectangle 1
    #define GL_TEXTURE_RECTANGLE_ARB          0x84F5
    #define GL_TEXTURE_BINDING_RECTANGLE_ARB  0x84F6
    #define GL_PROXY_TEXTURE_RECTANGLE_ARB    0x84F7
    #define GL_MAX_RECTANGLE_TEXTURE_SIZE_ARB 0x84F8
    #endif /* GL_ARB_texture_rectangle */

Versions.h tries to declare:

    const char* const GL_ARB_texture_rectangle = "GL_ARB_texture_rectangle";

Which means, if you've included glext.h before Versions.h, that the
compiler will see "const char* const 1 = ...", and rightly refuse to
continue.

The ham-fisted approach taken here is to rename the variables in
Versions.h with a leading underscore.  This does sort of undermine the
comment about "better to have the compiler do spelling checks", but.

Signed-off-by: Adam Jackson <ajax@redhat.com>

Make double underscore "__" an error for ES 300, but a warning for 310.

The 310 spec (and desktop specs) have clarified this is a waring, not an
error, but 300 tests still expect an error.

Building README: Update to CMake information.

Merge pull request #1 from antiagainst/markdown-readme

Format README using markdown.

Format README using markdown.

Final round for line endings.

Second round line ending clean up, from fresh round trip.

Add .gitattributes and normalize a few stray file's line endings

Make the "switch-label not followed by statement" warning an error, depending on version.

Add +x to two more scripts.

Add +x to two executables.

Make smoother use in GitHub

+x bison.exe, remove some extra files in runtests, and ignore more
created files.

glslang build: force the use of -std=c++11 when building with Clang.  David Neto <dneto@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31583 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: Implement AEP *_point_size extensions.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31560 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: Full turn-on of AEP tessellation semantics.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31538 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang: Finish turn on of AEP geometry shaders for ES 3.1.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31532 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: Don't emit blank lines for empty info logs. From Andrew Woloszyn <awoloszyn@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31531 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang testing: Remove dependence on testfiles from LunarGLASS.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31512 e7fa87d3-cd2b-0410-9028-fcbf551c1848

fix linux ^M issues with previous check-in

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31509 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang preprocessing: Add -E option to print out preprocessed GLSL, and do the work needed to generate a preprocessed stream.  From Andrew Woloszyn <awoloszyn@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31508 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: Added a callback mechanism for #line/#pragma/#version/#extension.  From Andrew Woloszyn <awoloszyn@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31507 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: track column numbers (they don't go anywhere yet, just get tracked).  Andrew Woloszyn <awoloszyn@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31506 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang: Add API override of version and profile for testing purposes.  From Lei Zhang <antiagainst@google.com>.

Add defaultProfile and forceDefaultVersionAndProfile into shader
compilation interface.

forceDefaultVersionAndProfile allows us to force parsing the input
shaders using defaultVersion and defaultProfile, regardless of
the #version directive in input shaders.

These two parameters enables us to programmatically invoke glslang
but specify version and profile from somewhere else like command
line.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31504 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang tests:  Fix error propagation.  From David Neto <dneto@google.com>.

The problem is that putting the while loop at the end of a pipeline was hiding any errors being generated by those tests.  The patch changes the script to use a case statement inside the while loop instead of "grep" on the outside.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31503 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang AEP: Full semantics for *_shader_io_blocks.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31490 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang AEP: Geometry shader features nominally working.  (Full semantic check and turn on pending.)  Also picked up partial tessellation shader interface, shader_io_blocks, and mirrored OES set of extensions functionality.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31487 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang -> SPV: swap arguments as needed for OpVectorTimesScalar and OpMatrixTimesScalar, and check for correct types for those as well as OpMatrixTimesVector, OpVectorTimesMatrix, and OpMatrixTimesMatrix.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31486 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang -> SPV: fix degenerate switch topologies (bug 14118).

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31485 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang ES reference: Don't allow std430 on uniform blocks (bug 14095).

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31484 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang AEP: The extension scheme, extension-enabled stage-existence testing, and compute-shader interface.  Still needs in/out blocks, unsized arrays, etc. before real testing can be done.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31479 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang front-end: Support relaxed error checking for bug 13724 "...initializer must be a constant expression..." because that change breaks existing shipping applications.  (Normal validation for portability testing still gives an error.)

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31475 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang portability: Fixed a slew of OSX compilation warnings (but not all).

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31469 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang portability: Resolve OSX errors, some other OS warnings.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31468 e7fa87d3-cd2b-0410-9028-fcbf551c1848

SPV compressor: Fixes a possible asserts.  Attribution: spvremapper@lunarg.com

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31451 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang: Allow programmatically prepending code to a shader without worrying about #version being first, to implement command-line-defined macros.  From Dejan Mircevski <deki@google.com>.

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31448 e7fa87d3-cd2b-0410-9028-fcbf551c1848

glslang ref. front-end: Add missing image format for desktop (but these are still yet to be used.)

git-svn-id: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang@31447 e7fa87d3-cd2b-0410-9028-fcbf551c1848