Order jump tables for deterministic or randomized emission.

Order jump tables for deterministic or randomized emission.

BUG=
R=stichnot@chromium.org, jvoung, stichnot

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

[ascull, 2015-08-04 19:28:45]

ascull@google.com changed reviewers:
+ jpp@chromium.org, qining@google.com

[ascull, 2015-08-04 19:28:45]

I used the constant pool randomization flag for jump tables too.

[Jim Stichnoth, 2015-08-04 20:22:52]

https://codereview.chromium.org/1260183008/diff/1/src/IceGlobalContext.cpp
File src/IceGlobalContext.cpp (right):

https://codereview.chromium.org/1260183008/diff/1/src/IceGlobalContext.cpp#ne...
src/IceGlobalContext.cpp:894: return A.getFunctionName() < B.getFunctionName()
|| A.getId() < B.getId();
Is this the right comparison function?

Usually when there is a primary and secondary key, the comparison function looks
like:
  if (A.primary != B.primary)
    return A.primary < B.primary;
  return A.secondary < B.secondary;

[ascull, 2015-08-04 20:36:38]

https://codereview.chromium.org/1260183008/diff/1/src/IceGlobalContext.cpp
File src/IceGlobalContext.cpp (right):

https://codereview.chromium.org/1260183008/diff/1/src/IceGlobalContext.cpp#ne...
src/IceGlobalContext.cpp:894: return A.getFunctionName() < B.getFunctionName()
|| A.getId() < B.getId();
On 2015/08/04 20:22:52, stichnot wrote:
> Is this the right comparison function?
> 
> Usually when there is a primary and secondary key, the comparison function
looks
> like:
>   if (A.primary != B.primary)
>     return A.primary < B.primary;
>   return A.secondary < B.secondary;

Done.

[Jim Stichnoth, 2015-08-04 22:25:56]

lgtm

[ascull, 2015-08-05 00:03:26]

Committed patchset #2 (id:20001) manually as
1eda90a112993eb171b8f60e5470b554e59542af (presubmit successful).