Remove isIdType(), which is now dead code.

Test bad 2nd target to OpGroupMemberDecorate

Assembly test for OpGroupMemberDecorate

Provides test coverage for OperandVariableIdLiteral
from the syntax table in the file 'opcode.inc'.

Uncomment some !<integer> tests.

Cleanup: Use EnumCase in OpDecorateSimpleTest

Cleanup: Use EnumCase for ExecutionMode test

Assembler test for optional operands of OpSource

These are the first tests to cover OperandOptionalId
and OperandOptionalLiteralString from the grammar in
opcode.inc

Remove SPV_OPERAND_TYPE_LITERAL

All uses of OptionalLiteral by the SPIR-V spec are used
for literal numbers.

Also rename:
- SPV_OPERAND_TYPE_OPTIONAL_LITERAL to
  SPV_OPERAND_TYPE_OPTIONAL_LITERAL_NUMBER.
- SPV_OPERAND_TYPE_VARIABLE_LITERAL to
  SPV_OPERAND_TYPE_VARIABLE_LITERAL_NUMBER.
- SPV_OPERAND_TYPE_VARIABLE_LITERAL_ID to
  SPV_OPERAND_TYPE_VARIABLE_LITERAL_NUMBER_ID.
- SPV_OPERAND_TYPE_VARIABLE_ID_LITERAL to
  SPV_OPERAND_TYPE_VARIABLE_ID_LITERAL_NUMBER.
- SPV_OPERAND_TYPE_LITERAL_IN_OPTIONAL_TUPLE to
  SPV_OPERAND_TYPE_LITERAL_NUMBER_IN_OPTIONAL_TUPLE.

Dissasembler test showing reordering of masks

Execution scope, memory semantics operands are IDs

They shouldn't be parsed or printed as masks.

Disassembler support for mask expressions.

Update Readme: Support Rev32; IDs never alias

Memory semantics Relaxed is a synonym for None

The disassembler should prefer to print Relaxed,
I think. (Untested.)

Assembler tests for remaining Debug instructions

OpString, OpName, OpMemberName, OpLine, OpNoLine,
OpSourceContinued.

Update core instruction syntax to Rev32

Many instructions added and a few changed structure.

Workarounds:
- Some operands can be enabled by either one of two
  capabilities.  The spv_operand_desc_t does not handle that
  now. For now just select the first one.

Fixes to tests:
- OpLoopMerge now takes a mandatory continue target.
- OpTypePipe drops the type argument.  Pipes are opaque.
- OpLine no longer takes a target ID argument.

The ID validator was fixed the OpLine and OpTypePipe
changes.  Those were the only ID validator tests affected.

The patch to the spec doc generator was updated so it handles
the two-capability case, even if in an hacky way.

Assembler test TODOs for image instructions.

Assembler supports new decorations in Rev32

They are:
 - NoContraction
 - InputTargetIndex
 - Alignment

Rev32: Remove DepthAny and mark Xfb as requiring TransformFeedback.

Fix Capability dependencies on capabilities, Rev32

Assembler support for new capabilities in Rev32

Support Image operand MinLod

Assembler supports new builtins in Rev32

They are VertexIndex and InstanceIndex.

Assembler support for Nontemporal memory access

Define two macros to simplify code for ExecutionMode operands.

Fix permissions on test/TextToBinary.Barrier.cpp

Support Dim InputTarget

Support StorageClass PushConstant

Support ExecutionMode IndependentForwardProgress

Assembler test for Memory Semantics enum

Enables mask expression parsing for Memory Semantics arguments,
e.g. on OpMemoryBarrier.

All values now represent symbolic names instead of mixed with numeric.

Also removed un-necessary heap-allocation of spv_named_id_table.
This removed the necessity to expose a function to create/destroy it
and simplified the interface.

Remove executable file mode bits on source files.

Update to Rev32 headers. Part 1.

Just enough fixes to code make it build and pass tests.

Core changes:
 - Fix spelling for: NoPerspective, NonWritable, NonReadable,
 - Remove NoStaticUse, RelaxedMask

GLSL changes:
 - Fixed spelling for: InverseSqrt, FaceForward, MatrixInverse,
   SmoothStep, FindILsb, FindSMsb, FindUMsb
 - Replace Mix with IMix and FMix
 - Remove AddCarry, SubBorrow, MulExtended

