alexdovzhanyn / AlchemyVM

WebAssembly Virtual Machine Built In Elixir
MIT License
188 stars 5 forks source link

Trace output #26

Closed fantypants closed 5 years ago

fantypants commented 5 years ago

Trace-Output:

alexdovzhanyn commented 5 years ago

This build is failing: https://travis-ci.org/ElixiumNetwork/WaspVM/builds/482724486#L480, its not compiling

fantypants commented 5 years ago

Fixed Compilation Issues, It had left git tags again and rearranged some stuff.

alexdovzhanyn commented 5 years ago

We're almost there with this PR. There are still some things that seem to have slipped through the cracks -- upon testing the PR it seems like the trace option crashes the VM when using if statements:

** (FunctionClauseError) no function clause matching in WaspVM.Executor.create_entry/1
(wasp_vm) lib/execution/executor.ex:1082: WaspVM.Executor.create_entry({:if, :no_res, :none, 8})

This won't work because there isn't a matching create_entry function clause. We need to add a case to match this (and we need a default case that emits a warning to IO and then skips logging it to the file.

alexdovzhanyn commented 5 years ago

We're almost there with this PR. There are still some things that seem to have slipped through the cracks -- upon testing the PR it seems like the trace option crashes the VM when using if statements:

** (FunctionClauseError) no function clause matching in WaspVM.Executor.create_entry/1
(wasp_vm) lib/execution/executor.ex:1082: WaspVM.Executor.create_entry({:if, :no_res, :none, 8})

This won't work because there isn't a matching create_entry function clause. We need to add a case to match this (and we need a default case that emits a warning to IO and then skips logging it to the file.

I've fixed this in the latest commit to this PR