[NaCl SDK] Include Ragel ncval in SDK for all validation.

[NaCl SDK] Include Ragel ncval in SDK for all validation.

BUG=226406
R=noelallen@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203767

[noelallen1, 2013-06-03 18:28:13]

lgtm

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_sdk.py:514: GypNinjaBuild('ia32',
gyp_py, all_gyp, 'ncval_new', out_dir)
What are we doing for PNaCl?

[binji, 2013-06-03 18:32:33]

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_sdk.py (right):

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_sdk.py:514: GypNinjaBuild('ia32',
gyp_py, all_gyp, 'ncval_new', out_dir)
On 2013/06/03 18:28:13, noelallen1 wrote:
> What are we doing for PNaCl?

pnacl has its own bitcode checker (included with the pnacl toolchain), but
according to dschuff doesn't allow inline asm, so doesn't need ncval.

We could pre-translate the pnacl code and validate that, but we can use the
existing translator for that case.

[Sam Clegg, 2013-06-03 18:41:52]

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/sdk_files.list (right):

https://codereview.chromium.org/16140015/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/sdk_files.list:982: [win]tools/ncval.exe
This file might be more manageable if it had a {EXE_EXT} variable?

But thats a different change I guess.

[binji, 2013-06-03 20:55:48]

Committed patchset #1 manually as r203767 (presubmit successful).