Require that the module symbol table can't appear after function.

Require that the module symbol table can't appear after function.

The bitcode parser assumes that the module symbol table, if it exists,
must appear before the first function
block (i.e. defininition). However, the parser did not check this
constraint. This patch fixes that omission.

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

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

[Karl, 2015-09-22 18:34:49]

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

[Jim Stichnoth, 2015-09-22 20:06:06]

lgtm

https://codereview.chromium.org/1359923002/diff/1/src/PNaClTranslator.cpp
File src/PNaClTranslator.cpp (right):

https://codereview.chromium.org/1359923002/diff/1/src/PNaClTranslator.cpp#new...
src/PNaClTranslator.cpp:3161: // If we have already processed a function block
(i.e.  we have already
single space after i.e.

[Karl, 2015-09-22 20:14:58]

Committed patchset #2 (id:20001) manually as
28fc2d7dce3693b40d2a99d84d4147274d66784c (presubmit successful).

[Karl, 2015-09-22 20:15:15]

https://codereview.chromium.org/1359923002/diff/1/src/PNaClTranslator.cpp
File src/PNaClTranslator.cpp (right):

https://codereview.chromium.org/1359923002/diff/1/src/PNaClTranslator.cpp#new...
src/PNaClTranslator.cpp:3161: // If we have already processed a function block
(i.e.  we have already
On 2015/09/22 20:06:06, stichnot wrote:
> single space after i.e.

Done.