Allow renderables to be specified in SequenceTemplate passes

Rename SequenceTemplate::Pass::renderable_name to slot_name

Avoid NaN issues in rare cases in the Cook-Torrance shader

The dot product may sometimes be very slightly over 1.0, making the
first argument to pow negative and giving undefined results.

Fix loading sRGB external images for textures

This fixes bugs introduced in 9087cc0 (sRGB state ignored when setting
internal format) and 635e5dc (external_image_srgb handled incorrectly).

Remove some vestigial code from Sequence

Apparently a few internal parts were left behind when I removed the
related interfaces.

Use a vector instead of a list for Sequence's steps

They're not large or numerous enough to warrant a list.

Add compatibility typedefs for the pipeline classes

Must be in a separate commit for git to pick up on the renames in the
previous one.

Rename Pipeline to Sequence

Pipeline::Pass is now Sequence::Step.  These better match what the
classes do.

New way of loading scenes in the viewer

Add a scene type property to the Blender exporter

Support loading scenes from Resources

Add a registry-based generic loader to Scene

This is rather similar to the one in Material, but there's just enough
differences and template complications that it isn't trivial to abstract.

Use latest OpenGL version in viewer

Use an Options struct to process command line options in viewer

Improve logic for exporting material sampler references

Add an option to automatically detect if tangents need to be exported

It's enabled by default, which should reduce hassle when authoring
models.

Add the group and weight attributes to msp_interface.glsl

This got forgotten in e1d0738.

Include only tangent in mesh data and calculate binormal on the fly

This saves one vertex attribute.

Fix incorrect definition of the builtin cross function in GLSL

Use unit 0 UVs for exporting TBNs if no UV layer is specified

Ensure that objects have unique names while exporting a scene

Compare materials when looking for object clones while exporting a scene

Recognize a certain configuration of nodes as inverting Y for normals

Models made for DirectX treat the normal map values as having positive
Y pointing toward the bottom of the image, which is opposite to what my
engine and Blender expect.  This allows fixing that without modifying
the original texture.

Force the usage of normalmap as RGB data

Use more efficient methods to manipulate and store texture data

Remove deprecated things from the maketex script

Fix a bug in makefont.py if no ranges are given

Convert the texture and font creation scripts to python3

Don't ignore structs entirely in UnusedVariableRemover

Structs may have arrays in them, and array sizes may have references to
specialization constants which must be kept.

Adjust specular calculation in the Phong shader

Since the look vector points towards the surface, it makes more sense
to calculate the reflected vector towards the light.  The calculation
is symmetric so the result is the same.

Refactor vertex transformations in the common shader

It makes more sense to have virtual accessors for the transformation
matrices so the returned matrix can be used multiple times.

Rearrange type specification in material datafiles

The type is now specified as a separate statement with no substatements.
This is what I wanted to do originally but couldn't figure out a clean
way to do it.

No legacy support because the new material system as a whole is still
fairly new and hasn't been used in many places.

Migrate from LoadableTypeRegistry to TypeRegistry

Fix normal map property name in material exporter

Fix incorrect plurals in exporter scripts

Replacing "map" with "atlas" was not correct in the plural case

Fix refactoring errors in scene exporter

Recognize textures through an RGB to BW node in Blender

Deprecated InstanceScene and Renderable::get_instance_key

InstanceArray is a superior way of rendering multiple instances of an
object.

Redesign Text as an ObjectInstance

It's rare to need several copies of the same text, but positioning texts
is useful.

Text no longer copies the Technique but instead sets the texture in
setup_render.

Overhaul texture management in rendering classes

Textures are now set based on the uniform name (tag) rather than texture
unit.  The Texturing class has been deprecated because it was making
things difficult when textures come from multiple sources, such as with
effects or decals.  RenderPass and Renderer now handle multiple textures
directly.

There's some compatibility code but things are likely to break.

Use a vector for storage in DefaultUniformBlock

Uniform locations are normally spaced no more than 4 apart, so even with
a few empty spots a vector is going to be more efficient than a map.

Split UniformBlock into two classes

There's basically no overlap between default and buffer-backed blocks,
so no point in having them in the same class.

Refactor ProgramData to store blocks in vectors

Sort uniform blocks from SPIR-V modules

Use the new _member utility functions to search and sort things

Remove unnecessary std:: qualifiers from .cpp files

Use Tag to identify uniforms in Program and ProgramData

Setting uniforms is a common operation and this should be faster than
using strings.

Make Tags convertible back to the original string in debug builds

This helps in producing more readable error messages

Clear the empty name flag after emitting a member access

Pass Tag by value, not by reference

Since the tag object is small, it can be passed in a register.

Keeping a deprecated version would make it hard to tell which version
is being called.

Add an option to compile SPIR-V in the command-line GLSL compiler

Add a couple more test cases

Test the SPIR-V backend using libspirv

Add a SPIR-V backend to the GLSL compiler

Fix a signedness warning

Disallow specializing when compiling a module

The specialization constants would be ignored anyway, so better be
explicit about it.

Add an output file option to the command-line GLSL compiler

Visit the entire call graph in DependencyCollector

Convert operands of shift operators to the same vector size

Add a separate constant to enable clipping in standard shaders

Declaring an array with zero size is not allowed, so using zero for
max_clip_planes doesn't work.

Validate GLSL flow control

In particular, it's an error to reach the end of a non-void function
without a return statement.

Make the GLSL compiler test runner able to verify non-error diagnostics

Remove unreachable code after a jump statement

Check that non-constant data is not accessed from constant expressions

Compare array sizes in TypeComparer

Rewrite type comparisons as a visitor

Require conditions to be booleans

Disallow certain types from appearing on variables

Adjust GLSL builtins

Remove some functions which are complicated to implement in SPIR-V due
to the use of ad-hoc struct types or pointers.  I may add them back
later if there turns out to be some use for them.

Add inline definitions for some overloads taking mixed vector and scalar
arguments.  SPIR-V operations require all operands to have the same
number of elements.  Builtin functions are always inlined even if used
more than once.

Leave no-op constructors as they are

In particular, don't break a matrix down into columns and construct
it again.  These could just be removed from the AST entirely, but the
expression resolver currently does not do node replacement and it's
easier to just skip this in the backend.

Fix GLSL member access index calculation

Also show it in debug output

Refactor layout qualifier creation with constructors

Show label for all interface blocks in GLSL debug output

Track composite assignments and references in UnusedVariableRemover

Refactor add_to_chain as a common utility function

Mark all expression result types as used

UnusedTypeRemover was missing several expression node types.  Now it
handles expressions through the general expression mechanism.

Make DumpTree able to dump arbitrary nodes

Remove that ability from Formatter since it makes assumptions about the
state of the nodes, like certain references existing.

Don't return references from apply functions

It's somewhat error prone, even though it hasn't caused any problems
yet.

Always use member access for interface blocks, even anonymous ones

Having variable references magically refer to variables inside interface
blocks causes trouble when generating SPIR-V.

This currently breaks one test case because UnusedVariableRemover can't
track access chains.

Output the interface block as is in the AST dump

This is more useful for debugging

Change the naming scheme for the interface block map

Adding a space in the keys for block name makes it impossible for them to
conflict with identifiers and allows instance names to be used directly.

Recognize matrix data order qualifiers and add one if missing

Require initializers on constant variables

Offset layout qualifiers are only allowed on uniform block members

Same goes for the alignment qualifier.

Add visitors to calculate offsets of struct members

It's not used yet, but will be required for creating SPIR-V modules.

Make function dependency collection its own visitor class

I need it for creating SPIR-V entry points too.

Store the index of the member in MemberAccess

Fix mismatched new[]/delete

Rearrange some utility struct constructors

Guard against hitting the end of the array in get_gl_type

Comments and cosmetic cleanups

Do an early return from Mesh::draw if there's no data

Otherwise if would try to set zero-sized storage for the buffers, which
is not allowed.

Also guard against empty batches in resize_buffers just in case.

Use uniform block and texture bindings from GLSL

Add a target version option to the command-line GLSL compiler

Better way to request features for different GLSL versions

Assign bindings to all uniform blocks and sampler uniforms

Add legacy conversion for binding layout qualifiers

These are not yet used by Program because there's no good way to detect
which uniforms had bindings in case the implementation does support the
qualifiers.  Anything missing a binding gets assigned binding 0, which
is also a valid manually specified binding.  Upcoming commits will move
bindings entirely to GLSL side.

Store constant id limit as range, not max value

Better checking of where the binding qualifier is allowed

Disallow the descriptor set qualifier for now

It's only meaningful for Vulkan, which isn't implemented yet