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

host-spec: Various cleanups #402

Merged
merged 17 commits into from
Mar 23, 2021
Merged

host-spec: Various cleanups #402

merged 17 commits into from
Mar 23, 2021

Conversation

FlorianFranzen
Copy link
Contributor

@FlorianFranzen FlorianFranzen commented Mar 3, 2021

This is a collection of various smaller changes to the host spec:

  • remove leftover version diff tag
  • delete legacy hostapi
  • fixup appendix lettering
  • add generated figure to gitignore
  • rename runtime entries to api
  • fix broken references
  • fix various mistakes in host api
  • update inherents section
  • document __heap_base
  • document :heappages;
  • document the use of imported memory
  • extend runtime API with version info
  • restructure runtime api sections
  • clean up tables in runtime api chapter
  • remove all indices from chapters

@FlorianFranzen FlorianFranzen added the specification Additions and Updates to the Specification label Mar 3, 2021
@FlorianFranzen FlorianFranzen requested a review from drskalman March 3, 2021 16:26
@FlorianFranzen FlorianFranzen self-assigned this Mar 3, 2021
@drskalman
Copy link
Contributor

  • Collection vs Byte array I didn't understand what was going on
  • "rename runtime entries to api
    " I remember we spceifically discussed not to do that as these are function called by the host where there many other runtime apis which are called within the runtime. We said we'll find a approperiate way of distinguishing the two in the startegy meeting.
  • Pushed here because I don't if I messed up with the labels or not
    e66f215

@FlorianFranzen
Copy link
Contributor Author

I will fix the typo and defined SCALE collection.

@lamafab
Copy link
Contributor

lamafab commented Mar 17, 2021

Went over this with Florian in a Google Meet call. LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
specification Additions and Updates to the Specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants