Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

WIP Move++ move-mv-llvm-compiler with Aptos FE #274

Closed

Conversation

jcivlin
Copy link
Collaborator

@jcivlin jcivlin commented Aug 11, 2023

This includes compilation of Aptos added move stdlib (find it under move-mv-llvm-compiler/tests/extern directory).

Code generation may have some problems, but FE passes, we can compile lambda, iterator, inline.

This does not support our old compilation path from the bytecode, since Aptos removed the old api.
It possibly may be restored - Aptos has some new building interfaces. I'm not sure we will need this, but I want to store in the repo as much of working code as I have.

Do not expect our tests "from bytecode" will work. But stdlib-tests does!

@ksolana
Copy link
Collaborator

ksolana commented Aug 12, 2023

You are in for a treat! see: https://github.com/move-language/move/pull/1066/files

@brson
Copy link
Collaborator

brson commented Aug 24, 2023

This is difficult to evaluate as is, since it is a huge diff that (I think) is copying aptos code into our non-aptos-based tree, creating a hybrid of mainline move and aptos move.

If we are going to switch to an aptos base I might do it like

  • first rebase our entire tree off move/main
  • then squash our entire diff into one patch
  • then rebase that one patch off the move/aptos-main branch

@dmakarov
Copy link
Collaborator

Closing due to over 6 months of being inactive. Please, reopen for new development.

@dmakarov dmakarov closed this Feb 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants