Assert in serializer_get_opcode

[seanshpark]

When dump op codes with iotjs, I get below assert.

ICE: Assertion 'oc < bytecode_data.opcodes_count' failed at /.../iotjs/deps/jerry/jerry-core/parser/js/serializer.cpp(serializer_get_opcode)

how to reproduce

1. build iotjs

./tools/build.py --jerry-memstats --no-checktest

branch is c87ab8db64fc0f40a25b941510b3a119c831932a
2) run test

cd test/run_pass
../../build/(core)/debug/iotjs/iotjs test_net.js

where (core) depends on your build machine.
oc is 66 and bytecode_data.opcodes_count is 37 when assert failed

[ruben-ayrapetyan]

Failure is caused by usage of incorrect condition (bytecode_data.opcodes != NULL) for determining printed byte-code instruction location. While generation of a byte-code array is in process, instructions are located in scopes tree, then the instructions are removed from scope tree and byte-code array is added to bytecode_data.opcodes list. So, bytecode_data.opcodes != NULL condition shows whether there is at least one already generated byte-code array.

Actually, caller of pp_op_meta (byte-code print helper) should specify whether to print byte-code from current scope, or from some of already generated byte-code arrays.

Pull request with fix: #227

[seanshpark]

Thank you!