Change-Id: Ie8236430bb9e30a9be2e0c1573c42183c2f4d0d4

To ensure back branches always go to a header block, create a header
block even for !testFirst loops.  Then unify common code between the
testFirst/!testFirst cases.

Generate the header-block code first, so update golden files.

Realize that certain infinite loops generate invalid SPIR-V, so put a
TODO to instead abort code generation in such cases.

Change-Id: I1e173c8f73daad186cfc666b7d72bd563ed7665d

Also update breakForLoop and call builder.closeLoop() in code
generation.

Remove dead code.

Change-Id: Ic8ce5b208530f8787989ef45a2efa405f1b45310

Also remove dead code.

Change-Id: I2c0177d8cab48b7d6f9442715aecb7951597f3c8

Change-Id: I5b53cc008349afad94b14500506fcab4d6e64d2e

Add a test for loop without a condition.

Change-Id: Idd7fc462218a84b1e745207e2975a3f2897d30a0

Change-Id: Id3bdf8b7a5606e7ce5d856ef225d5ddbe59a584b

SPV: Handle stride decorations for arrays of arrays, and using multiple type instances when strides are used.

Before, it was only including explicit interface, sufficient for IO-Block-declared
oriented interface, but not sufficient for all modes GLSL might be used with
SPIR-V.

Two things are accomplished now:
1) each id will appear exactly once
2) the OpEntryPoint list will union static use with declarations

This includes doing structure uniqueness modulo majorness, for shared nested structures.

This also distinguishes between the same struct included in multiple parent packings.

SPV: Fix an issue related with imageStore() for 2DMS.

Parser: Correct the storage qualifier of gl_HelperInvocation.

SPV: Implement GLSL built-in bitcast functions.

Display the filename if we have output, even if validating  a single file

This also made the function easier to read by identifying
left and right operands more clearly.

Fix compile warning/error due to assignment in while() loop

This causes build failures with -Werror=parentheses

An extra line of error recovery code set a non-sense combination of
es version and core profile.

'invocations' was using 0 as the "shader never set" flag and as the
default. Formalize this and explicit set to 1 and link time.

To do this, more generally use a named -1 as a not set value.