Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Develop #266

Merged
merged 91 commits into from
Mar 2, 2023
Merged

Develop #266

merged 91 commits into from
Mar 2, 2023

Conversation

krlosMata
Copy link
Contributor

develop to main

laisolizq and others added 30 commits December 20, 2022 12:51
@laisolizq
update tmpvars & expAD counters
85f4a7f
@ignasirv
Fix jmpn
6b07065
@krlosMata
Merge pull request #212 from 0xPolygonHermez/feature/fix-tmpvar
70d4a8a 
update tmpvars & expAD counters
@krlosMata
Merge pull request #214 from 0xPolygonHermez/hotfix/jmpn-typo
308be4d 
Fix jmpn
@krlosMata
read entire bytecode program at once
171827a
@krlosMata
Merge pull request #219 from 0xPolygonHermez/feature/set-get-program
491b7d3 
read entire bytecode program at once
@ignasirv
Fix computeGasSendCall
1c627c4 
Fix saveMem free input


fix
@ignasirv
Performance improvement
2987a82
@zkronos73
remove obsolete TODO comment
4c2763a
@krlosMata
Merge pull request #222 from 0xPolygonHermez/fix-remove-mulpointec-todo
9d59c89 
remove obsolete TODO comment
@ignasirv
Refactor saveMem
56c1d1b
@ignasirv
Fix offset utils + others
8938e62
@ignasirv
Merge pull request #221 from 0xPolygonHermez/hotfix/fix-free-input
bf6b6c5 
Fix computeGasSendCall
@ignasirv
First approach
ad8770b 
Apply jmpnz


More functions


Finished counters tests from utils


Setup gha for counter tests


Arith counters tests
@ignasirv
calldata return code counter tests
6c28750
@zkronos73
fix FNEC_DIV_TWO constant and update test
68c10f1
@ignasirv
Finished tests + readme
e8730ba
@krlosMata
Merge pull request #225 from 0xPolygonHermez/fix-fnecdivtwo-value
14c5ca7 
fix FNEC_DIV_TWO constant and update test
@ignasirv
Add counters at handle gas
7346302
@krlosMata
Merge pull request #223 from 0xPolygonHermez/feature/counter-tests
5c05b98 
Feature/counter tests
@krlosMata
RLP checks & clean code
7303da4
@krlosMata
fix counters includes
a397683
@krlosMata
Merge pull request #224 from 0xPolygonHermez/feature/fixes
8c96a2b 
RLP checks & clean code
@ignasirv
Audit PR fixes
36ab939
@ignasirv
Review fixes
96ed323
@ignasirv
JMP to CALL small improvement
01dae1a
@krlosMata
Merge pull request #226 from 0xPolygonHermez/hotfix/audit-axic-pr
38fe011 
Audit jan fixes
@krlosMata
safe range max counters
7bb5384
@krlosMata
Merge pull request #227 from 0xPolygonHermez/feature/counters-and-opt
af786a8 
Feature/counters and comments
@ignasirv
Boundaries check fixes
de1f009
krlosMata and others added 24 commits February 6, 2023 18:20
@krlosMata
revert when calling a unsupported precompiled
e8e868f
@krlosMata
padding counter check, fix free inputs
4460dd3
@krlosMata
update packages to set commits
592286e
@krlosMata
Merge pull request #247 from 0xPolygonHermez/feature/revert-precompiled
9ea22fe 
Feature/revert precompiled
@krlosMata
fixes identity and not set CTX to 0
5d164c5
@krlosMata
Merge pull request #248 from 0xPolygonHermez/feature/fix-identity
b8a616f 
fixes identity and not set CTX to 0
@krlosMata
ger manager address L2
0e8b566
@krlosMata
Merge pull request #251 from 0xPolygonHermez/feature/address-ger-manager
0dbd921 
ger manager address L2
@krlosMata
ger manager address L2
9840dde
@krlosMata
fix adding bytes to batchHashData
f2d0e0e
@krlosMata
update pacjage remove MAXMEM
fb251a0
@krlosMata
add onUpdateStorage in SLOAD/SSTORE oopcodes
c390711
@krlosMata
Merge pull request #257 from 0xPolygonHermez/feature/update-storage
b0194d3 
add onUpdateStorage in SLOAD/SSTORE oopcodes
@krlosMata
update package versions
9735552
@krlosMata
Merge pull request #252 from 0xPolygonHermez/feature/fix-rlp
70012ac 
Feature/fix rlp
@laisolizq
fix MSTOREX2
980e7ab
@laisolizq
add mstorex test
5d9ad79
@laisolizq
update package tests
3d42356
@krlosMata
fix jmpn, reduntant jump and typos
f76bf6b
@krlosMata
Merge pull request #263 from 0xPolygonHermez/feature/mstorex-fix
37bfbd2 
Feature/mstorex fix
@krlosMata
update to forkid 2 & fix comments
af734fc
@krlosMata
Merge pull request #264 from 0xPolygonHermez/feature/forkid-2
6adc152 
update to forkid 2 & fix comments
@krlosMata
fork.2 update packages
058127b
@krlosMata
Merge pull request #265 from 0xPolygonHermez/feature/update-package-f…
7c2f57e 
…ork.2

fork.2 update packages
@cla-bot cla-bot bot added the cla-signed label Mar 2, 2023
ignasirv
ignasirv approved these changes Mar 2, 2023
View reviewed changes
Copy link
Contributor

@ignasirv ignasirv left a comment

Choose a reason for hiding this comment

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

💯

@krlosMata krlosMata merged commit 2ddeffb into main Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ignasirv ignasirv ignasirv approved these changes

@invocamanman invocamanman Awaiting requested review from invocamanman invocamanman is a code owner

@laisolizq laisolizq Awaiting requested review from laisolizq laisolizq is a code owner

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@krlosMata @ignasirv @laisolizq @zkronos73