pharo-project / pharo-vm

This is the VM used by Pharo
http://pharo.org
Other
110 stars 67 forks source link

Fix Slang block inlining #430

Open PalumboN opened 2 years ago

PalumboN commented 2 years ago

This method https://github.com/pharo-project/pharo-vm/blob/81891cf884f8b42e694bc178f63239df2aae43f8/smalltalksrc/VMMaker/InterpreterPrimitives.class.st#L5039-L5052 cannot be translated.

But this one yes: https://github.com/pharo-project/pharo-vm/blob/81891cf884f8b42e694bc178f63239df2aae43f8/smalltalksrc/VMMaker/InterpreterPrimitives.class.st#L4999-L5010

PalumboN commented 2 years ago

Work started at https://github.com/PalumboN/opensmalltalk-vm/tree/fix-slang-block-inline

GitHub
GitHub - PalumboN/opensmalltalk-vm at fix-slang-block-inline
This is the VM used by Pharo. Contribute to PalumboN/opensmalltalk-vm development by creating an account on GitHub.
guillep commented 2 years ago

The test that tests this case is in https://github.com/pharo-project/pharo-vm/pull/431#discussion_r894578447