SPV_FAILED_MATCH error mutes DiagnosticStream.

The DiagnosticStream will not emit the accumulated message
text if the error is SPV_FAILED_MATCH.

Change various interfaces to accept the intended error
code instead of a boolean "is_optional".  This allows
us to avoid repeating the following type of logic deep
inside helper methods:

  if (is_optional) return SPV_FAILED_MATCH;
  return diagnostic() << " message text ";

Removed un-needed comments and fixed merge issue.

Updated string escaping in the assembler.

Strings are now escaped correctly when assembling.
TODO dissassembler support for strange strings (newlines for example).

Update readme. Long instructions, literal number encoding

Add negative-coverage location to operand-class-test-coverage.csv.

Parse and encode literal integers to the right width

Affects OpConstant, and OpSwitch.

Adds constant libspirv::kUnknownType for readability.

Adds tests for hexadecimal number parsing.

Updates syntax.md to describe hex parsing, including
sign extension.

Add AssemblyContext::parseNumber

It parses a text string for a value of a given target type.

DiagnosticStream can convert to a stored error code

Use this to shorten error return code in the assembler.

For example, change this:

   if (error = something()) {
      diagnostic() << " Bad integer literal " << value;
      return SPV_ERROR_INVALID_VALUE;
   }

to this:

   if (error = something())
      return diagnostic() << " Bad integer literal " << value;

Also shorten code due to the fact that binaryEncodeU32 and
binaryCodeU64 can't fail (short of failure to expand a std::vector).

Negative test for FunctionControl.

Negative test for LoopControl.

Negative test for SelectionControl.

Negative test for GroupOperation.

Negative test for LinkageType.

Negative test for BuiltIn.

Negative test for AccessQualifier.

Negative test for FuncParamAttr.

Negative test for Decoration.

Negative test for FPRoundingMode.

Negative test for FPFastMathMode.

Negative test for ImageFormat.

Make compilation input strings const.

Also rerun clang-format.

Negative test for SamplerFilterMode.

Negative test for SamplerAddressingMode.

Clarify handling of OperandImageOperands.

Negative test for ImageOperands.

Negative test for Dim.

Test OpTypeForwardPointer.

Added tracking to types and validated certain instructions.

We need to know how to generate correct SPIRV for cases like
OpConstant %int64 42 since the current parser will encode the 42 as a
32-bit value incorrectly.

This change is the first of a pair. This one tracks types, and makes
sure that OpConstant and OpSpecConstant are only ever called with
Integer or Float types, and OpSwitch is only called with integer
generating values.

Remove spv_opcode_flags_t and flags fields.

The flags fields in both spv_opcode_desc_t and spv_operand_desc_t
are redundant with the capabilities mask field in the same
structure.

Negative test for ExecutionMode.

Negative test for AddressingModel, MemoryModel.

Negative test for ExecutionModel.

Negative test for SourceLanguage.

Also, use "class" instead of "type" in the coverage CSV.

Accept long UTF-8 Literal Strings

Support strings with up to 65535 chars, and null.

Move the definition of spv_instruction_t to an internal
header file, since it now depends on C++ and is not
used by the external interface.

Use a std::vector<uint32_t> in spv_instruction_t
instead of a fixed size array.

Remove stale TODO. Fix use of OpCode to 'instruction'

Update readme: capabilities are up to date

Define spv_capability_mask_t for readability.

Test capabilities for enums sections 3.21-3.31

Fixes dependencies among capabilities.  (The table should store
the mask of capabilites, not the capability enum.)

Remove the old spot check test for capabilities of enums.

Fix capabilities for enums in Rev32, sec 3.3-3.20

Move AutoText and EnumCase into spvtest namespace

Also fix using declaration order.
Also put a few tests into the anonymous namespace.

Update readme: Finish assembler !<integer> support

Also internal cleanups and additional tests.

Test operand-table capabilities.

Add TODO for fixing endianness testing.

Currently it uses a union, a write to one field, and read from
another of a different type.

Moves the endianness-determining macro close to the variable
it references.

Test SPV_CAPABILITY_AS_MASK macro

Use 3-arg MakeInstruction to reduce logic in tests

Avoids open-coded vector inserts and two variables in various
tests.  Should improve readability.

Test opcode table capability entries.

Make capabilities a bitfield in opcode/operand tables.

Create spv_language_capabilities_t and widen capabilities fields.

If this is acceptable, I'll do a wholesale switch of operand and opcode
definitions to capability bitfields.

Add assembler test for OpEnqueueKernel.

Adds a form of spvtest::MakeInstruction which takes two vectors
of operands.  That leads to simpler test code.
We can clean up some other test code, in another CL.

Clarify !<integer> parsing.

Implement some outstanding feedback from
Ic29c5a4a8178a62a5a1acad13d02f19cc1307097:

 - use "token" instead of "word" when referring to assembly text

 - specify how the numbers are parsed

Add a test for negative numbers.

Try to fix the Android build.

Detect overflow in !<integer>.

Add a .csv showing test coverage for operand class

It shows which spv::OperandClass values are used in
the syntax table in source/opcode.inc, and what tests
exercise those operand types.

Simplify (and test) the alternate-parsing pattern.

Assembly test for OpSwitch

Removes old weak and fragile tests for OpSwitch.

Adds spvtest::TextToBinaryTest::CompileWithFormatFailure

Assembly failure cases for OpMemoryBarrier

Fully tests OperandMemorySemantics from the syntax table.

Assembler failing cases for OpCapability

Document non-use of certain operand types

Operand class enum values not used in the syntax table:
- Image channel order: only used to describe a return value
- Image channel type: only used to describe a return value
- "Image operands": used to make a section in the spec
  to describe values used to construct an optional image id,
  but does not itself appear in the syntax table.

Removes some TODOs from opcode.cpp.

Refactored dynamic and static state out of text processing.

This reduces the number of arguments required to be passed
to every single function. This is in preparation for adding
id tracking.

Run clang-format.

Implement alternate-parsing mode for !<integer>.

Readme: capabilities for instructions are ok

Capabilities for enumerants may be out of date.

Updated syntax.md to remove references to % numerical ids.

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.