Subzero: Fix the g++ build (e.g. Windows).

Subzero: Fix the g++ build (e.g. Windows).

BUG= https://code.google.com/p/nativeclient/issues/detail?id=4006
R=kschimpf@google.com

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

[Jim Stichnoth, 2014-12-04 21:53:54]

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

[Karl, 2014-12-04 21:58:16]

LGTM after nits fixed.

https://codereview.chromium.org/773583004/diff/1/src/IceOperand.cpp
File src/IceOperand.cpp (right):

https://codereview.chromium.org/773583004/diff/1/src/IceOperand.cpp#newcode255
src/IceOperand.cpp:255: }
nullptr?

https://codereview.chromium.org/773583004/diff/1/src/IceOperand.cpp#newcode269
src/IceOperand.cpp:269: return NULL;
nullptr?

[Jim Stichnoth, 2014-12-04 22:09:16]

https://codereview.chromium.org/773583004/diff/1/src/IceOperand.cpp
File src/IceOperand.cpp (right):

https://codereview.chromium.org/773583004/diff/1/src/IceOperand.cpp#newcode255
src/IceOperand.cpp:255: }
On 2014/12/04 21:58:15, Karl wrote:
> nullptr?

I actually meant to keep it NULL to be consistent with the rest of the file.  I
was meaning to do a nullptr pass over the entire code base at some point.

[Jim Stichnoth, 2014-12-04 22:09:26]

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