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

Simplify ObjectModel, and try to get rid of the StorageAccessors again #375

Open
smarr opened this issue Feb 20, 2021 · 1 comment
Open
Labels
enhancement Improves the implementation with something noteworthy

Comments

@smarr
Copy link
Owner

smarr commented Feb 20, 2021

In SOM-st/TruffleSOM#47, TruffleSOM uses a "better" approach to initialize the unsafe accesses for the object model.
Instead of needing instances of objects, which have their own virtual methods, we can keep all the logic in one place, and only read the offsets from the StorageAnalyzer.

This might be a benefit for interpreter performance, though, unmeasured, and I don't remember what the impact was for introducing the current design in SOMns.

@smarr smarr added the enhancement Improves the implementation with something noteworthy label Feb 20, 2021
@smarr
Copy link
Owner Author

smarr commented Feb 20, 2021

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

No branches or pull requests

1 participant