Subzero: Expand the liveness consistency check.

Subzero: Expand the liveness consistency check.

Liveness analysis includes a consistency check on each node, to verify that variables referenced in only one block do not appear to be live coming into a block (and are therefore live across multiple blocks).  This check was disabled in the entry block because there might be function arguments that are referenced only in the entry block but are still live coming in.

It seems that this entry-block exclusion has been largely unnecessary for some time.  This is because input arguments and other special variables are now pre-marked as multi-block.  The exclusion masks problems in some single-block lit tests, so it's best if it can be removed.

This CL removes the exclusion, and fixes some minor issues uncovered in the MIPS and ARM target lowering.  A key issue is that when implementing a new target lowering and using --skip-unimplemented to make progress with existing tests, it may be necessary to add FakeDef instructions to avoid liveness inconsistency errors.

Note that when this patch is applied to 448c16f0f6905460a3d27e728ed0f14a1c08ff69, it correctly identifies the liveness consistency error (as shown by a "make check-lit" failure) that was fixed in 59f2d925252faa7ae7cc688cfdfcf7c4a6f62cce.

BUG= none
R=jpp@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=b3bfcbcb11c437480c575a78c1cd0d1c213721c7

[Jim Stichnoth, 2015-08-01 14:08:18]

stichnot@chromium.org changed reviewers:
+ jpp@chromium.org, jvoung@chromium.org

[John, 2015-08-01 15:39:44]

lgtm

[Jim Stichnoth, 2015-08-02 01:46:20]

Committed patchset #1 (id:1) manually as
b3bfcbcb11c437480c575a78c1cd0d1c213721c7 (presubmit successful).