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

Rename builtin natural to nat and boolean to bool #1588

Merged
merged 4 commits into from
Oct 14, 2022
Merged

Conversation

paulcadman
Copy link
Collaborator

@paulcadman paulcadman commented Oct 14, 2022

The builtin labels for natural and boolean are renamed to nat and bool.

This PR also updates the reference to the stdlib containing updates to the stdlib Nat and Bool definitions.

builtin nat
inductive Nat  {
  zero : Nat;
  suc : Nat → Nat;
};
builtin bool
inductive Bool {
  true : Bool;
  false : Bool;
};

Now that Bool is builtin, we do not need the BackendBool type and bridge from the Juvix Bool inductive type and the bool C type.

Updating stdlib submodule

After merging this PR - the juvix-stdlib PR anoma/juvix-stdlib#18 can be merged, then the juvix-stdlib submodule should be updated to point to juvix-stdlib main.

@paulcadman paulcadman added this to the 0.2.6 milestone Oct 14, 2022
@paulcadman paulcadman self-assigned this Oct 14, 2022
@paulcadman paulcadman changed the title Rename builtin natural to nat Rename builtin natural to nat and boolean to bool Oct 14, 2022
Now that Bool is builtin we don't need the Juvix to C boolean translation
@paulcadman paulcadman marked this pull request as ready for review October 14, 2022 13:15
@lukaszcz lukaszcz merged commit 504b5ec into main Oct 14, 2022
@lukaszcz lukaszcz deleted the rename-builtins branch October 14, 2022 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants