Subzero: Fix nondeterministic behavior in constant pool creation.

Subzero: Fix nondeterministic behavior in constant pool creation.

This issue was discovered as the result of a spurious "make check-lit" failure in undef.ll.

The problem is that constant pool label strings depend on the order the constants are created, and this order can be different with multithreaded translation.

Even -filetype=obj is affected by this, because the label string is put into the ELF .o file.  This means that different runs of Subzero on the same input could potentially produce slightly different output.

The solution is to base the label name on the actual value of the constant.  We do this by using the hex representation of the constant, rather than the sequence number of the constant within the pool.  This actually simplifies things a bit, as we no longer need to track the sequence number.

In addition, for floating-point constant labels in asm-verbose mode, include a human-readable rendering of the value in the label name.

BUG= none
R=kschimpf@google.com

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

[Jim Stichnoth, 2015-10-03 17:54:51]

stichnot@chromium.org changed reviewers:
+ jpp@chromium.org, kschimpf@google.com

[Jim Stichnoth, 2015-10-05 13:46:34]

Updated the CL to append a human-readable value to the label for floating-point
constants.

[Karl, 2015-10-05 17:54:51]

Otherwise, LGTM.

[Jim Stichnoth, 2015-10-05 20:55:55]

Committed patchset #3 (id:40001) manually as
b36757e1cf19a443ed40128b43b8e2f1f8579eb0 (presubmit successful).