Replace header OpenCLLib.h with OpenCL.std.h

TODO:
 - Regenerate the core instruction syntax table (source/opcode.inc)
 - Add test coverage for new enums and instructions.

Switched VecTypeHint to take a LiteralNumber instead of an ID

Removed duplicate code due to what looks like a merge issue

Cleanups for EnumCase

Make it a class, since it has non-trivial behaviour for converting
the enumerated value to a uint32_t value. (Comply with style guide.)

Merge EnumCaseWithOperands into EnumCase.

Put the test fixture into spvtest namespace.

All test utility code should go into the spvtest namespace.

Fix MSVC build for boolean opeartions on enums.

Fix the Supported Features section.

Readme: Assembler supports all of Rev31, for 32-bit code

Also say that we have fixed problems building with MSVC 2013.
Also document other recent changes.

Put const back on const char* in test case structs

A bunch of small fixes to build in MSVC2013.

Fixed an issue where some of the tests were testing
the wrong word with the wrong operation. (| != ||).

Coalesced the many versions of EnumCase into one.
Added a get_value() to EnumCase to convert to a uint32_t.

Replaces ASSERT_TRUE(pointer), with ASSERT_NE(nullptr, pointer),
so that we do not do implicit pointer->bool conversion.

Removed const from some test structs since gtest needs to be
able to swap them.

Assembler support for image operands from Rev31

Rev32 and later add many more image operands, and
rearrange their values.

Let EncodeAndDecodeSuccessfully remove preamble comments.

Clean up code for encoding literal operands.

Remove SPV_OPERAND_TYPE_VARIABLE_MEMORY_ACCESS

If a memory mask operand is present, it is a mask.  The mask appears
only once, so just use SPV_OPERAND_TYPE_OPTIONAL_MEMORY_MASK.

The "variable literals" aspect comes into play as follows: if the
Aligned bit is set in the mask, then the parser will be made to
expect the alignment value as a literal number operand that follows
the mask.  That is done through mask operand expansion.

Assembler: mask expressions where 1 bits imply operands

Properly support a memory access mask with a combination
of bits, including the Aligned bit. When the Aligned bit is
set, the parser should expect an alignment value literal operand.

Document the syntax of mask expressions

Assembler support for simple mask expressions

For example, support combining mask enums with "|",
such as "NotNaN|AllowRecip" for the fast math mode.

This is supported for mask values that don't modify the
expected operand pattern:
 - fast math mode
 - function control
 - loop control
 - selection control

TODO: disassembler support to print them as mask expressions.

Generalize spvOperandTableNameLookup to take string length.

This is preparation for parsing mask expressions.

Forbid !<integer> preceding or succeeding '='.

Assembler test for Dim, ImageFormat enums.

Adds SPV_OPERAND_TYPE_SAMPLER_IMAGE_FORMAT, and the
translation tables for its values from the spv::ImageFormat values.

Document inability to use names from 3.12, 1.13

You can't use names from 3.12 Image Channel Order and
3.13 Image Channel Data Type since in the intstruction grammar,
they are only used as return values, but never named arguments
to instructions.

Use generalized EnumCase in AccessQualifier assembly tests

This is in preparation for coming tests that will also
use the templated EnumCase instead of making their
own structs.

Also reformat AccessQualifier test.

Handle IDs among !<integer> operands.

Assembler test for SamplerAddressingMode, SamplerFilterMode

Uses OpConstantSampler instruction.

Assembler test for Storage Class enum values

This covers the storage classes in SPIR-V Rev31.  Rev32 has more.

Support multiple word literal numbers as constants.

Add a new operand type SPV_OPERAND_TYPE_MULTIWORD_LITERAL_NUMBER
to represent those operands that can expand into multiple words.
Now only OpConstant and OpSpecConstant have such kind of operand.

Update CHANGES and Future Work

Assembler test for ExecutionScope, with OpGroupIAdd

Use spvOperandTypeStr to generalize the error message
for failure to parse an ID.
Extend spvOperandTypeStr to cover SPV_OPERAND_TYPE_EXECUTION_SCOPE.

Assembler test for OpTypePipe

