Subzero: Fix an incorrect StringPool assert.

Subzero: Fix an incorrect StringPool assert.

BUG= none
R=jpp@chromium.org

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

[Jim Stichnoth, 2016-04-07 12:20:44]

Description was changed from

==========
Subzero: Fix an incorrect StringPool assert.

BUG= none
==========

to

==========
Subzero: Fix an incorrect StringPool assert.

BUG= none
==========

[Jim Stichnoth, 2016-04-07 12:20:45]

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

[Jim Stichnoth, 2016-04-07 12:20:50]

https://codereview.chromium.org/1866993002/diff/1/src/IceStringPool.h
File src/IceStringPool.h (right):

https://codereview.chromium.org/1866993002/diff/1/src/IceStringPool.h#newcode115
src/IceStringPool.h:115: if (ThisHasString && OtherHasString)
Here we test for both being true.

https://codereview.chromium.org/1866993002/diff/1/src/IceStringPool.h#newcode118
src/IceStringPool.h:118: if (!ThisHasString && !OtherHasString)
Here we test for both being false.

https://codereview.chromium.org/1866993002/diff/1/src/IceStringPool.h#newcode121
src/IceStringPool.h:121: assert(ThisHasString != OtherHasString);
This is just meant to validate that one is true and the other is false.

[John, 2016-04-07 14:03:35]

lgtm

[Jim Stichnoth, 2016-04-07 14:17:39]

Description was changed from

==========
Subzero: Fix an incorrect StringPool assert.

BUG= none
==========

to

==========
Subzero: Fix an incorrect StringPool assert.

BUG= none
R=jpp@chromium.org

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

[Jim Stichnoth, 2016-04-07 14:17:40]

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