Skip to content

EIP-1231: Reduce CALL cost for precompiles#1231

Closed
mattdf wants to merge 4 commits into
ethereum:masterfrom
mattdf:eip-reduce-call-cost-precompiles
Closed

EIP-1231: Reduce CALL cost for precompiles#1231
mattdf wants to merge 4 commits into
ethereum:masterfrom
mattdf:eip-reduce-call-cost-precompiles

Conversation

@mattdf
Copy link
Copy Markdown
Member

@mattdf mattdf commented Jul 18, 2018

EIP to reduce call cost to precompiles to pre-spurious dragon HF values. Precompiles don't have read overhead and as such should never have been subject to the penalty.

Comment thread EIPS/eip-1231.md

| Opcode name | Opcodes | Current Gas Cost | Updated Gas Cost |
| ------------------------------ | --------- | ----------------------------- | ------------------- |
| `CALL` (to precompiles) | `0xf1` | 700<sup>[1]</sup> | 40 |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is the table necessary, since it's the same for all types of call?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think it can be removed..

@axic
Copy link
Copy Markdown
Member

axic commented Aug 31, 2018

Should this be merged with https://ethereum-magicians.org/t/eip-1109-remove-call-costs-for-precompiled-contracts/447/8 or makes sense to keep track of both proposals?

@axic
Copy link
Copy Markdown
Member

axic commented May 19, 2019

I think this could be closed after #2046 is meged.

@axic
Copy link
Copy Markdown
Member

axic commented May 23, 2019

@mattdf since #2046 was merged, are you OK closing this?

@axic axic added EIP labels Jun 20, 2019
@axic
Copy link
Copy Markdown
Member

axic commented Jun 24, 2019

Since the author has not responded for a year and there is a successor EIP proposal, I suggest this is closed.

@axic
Copy link
Copy Markdown
Member

axic commented Jun 27, 2019

Since conversation is already happening around #1109 and #2046 I'm closing this to avoid confusion. @mattdf if you think this should be kept open please respond so it can be reopened.

@axic axic closed this Jun 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants