Subzero: Fix some -Winconsistent-missing-override that clang 3.6 warns about.

Subzero: Fix some -Winconsistent-missing-override that clang 3.6 warns about.

I'd like to bump the *trusted* clang compiler also,
since the really old trusted clang compiler seems to crash
if we pair old clang with new libcxx. (So the merge will
probably have to bump the trusted clang compiler to
a newer rev).

BUG= https://code.google.com/p/nativeclient/issues/detail?id=4026
R=stichnot@chromium.org

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

[jvoung (off chromium), 2015-02-03 03:06:14]

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

[Jim Stichnoth, 2015-02-03 05:31:19]

LGTM.

How would you bump the clang compiler version?  Have the makefile use the clang
in LLVM_BIN_PATH?

https://codereview.chromium.org/898693002/diff/1/src/IceOperand.h
File src/IceOperand.h (left):

https://codereview.chromium.org/898693002/diff/1/src/IceOperand.h#oldcode107
src/IceOperand.h:107: using Operand::dump;
Can you also remove this line?

[jvoung (off chromium), 2015-02-03 16:17:56]

Thanks -- To update clang, I would update DEPS like
https://codereview.chromium.org/886943004/, which updates the script which
updates the clang at CLANG_PATH, so the Makefile shouldn't need to change.

https://codereview.chromium.org/898693002/diff/1/src/IceOperand.h
File src/IceOperand.h (left):

https://codereview.chromium.org/898693002/diff/1/src/IceOperand.h#oldcode107
src/IceOperand.h:107: using Operand::dump;
On 2015/02/03 05:31:18, stichnot wrote:
> Can you also remove this line?

Done.

[jvoung (off chromium), 2015-02-03 16:27:50]

Committed patchset #2 (id:20001) manually as
3ce1a991150c1f1954f542d0d998f6f75e89fbc6 (presubmit successful).