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.

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

Update Truffle to 0.25 #132

Merged
merged 17 commits into from
Apr 23, 2017
Merged

Update Truffle to 0.25 #132

merged 17 commits into from
Apr 23, 2017

Conversation

smarr
Copy link
Owner

@smarr smarr commented Apr 23, 2017

This is a major change to support the latest Truffle version.

To make it work properly, I am removing much static state, but not yet all.
First step is to make all runtime classes part of the context, which is realized with the VM class.
VM class and SomLanguage instance have a 1-to-1 mapping. At the moment, I don't anticipate to need more than that.

@smarr smarr added bug Fixes an issue, incorrect implementation enhancement Improves the implementation with something noteworthy labels Apr 23, 2017
@smarr smarr added this to the v0.4.0 milestone Apr 23, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.011% when pulling 403b0c9 on update-truffle-0.25 into cf0b701 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.011% when pulling 3e68c7e on update-truffle-0.25 into cf0b701 on master.

- add source section to root node subclasses

Signed-off-by: Stefan Marr <git@stefan-marr.de>
@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.011% when pulling 9210438 on update-truffle-0.25 into dbcd883 on master.

smarr added 16 commits April 23, 2017 19:13
Signed-off-by: Stefan Marr <git@stefan-marr.de>
- first step to remove all static state
- might need further refactorings
- might need futher simplification: do I really need SomLanguage and VM?

Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
We use VM as execution context for the PolyglotEngine, and remove much of the static fields to access it to make the VM more reentrant (possibly allow multiple instance in the same Java process).

- pass VM explicitly where required
- actors needs VM for debugging
- message sending needs actorPool access
- move all fork/join pools to VM
- move UncaughtExceptions to separate file

Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
- add IS_NULL support
- use hack to get to VM object
- remove dead code

Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
This is already handled in UncaughtExceptions

Signed-off-by: Stefan Marr <git@stefan-marr.de>
…ymore

Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr merged commit 8004a11 into master Apr 23, 2017
@smarr smarr deleted the update-truffle-0.25 branch April 23, 2017 17:15
@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.011% when pulling 1584166 on update-truffle-0.25 into dbcd883 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes an issue, incorrect implementation enhancement Improves the implementation with something noteworthy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants