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

DO NOT MERGE: Flambda tests #928

Draft
wants to merge 1 commit into
base: next
Choose a base branch
from
Draft

Conversation

Stevendeo
Copy link
Collaborator

@Stevendeo Stevendeo commented Nov 8, 2023

DO NOT MERGE

This branch can be used to test alt-ergo with flambda2. Adresses #906

@Stevendeo Stevendeo changed the title DO NOT MERGE: Flambda compatible DO NOT MERGE: Flambda tests Nov 8, 2023
@bclement-ocp
Copy link
Collaborator

I have tried to follow the instructions to setup a flambda option on the Marvin bot but I get the following error on the make _install step:

File "middle_end/flambda2/parser/flambda_parser.ml", line 5, characters 2-54:
5 |   CamlinternalMenhirLib.StaticVersion.require_20210419
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Error: Unbound value CamlinternalMenhirLib.StaticVersion.require_20210419
Hint: Did you mean require_20220210?

Is that something you encountered?

(Also paging @Gbury )

@bclement-ocp
Copy link
Collaborator

configure: Configuring Flambda backend version 5.1.1+jst

Ah! Probably I should not use the main branch :)

@bclement-ocp
Copy link
Collaborator

Well I tried with last-4.14 but I get a bunch of errors due to unused variables and magic potions. Will ask the flambda team.

@Gbury
Copy link
Collaborator

Gbury commented Nov 28, 2023

Well... the flambda2 repo is currently in the middle of a big backport to bring it up to date with the multicore release, and I think that as of now, installing flambda2 with opam is slightly broken. cc @lthls who might know of a way

@Stevendeo
Copy link
Collaborator Author

Is that something you encountered?

It may have been one of the issues I encountered, but iirc updating the menhir version was enough for this one. I had some issues with the compilation of dune actually...
Anyway, we said we would give up with flambda2, I'm trying with flambda

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants