V7_BCODE_DUMP broken - Githubissues

cesanta / v7

Embedded JavaScript engine for C/C++

Other

1.42k stars 177 forks source link

V7_BCODE_DUMP broken #553

Open joeedh opened 8 years ago

joeedh commented 8 years ago

Hi. V7_BCODE_DUMP is broken. The (recent?) addition of immediate strings to the bytecode format has made opcodes that use bcode_decode_lit (OP_PUSH_LIT, OP_SET_VAR, OP_GET_VAR, OP_SAFE_GET_VAR) ambiguous. This broke dump_op (which does not use bcode_decode_lit, and seems to just assume integer operands).

Thanks.

dimonomid commented 8 years ago

Yeah, sorry about that. We actually plan to implement it not as a compile-time option, but as a run-time flag. Hopefully will get it sorted in a few weeks.

goniz commented 8 years ago

Any news on this issue?

dimonomid commented 8 years ago

Sorry but not yet. We still struggle with other things which considered higher priority.