tweag/asterius

Making the "info table check" optional

Open

#299 opened on Sep 17, 2019

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)Haskell (56 forks)batch import
P3good first issuetype: feature request

Repository metrics

Stars
 (1,949 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Currently, we collect all info table addresses and emit them into the .lib.mjs script; at runtime, the storage manager looks up the info table set when it encounters an info table, as a form of sanity check. This sanity check should be made optional and only enabled in the debug mode since the info table addresses bloat the .lib.mjs script size considerably and aren't strictly required for normal wasm execution.

Contributor guide

Research direction
Modify the storage manager to conditionally perform the info table check based on a flag; move the info table address collection out of the default production build.
Tech stack
haskell
Domain
compilers
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
HaskellWebAssembly
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30