-
Notifications
You must be signed in to change notification settings - Fork 384
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.
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
fix(gnovm): follow up works on loop scope(#2429) #2440
Merged
jaekwon
merged 53 commits into
gnolang:dev/jae/loopescape
from
ltzmaxwell:dev/maxwell/dev/jae/loopscope
Jul 25, 2024
Merged
Changes from 48 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
c6411d6
add test
ltzmaxwell 99b2a1e
alloc heapItem for ref
ltzmaxwell e7761ea
Merge remote-tracking branch 'upstream/dev/jae/loopescape' into 2/max…
ltzmaxwell 26580ab
Merge remote-tracking branch 'upstream/dev/jae/loopescape' into 2/max…
ltzmaxwell 0d2bbb1
test ref
ltzmaxwell e7b4601
Merge remote-tracking branch 'upstream/dev/jae/loopescape' into 2/max…
ltzmaxwell 59ace4e
add forloop tests
ltzmaxwell 348b3f6
revert
ltzmaxwell 256c289
fixup
ltzmaxwell c2f0d56
clean
ltzmaxwell 9b50f5a
fixup
ltzmaxwell bc93048
fixup
ltzmaxwell 17e51d1
fixup
ltzmaxwell bb83812
fixup
ltzmaxwell e6ec47e
fixup
ltzmaxwell 2332e31
Merge remote-tracking branch 'upstream/dev/jae/loopescape' into 2/max…
ltzmaxwell e0bd54f
Merge remote-tracking branch 'upstream/dev/jae/loopescape' into 2/max…
ltzmaxwell cd0f1e8
closure works
ltzmaxwell ad68fb0
fix copy
ltzmaxwell 69766a0
heapItem work, no loopVar
ltzmaxwell e20045c
save
ltzmaxwell cb9fe50
debug
ltzmaxwell 81e0f00
make gotoloop pass
ltzmaxwell 6843928
alloc per loop on for
ltzmaxwell 20f04f5
fixup
ltzmaxwell 9645142
range work
ltzmaxwell 046f07e
range key and value
ltzmaxwell cfd1cbc
intersection goto loop
ltzmaxwell 9a0fc7f
fix for loop && re-orgnize test files
ltzmaxwell 51574f6
fixup
ltzmaxwell 694cedf
save with logic attempt for optimization
ltzmaxwell 2840f08
test pass
ltzmaxwell 1e088f9
clean
ltzmaxwell b8a3f22
loopvar implicit decl in runtime
ltzmaxwell 0ad5132
Revert "loopvar implicit decl in runtime"
ltzmaxwell 6cafbb6
more tests
ltzmaxwell 2c2bb15
fixup && rename files
ltzmaxwell 48be792
tune test files
ltzmaxwell a0ef113
clean
ltzmaxwell 7da572e
fix check locally defined
ltzmaxwell e08ad98
solve todo
ltzmaxwell b174eea
fixup
ltzmaxwell 67ef081
fix tests
ltzmaxwell 0e49090
correct init order for func with unnamed return, all tests passed
ltzmaxwell 1d629f7
fixup
ltzmaxwell 7e83eb1
tune
ltzmaxwell b24a779
add preprocessed infos
ltzmaxwell f253108
add TODO
ltzmaxwell 20e64f9
delete attr
ltzmaxwell faecc70
fixup
ltzmaxwell 921b293
fixup
ltzmaxwell fc9eaa1
fixup
ltzmaxwell d24a3d5
Merge branch 'dev/jae/loopescape' into dev/maxwell/dev/jae/loopscope
jaekwon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,9 +24,9 @@ func main() { | |
|
||
// Realm: | ||
// switchrealm["gno.land/r/test"] | ||
// u[a8ada09dee16d791fd406d629fe29bb0ed084a30:4]={ | ||
// u[a8ada09dee16d791fd406d629fe29bb0ed084a30:3]={ | ||
// "ObjectInfo": { | ||
// "ID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:4", | ||
// "ID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:3", | ||
// "ModTime": "7", | ||
// "OwnerID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:7", | ||
// "RefCount": "1" | ||
|
@@ -38,8 +38,8 @@ func main() { | |
// }, | ||
// "V": { | ||
// "@type": "/gno.RefValue", | ||
// "Hash": "627e8e517e7ae5db0f3b753e2a32b607989198b6", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:5" | ||
// "Hash": "54bef18ac963dd67ad0634eac6b75a2c4a733b97", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:4" | ||
// } | ||
// } | ||
// } | ||
|
@@ -162,8 +162,8 @@ func main() { | |
// "@type": "/gno.PointerValue", | ||
// "Base": { | ||
// "@type": "/gno.RefValue", | ||
// "Hash": "6da365f0d6cacbcdf53cd5a4b125803cddce08c2", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:4" | ||
// "Hash": "ce8be5b209e8f322bf1b15c854275064d5645741", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:3" | ||
// }, | ||
// "Index": "0", | ||
// "TV": null | ||
|
@@ -210,7 +210,7 @@ func main() { | |
// }, | ||
// "V": { | ||
// "@type": "/gno.RefValue", | ||
// "Hash": "ff1a50d8489090af37a2c7766d659f0d717939b5", | ||
// "Hash": "da7583511e84b52e256a57deabe5b390d875407d", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:7" | ||
// } | ||
// } | ||
|
@@ -237,6 +237,25 @@ func main() { | |
// "Values": [ | ||
// { | ||
// "T": { | ||
// "@type": "/gno.PointerType", | ||
// "Elt": { | ||
// "@type": "/gno.RefType", | ||
// "ID": "gno.land/p/demo/avl.Node" | ||
// } | ||
// }, | ||
// "V": { | ||
// "@type": "/gno.PointerValue", | ||
// "Base": { | ||
// "@type": "/gno.RefValue", | ||
// "Hash": "e925bb6bf884ece9620dd27f831dc510b5bc0e55", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:6" | ||
// }, | ||
// "Index": "0", | ||
// "TV": null | ||
// } | ||
// }, | ||
// { | ||
// "T": { | ||
// "@type": "/gno.FuncType", | ||
// "Params": [], | ||
// "Results": [] | ||
|
@@ -246,11 +265,11 @@ func main() { | |
// "Closure": { | ||
// "@type": "/gno.RefValue", | ||
// "Escaped": true, | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:3" | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:5" | ||
// }, | ||
// "FileName": "main.gno", | ||
// "IsMethod": false, | ||
// "Name": "init.0", | ||
// "Name": "init.1", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is this introduced by the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes. |
||
// "NativeName": "", | ||
// "NativePkg": "", | ||
// "PkgPath": "gno.land/r/test", | ||
|
@@ -282,7 +301,7 @@ func main() { | |
// "Closure": { | ||
// "@type": "/gno.RefValue", | ||
// "Escaped": true, | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:3" | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:5" | ||
// }, | ||
// "FileName": "main.gno", | ||
// "IsMethod": false, | ||
|
@@ -306,25 +325,6 @@ func main() { | |
// "Results": [] | ||
// } | ||
// } | ||
// }, | ||
// { | ||
// "T": { | ||
// "@type": "/gno.PointerType", | ||
// "Elt": { | ||
// "@type": "/gno.RefType", | ||
// "ID": "gno.land/p/demo/avl.Node" | ||
// } | ||
// }, | ||
// "V": { | ||
// "@type": "/gno.PointerValue", | ||
// "Base": { | ||
// "@type": "/gno.RefValue", | ||
// "Hash": "ae86874f9b47fa5e64c30b3e92e9d07f2ec967a4", | ||
// "ObjectID": "a8ada09dee16d791fd406d629fe29bb0ed084a30:6" | ||
// }, | ||
// "Index": "0", | ||
// "TV": null | ||
// } | ||
// } | ||
// ] | ||
// } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
package var is inited firstly in
initStaticBlocks
, the the order changes.