feat: full no_std support

Signed-off-by: Henry Gressmann <[email protected]>

Author: explodingcamera

.github/workflows/test.yaml

@@ -41,10 +41,10 @@ jobs:
         run: rustup update nightly
 
       - name: Build (nightly, no default features)
-        run: cargo +nightly build --workspace --exclude wasm-testsuite --no-default-features
+        run: cargo +nightly build --workspace --exclude wasm-testsuite --exclude rust-wasm-examples --no-default-features
 
       - name: Run tests (nightly, no default features)
-        run: cargo +nightly test --workspace --exclude wasm-testsuite --no-default-features
+        run: cargo +nightly test --workspace --exclude wasm-testsuite --exclude rust-wasm-examples --no-default-features
 
       - name: Run MVP testsuite (nightly)
         run: cargo +nightly test-mvp

Cargo.lock

@@ -1,6 +1,5 @@
 [workspace]
 members=["crates/*"]
-default-members=["crates/cli"]
 resolver="2"
 
 [workspace.package]
@@ -9,3 +8,12 @@ edition="2021"
 license="MIT OR Apache-2.0"
 authors=["Henry Gressmann <[email protected]>"]
 repository="https://github.com/explodingcamera/tinywasm"
+
+[package]
+name="tinywasm-root"
+publish=false
+edition="2021"
+
+[dev-dependencies]
+color-eyre="0.6"
+tinywasm={path="crates/tinywasm"}

Cargo.toml

@@ -55,7 +55,7 @@ $ cargo add tinywasm
 - **`parser`**\
   Enables the `tinywasm-parser` crate. This is enabled by default.
 
-With all these features disabled, TinyWasm does not depend on any external crates and can be used in `no_std` environments.
+With all these features disabled, TinyWasm only depends on `core`, `alloc` and `libm` and can be used in `no_std` environments.
 
 <!-- # 🎯 Goals
 

README.md

@@ -1,5 +1,5 @@
+use crate::log;
 use alloc::{boxed::Box, format, string::ToString, vec::Vec};
-use log::debug;
 use tinywasm_types::*;
 use wasmparser::{FuncValidator, OperatorsReader, ValidatorResources};
 
@@ -239,7 +239,7 @@ pub fn process_operators<'a>(
     let mut labels_ptrs = Vec::new(); // indexes into the instructions array
 
     for op in ops {
-        debug!("op: {:?}", op);
+        log::debug!("op: {:?}", op);
 
         let op = op?;
         validator.op(offset, &op)?;
@@ -274,7 +274,7 @@ pub fn process_operators<'a>(
             }
             End => {
                 if let Some(label_pointer) = labels_ptrs.pop() {
-                    debug!("ending block: {:?}", instructions[label_pointer]);
+                    log::debug!("ending block: {:?}", instructions[label_pointer]);
 
                     let current_instr_ptr = instructions.len();
 

crates/parser/src/conversion.rs

@@ -14,7 +14,9 @@ use log;
 #[cfg(not(feature = "logging"))]
 mod log {
     macro_rules! debug    ( ($($tt:tt)*) => {{}} );
+    macro_rules! error    ( ($($tt:tt)*) => {{}} );
     pub(crate) use debug;
+    pub(crate) use error;
 }
 
 mod conversion;

crates/parser/src/lib.rs

@@ -181,9 +181,9 @@ impl ModuleReader {
                 validator.end(offset)?;
                 self.end_reached = true;
             }
-            CustomSection(reader) => {
+            CustomSection(_reader) => {
                 debug!("Found custom section");
-                debug!("Skipping custom section: {:?}", reader.name());
+                debug!("Skipping custom section: {:?}", _reader.name());
             }
             // TagSection(tag) => {
             //     debug!("Found tag section");

crates/parser/src/module.rs

@@ -16,6 +16,7 @@ path="src/lib.rs"
 log={version="0.4", optional=true}
 tinywasm-parser={version="0.3.0-alpha.0", path="../parser", default-features=false, optional=true}
 tinywasm-types={version="0.3.0-alpha.0", path="../types", default-features=false}
+libm={version="0.2", default-features=false}
 
 [dev-dependencies]
 wasm-testsuite={path="../wasm-testsuite"}

crates/tinywasm/Cargo.toml

@@ -1,5 +1,5 @@
+use crate::log;
 use alloc::{boxed::Box, format, string::String, string::ToString, vec, vec::Vec};
-use log::{debug, info};
 use tinywasm_types::{FuncAddr, FuncType, ValType, WasmValue};
 
 use crate::{
@@ -34,7 +34,7 @@ impl FuncHandle {
 
         // 4. If the length of the provided argument values is different from the number of expected arguments, then fail
         if func_ty.params.len() != params.len() {
-            info!("func_ty.params: {:?}", func_ty.params);
+            log::info!("func_ty.params: {:?}", func_ty.params);
             return Err(Error::Other(format!(
                 "param count mismatch: expected {}, got {}",
                 func_ty.params.len(),
@@ -62,7 +62,7 @@ impl FuncHandle {
         };
 
         // 6. Let f be the dummy frame
-        debug!("locals: {:?}", locals);
+        log::debug!("locals: {:?}", locals);
         let call_frame = CallFrame::new(func_inst, params, locals);
 
         // 7. Push the frame f to the call stack

crates/tinywasm/src/func.rs

@@ -4,7 +4,7 @@ use core::fmt::Debug;
 
 use crate::{
     func::{FromWasmValueTuple, IntoWasmValueTuple, ValTypesFromTuple},
-    LinkingError, Result,
+    log, LinkingError, Result,
 };
 use alloc::{
     collections::BTreeMap,