Covers all AccessQualifier enums.

In Rev32, OpTypePipe no longer has a type argument.
Added TODOs for this.

Assembler test for single-valued Memory Access flag.

Assembler test for single-valued FunctionControl mask

Fixed name of "Inline" enumerant.

Support for combinations of function control masks should come later.

Remove useless variable.

Assembler test for single-valued LoopControl mask

Support for combining masks should come later.

Assembler test for single-valued SelectionControl mask

Support for combining masks should come later.

Clarify the syntax. Some named enumerants are unusable

You can't use a named enumerant if it's only meaningful
in an operand supplied as an ID to a target instruction.
The place where you'd use the name is something like an
OpConstant, but there's not enough context to bring those
names into scope, unless you're willing to tolerate
potential collisions.

Occurs for the names in:
- 3.25 Memory Semantics
- 3.27 Scope ID
- 3.29 Kernel Enqueue Flags
- 3.30 Kernel Profiling Info

Simplify code for handling literals in disassembler.

Move the syntax description into its own file

Add patch for generating opcode.inc syntax table.

Run clang-format.

Assembler test for OpExecutionMode

Assembler test for OpEntryPoint

Simplify format indication in CompileSuccesfully().

Limit the use of spvCheck and spvCheckReturn to validator.

spvCheck is indeed just an if-statement. Defining such a macro
doesn't help much.

Update readme with latest news.

Move bitwisecast.h to source/ and add include guard.

Fix an infinite loop during message generation.

Rename getWord to spvGetWord and unit test it.

Add option to change assembly syntax format in the dis tool.

Handle operands when OpCode is !<integer>.

Add TODO for capability dependencies for Rev32+

Assembler test for OpMemoryModel

Add library function parameter to set assembly format for disassembler.

Add option to change assembly syntax format in the as tool.

Assembler tests for OpCapability.

Adds test file for instructions in the Mode-Setting section
of the SPIR-V spec.

Assembler test for LinkageAttributes decorations

Assembler tests for all decorations except Linkage attributes

Fixes typos in various parser tables.

Updates readme.md with the fixes.

Run clang-format.

Add an API parameter to choose assembly syntax format for assembler.

Added a new enum for supported assembly syntax formats:
Canonical Assembly Format (CAF) and Assignment Assembly Format (AAF).

Updated assembler interface functions to support choice of assembly
syntax format.

Simplify checking for !<integer> first word.

Assembler test for OpSourceExtension.

Add test utility function MakeVector(std::string) to encode
a string as a vector of 32-bit words, as per the SPIR-V literal
string encoding.

Added missing diagnostic when disassembling empty file.

Added diagnostic messages for what should be an internal failure
that never happens. I figure if we return "failed" for something the
user cannot control we should print a message for it.

Test assembly of OpSource using all language enums

Adds a file to test assembly of instructions in the "Debug" section
of the SPIR-V spec.

Update CHANGES in the readme.md

Avoid undefined pointer given to spvTextDestroy

The undefined pointer might get through to the call
to spvTextDestroy if the test fails on an earlier step.
In that case I just want to see the test failure, and
not a test failure followed by undefined behaviour.

Fix build on MSC

Visual C++ compiler doesn't allow ternary select where type depends on
condition even if there is a user-defined conversion operator to a common
type.

Add OpNop, OpUndef tests for spvTextToBinary

Add text_fixture::TextToBinaryTestBase::CompiledInstructions,
to more easily just examine the generated instructions by skipping
over the header.

Add spvtest::MakeInstruction utility function to easily generate
a vector containing an opcode and its operands.

Fix interface function calls to the new API.

Preserve float values during encoding and decoding roundtrip.

Removed the ambiguity in the API for cleaning input vs output data.

Previous the api used spv_text_t and spv_binary_t for both input
and output, but depending on the usage, you either MUST
call spvBinaryDestroy or you MUST NOT call spvBinaryDestroy on the
pointer.

Simplify uses of spvBinaryDestroy in tests

Always try to destroy the binary during common methods
of test fixtures.  This is safe if no other code in the test
attempted to destroy the binary.

Take advantage of the fact spvBinaryDestroy is a no-op on a nullptr,
by eliminating the null pointer check in the caller.