chore: clean up todos, add comments

Author: greenhat

frontend-wasm/src/component/translator.rs

@@ -131,6 +131,7 @@ impl<'a, 'data> ComponentTranslator<'a, 'data> {
                 }
                 self.module_instances_source.push(*static_module_idx);
                 // TODO: create and init module instance tables
+                // see https://github.com/0xPolygonMiden/compiler/issues/133
                 let module = &self.parsed_modules[*static_module_idx].module;
                 let mut module_args: Vec<ModuleArgument> = Vec::new();
                 for (idx, arg) in args.iter().enumerate() {
@@ -222,6 +223,7 @@ impl<'a, 'data> ComponentTranslator<'a, 'data> {
                 }
                 EntityIndex::Table(_idx) => {
                     // TODO: init the exported table with this module's table initialization values
+                    // see https://github.com/0xPolygonMiden/compiler/issues/133
                     Ok(ModuleArgument::Table)
                 }
                 EntityIndex::Memory(_) => {
@@ -301,7 +303,8 @@ impl<'a, 'data> ComponentTranslator<'a, 'data> {
                 "Exporting of an imported module is not supported".to_string(),
             )),
             Export::Type(_) => {
-                // TODO: implement type exports
+                // Besides the function exports the individual type are also exported from the component
+                // We can ignore them for now
                 Ok(())
             }
         }

frontend-wasm/src/component/types/mod.rs

@@ -2,7 +2,6 @@
 
 // Based on wasmtime v16.0 Wasm component translation
 
-// TODO: remove this once Wasm CM support is complete
 #![allow(dead_code)]
 
 pub mod resources;

frontend-wasm/src/module/build_ir.rs

@@ -54,7 +54,9 @@ pub fn build_ir_module(
     build_globals(&parsed_module.module, &mut module_builder, diagnostics)?;
     build_data_segments(parsed_module, &mut module_builder, diagnostics)?;
     let mut func_translator = FuncTranslator::new();
-    // TODO: Ugly! Find a better way to consume the function body inputs
+    // Although this renders this parsed module invalid(without functiong
+    // bodies), we don't support multiple module instances. Thus, this
+    // ParseModule will not be used again to make another module instance.
     let func_body_inputs = mem::take(&mut parsed_module.function_body_inputs);
     for (defined_func_idx, body_data) in func_body_inputs {
         let func_index = &parsed_module.module.func_index(defined_func_idx);