Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* Brought issue #152 into the repo as a draft EIP.
Thanks @tjade273!
* Make the draft EIP consistent with the template
Also added myself as an author
* Break backwards compatibility into its own section
* Added notes about the in-progress implementation
Should have a working geth precompile and initial benchmarks shortly
* Specify EIP-2046 as a requirement
While 2046's cheaper precompile contract calls aren't a requirement for
this EIP's implementation, shipping this precompile without EIP-2046
would make the F function expensive for some of the motivating usecases.
* Don't use ABI encoding for precompile
Replace the existing ABI encoding interface to the BLAKE2b `F`
precompile with a loosely pack struct that's `staticcall`-friendly.
H/t to @pdyraga for putting together the interface!
* Add @pdyraga to the EIP authors.
* Remove less relevant EIP rationale
Let's not relitigate precompiles, WASM, etc in thie EIP :)
* Use 0x09 as the precompile address
If a conflicting EIP is moving forward the EIP editor can assign a new
address
* Choosing an EIP number
Contributing docs suggest EIPs be named `eip-draft-with-a-title` until
an editor has been assigned, but discussing this work off-platform
without a number is a problem.
Assigning 152 as the issue number where the `F` precompile was
originally raised (https://github.com/ethereum/EIPs/issues/152)
* Add a missing colon
Thanks @axic 🙌
* Spelling updates
* Add @MadeOfTin to the authors list
* Prefer the original issue for discussion
* Clarify the precompile's initial implementation
* Make the precompile return value clear
* Clean up references wording
* More rationale around this BLAKE2b approach
* Fix a couple misspellings
* Updated the interface for F precompile
- F precompile accepts now `abi.encodePacked` parameters taking
exactly 213 bytes. This is safer and does not require left-padding data
- `rounds` parameter is now the first one as the gas cost depends only
on this parameter
* Updated gas cost section proposing GFROUND=1
* Detailed benchmarks moved to appendix section
* Benchmark stats are compared against ecRecover as a baseline
* Clarification: f parameter is true if it is nonzero
This rule is compatible with Solidity for boolean.
* Avoid referring to abi.encodePacked
The specification should not be Solidity-specific. Instead of
referring to abi.encodePacked we now just say "tightly encoded".
* Fixed incorrect link
"specified in the RFC" linked to the geth PR for F precompile
instead of linking to the BLAKE2b RFC.
* Shortened the description about when parameter f is considered as true
* Minor grammar improvement
* Updated information about endianness of F precompile inputs
BLAKE2b is consistently little-endian. abi.encodePacked encodes each
of its arguments in big-endian order. We need to be clear which parameters
should go as little-endian (h, m, t) and which parameters should go as
big-endian (rounds, f).
* Strict validation of f parameter
* Initial test vectors for F precompile
Test cases covered:
- input length too short
- input length too long
- malformed f flag encoding
- correct input, test vector from BLAKE2b RFC, Appendix A
* Test vector for a non-final round
* Test vector for the maximum number of rounds
* Test vector for a single round
* Added test vector for empty input
* The final block indicator (8-bit word) does not have endianness
* Clarify state vector encoding does not change in the output
* Put too short input test vector next to empty input test vector
* Added test vector for zero-rounds BLAKE2b case
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* Add common metrics EIP.
* Fix spelling error.
* Assign EIP number, added more information on what Prometheus does.
* Add link to prometheus website for further info.
* Fix link.
* Fix discussion link.
* Switch to standards track - interface instead of informational.
* Add motivation.
* Link to PR implementing EIP in Pantheon.
* Move EIP-2159 to last call.
* Add review period end
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* Fix Markdown rendering within summary/detail
* Add vendor to .gitignore
* Remove duplicate github-pages entry from Gemfile
* Require github-pages 198
This brings in fixes to kramdown.
* Remove explicit jekyll version as github-pages brings it in as a dependency
* Update bundler dependency tree
* Fake bundler version
* use Ruby 2.3.0
* Set sane defaults for kramdown
* Fix links after kramdown update
* Remove <details> formatting from EIP-1474 as it is not working with embedded markdown
* Revert "Fix Markdown rendering within summary/detail"
* Fix email in EIP-1812
* Remove <details> formatting from EIP-1620 as it is not working with embedded markdown
* EIP 2124: Zero RTT netsplit on chain mismatch
* Assign number to EIP 2124
* Address review comments on EIP 2124.
* Rename EIP 2124 to better reflect content
* Fix cardinality
* Fix email address in 2124
* Update fork identifier EIP based on review discussions.
* Fixup EIP name for the fork identifier.
* Remove stale mention of ENR from forkid EIP.
* Add RLP encoding test cases for forkid EIP.
* Add Felix too to the forkid EIP author list.
* Get rid of mentioning the MTU stuffin the forkid EIP.
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* EIP-1344: Move to Accepted after Last Call Review
- One additional reviewer added comment
- No technical updates suggested
* EIP-1344: Remove review period
* EIP-1344: Update status to Final
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* Update eip-1.md
* Update EIPS/eip-1.md
Co-Authored-By: timbeiko <t.beiko23@gmail.com>
* Update EIP-1 based on reviewer feedback
* Remove qualifiers
Based on @carver's comments.
* Rephrase based on review feedback
Rephrase based on review feedback
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* Add ERC - dType Storage Extension - Decentralized Type System for EVM
* Rename eip-dtype_storage_extension.md to eip-2157.md
* Fix spelling mistakes
* Add EIP number 2157 to document
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* EIP-????: Add OpenRPC Service Discovery To JSON-RPC Services
* fix:
* fix(openrpc): add openrpc.json to eip assets
* fix(openrpc): reference eip asset
* Update EIPS/eip-1901.md - document link
* Delete openrpc.json
* Update eip-1901: move link to spec to top
* fix(1901): preamble + author github usernames
* fix(1901): add copyright info
* fix(1901): add link to eth json rpc spec
* fix(1901): type and category in preamble + spelling
* fix(1901): motivation section and use vendored assets
* fix(1901): add link to multi-geth OpenRPC discovery
* fix(1901): change ordering + small fixes
* fix(1901): typo
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
Hi, I'm a bot! This change was automatically merged because:
- It only modifies existing Draft or Last Call EIP(s)
- The PR was approved or written by at least one author of each modified EIP
- The build is passing
* EIP-1895: Better constants and address range description
Address @Arachnid's comments about presenting constants and how an address value is limited.
* EIP-1985: Fix a typo
* EIP-1985: Use "instructions" instead of "opcodes"
Adrian Sutton
Matt Luongo
MrChico
Gabriel Barros
Erik Marks
Martin Holst Swende
Tim Beiko
Juliano Rizzo
Alex Beregszaszi
Péter Szilágyi
Filip Š
Adam Schmideg
Bryant Eisenbach
William Entriken
James Hancock
Pelle Braendgaard
Tom Brand
Alan Lu
xinbenlv
Loredana Cirstea
Witek
Greg Colvin
Frank.R.Wu
Micah Zoltu
Antonio Salazar Cardozo
Shane
Adrian Sutton
5chdn
James Sangalli
Talha Cross
Paweł Bylica
Lucas Cullen