Skip to content

Reassign opcodes to proposed instructions#665

Merged
chfast merged 1 commit intomasterfrom
opcodes
Jun 21, 2023
Merged

Reassign opcodes to proposed instructions#665
chfast merged 1 commit intomasterfrom
opcodes

Conversation

@chfast
Copy link
Copy Markdown
Member

@chfast chfast commented Jun 18, 2023

Follows the opcode re-assignment of proposed instructions specified in ethereum/execution-specs#764.

@chfast chfast requested review from axic, gumb0 and rodiazet June 18, 2023 22:16
@chfast chfast force-pushed the opcodes branch 2 times, most recently from 09d8d49 to 6c24499 Compare June 19, 2023 09:19
@chfast chfast changed the base branch from master to eof-delegatecall June 19, 2023 09:19
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 19, 2023

Codecov Report

Merging #665 (d98e9d9) into master (94a948f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #665   +/-   ##
=======================================
  Coverage   97.40%   97.40%           
=======================================
  Files          81       81           
  Lines        8005     8008    +3     
=======================================
+ Hits         7797     7800    +3     
  Misses        208      208           
Flag Coverage Δ
blockchaintests 62.95% <ø> (ø)
statetests 74.05% <ø> (ø)
unittests 94.98% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
test/unittests/eof_validation_test.cpp 100.00% <100.00%> (ø)
test/unittests/evm_eof_function_test.cpp 100.00% <100.00%> (ø)

Base automatically changed from eof-delegatecall to master June 19, 2023 14:14
Follows the opcode re-assignment of proposed instructions specified in
ethereum/execution-specs#764.
@chfast chfast merged commit 9399750 into master Jun 21, 2023
@chfast chfast deleted the opcodes branch June 21, 2023 07:26
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.

2 participants