Fix the build for clang 3.8 and MINIMAL=1.

Fix the build for clang 3.8 and MINIMAL=1.

3.8 has stricter warnings which LLVM doesn't pass so we turn them off when
including the offending headers.

BUG=
R=stichnot@chromium.org

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

[ascull, 2015-09-09 18:44:40]

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

[ascull, 2015-09-09 18:44:40]

Thanks to jpp@ for the fix.

[Jim Stichnoth, 2015-09-09 18:52:00]

lgtm

https://codereview.chromium.org/1318883007/diff/1/src/IceTargetLoweringARM32.cpp
File src/IceTargetLoweringARM32.cpp (left):

https://codereview.chromium.org/1318883007/diff/1/src/IceTargetLoweringARM32....
src/IceTargetLoweringARM32.cpp:59: const size_t TableIcmp32Size =
llvm::array_lengthof(TableIcmp32);
I assume the problem was an unused variable in NODEBUG mode?

Would constexpr have fixed it?

[ascull, 2015-09-09 18:55:06]

https://codereview.chromium.org/1318883007/diff/1/src/IceTargetLoweringARM32.cpp
File src/IceTargetLoweringARM32.cpp (left):

https://codereview.chromium.org/1318883007/diff/1/src/IceTargetLoweringARM32....
src/IceTargetLoweringARM32.cpp:59: const size_t TableIcmp32Size =
llvm::array_lengthof(TableIcmp32);
On 2015/09/09 18:52:00, stichnot wrote:
> I assume the problem was an unused variable in NODEBUG mode?
> 
> Would constexpr have fixed it?

I gave it a go but it still complained it was unused.

[ascull, 2015-09-09 18:56:14]

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