Skip to content
This repository has been archived by the owner on Jan 5, 2019. It is now read-only.

fixes for stCallCodes/callcall_00 #246

Closed
wants to merge 13 commits into from
Closed

fixes for stCallCodes/callcall_00 #246

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
15ad214
use latest cpp-ethereum/hera
cdetrio May 16, 2018
7c8190c
try develop
cdetrio May 16, 2018
095e532
try older commit
cdetrio May 17, 2018
d84ae51
try develop 2
cdetrio May 17, 2018
5a03555
update EEI params for CALL
cdetrio May 17, 2018
c039bd0
dump state in cpp-ethereum
cdetrio May 17, 2018
5c5d5cb
fix endianness for i128 and address outputs
cdetrio May 17, 2018
1cb96f0
use hera branch code-size
cdetrio May 17, 2018
f9e6a30
CALL with 2300 gas subsidy
cdetrio May 17, 2018
27aab0e
let Hera handle the 2300 gas subsidy for CALL
cdetrio May 17, 2018
225fde0
delete 6th and 7th CALL params from stack
cdetrio May 18, 2018
b4fd098
unpin hera
cdetrio May 18, 2018
6a797ef
flip CALL result from EEI/POSIX convention to EVM convention
cdetrio May 18, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion wasm/generateInterface.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -370,16 +370,27 @@ function generateManifest (interfaceManifest, opts) {
call += '(get_local $callback)'
}

if (opcode === 'CALL') {
call =
`(i64.store
(i32.add (get_global $sp) (i32.const ${spOffset * 32}))
(i64.extend_u/i32
(i32.xor (i32.const 1) ${call}) ;; flip CALL result from EVM convention to POSIX convention, 0 -> 1, 1 -> 0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not enough, because it could return 2, which is xor'd into 3.

)))`
} else {
call =
`(i64.store
(i32.add (get_global $sp) (i32.const ${spOffset * 32}))
(i64.extend_u/i32
${call})))
${call})))`
}

call += `
;; zero out mem
(i64.store (i32.add (get_global $sp) (i32.const ${spOffset * 32 + 8 * 3})) (i64.const 0))
(i64.store (i32.add (get_global $sp) (i32.const ${spOffset * 32 + 8 * 2})) (i64.const 0))
(i64.store (i32.add (get_global $sp) (i32.const ${spOffset * 32 + 8})) (i64.const 0))`

} else if (output === 'i64') {
if (useAsyncAPI && op.async) {
call += '(get_local $callback)'
Expand Down
Loading