Optimize side table masks (#713)

Based on experiments, these are the smallest masks to pass CI. In other words, `SideTableEntry` only requires `u35` at this point. I'll add the fields from `func_type()` and `func()` in `module.rs` to the side table in #711, and `SideTableEntry` as `u64` might not be sufficient. #46 --------- Co-authored-by: Zhou Fang <[email protected]>
google · Jan 3, 2025 · b9de9e9 · b9de9e9
1 parent a877f75
commit b9de9e9
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/crates/interpreter/src/side_table.rs b/crates/interpreter/src/side_table.rs
@@ -32,9 +32,9 @@ pub struct SideTableEntryView {
 
 impl SideTableEntry {
     const DELTA_IP_MASK: u64 = 0xffff;
-    const DELTA_STP_MASK: u64 = 0xffff << 16;
-    const VAL_CNT_MASK: u64 = 0xffff << 32;
-    const POP_CNT_MASK: u64 = 0xffff << 48;
+    const DELTA_STP_MASK: u64 = 0x7fff << 16;
+    const VAL_CNT_MASK: u64 = 0x3 << 31;
+    const POP_CNT_MASK: u64 = 0x3 << 33;
 
     pub fn new(view: SideTableEntryView) -> Result<Self, Error> {
         let mut fields = 0;