-
Notifications
You must be signed in to change notification settings - Fork 781
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[Constantinople] EIP 1014 CREATE2 Opcode #356
Comments
@holgerd77 just out of curiosity do you plan a release of the intermediate state (e.g. 2.4.1) or only a release once everything is done. I would love to create a PR on ganache-core to enable constantinople testing (via a flag) and would love to have create2 in there :D |
@rmeissner Hmm, I have a strong tendency to wait until things are completed and then do a bigger release, sorry, please give this another 2-3 weeks. The StateManager changes by @mattdean-digicatapult are coming to a close with I think one PR + the API documentation missing, also Constantinople is close to the finish line. Currently hanging is actually only the EXTCODEHASH implementation #326, with the tests not yet all written on the Once these things come together I am thinking of eventually do even a |
I will take a look at #326 but this might have to wait a little :) |
Hey guys what's the update here? I'm still getting |
@rmeissner @holgerd77 assembly {
proxy := create2(0, add(initcode, 0x20), mload(initcode), 0)
} |
@rstormsf would be nice to see an example, since we have some test with create2 and ganache running for the Gnosis Safe (see https://github.com/gnosis/safe-contracts/blob/development/contracts/proxies/ProxyFactory.sol#L50, https://github.com/gnosis/safe-contracts/blob/development/test/gnosisSafePersonalEditionCreation2.js#L47 and https://travis-ci.org/gnosis/safe-contracts/builds/506170858#L750) |
I'll try to re-use this new |
@rmeissner False positive. https://www.youtube.com/watch?v=LcDIX-ZQksU&feature=youtu.be @snario take a look. it's working with ganache-cli |
EIP: EIP 1014
Work in progress: PR #329
Test Situation:
Many tests implemented
Tests | Filler
The text was updated successfully, but these errors were encountered: