--float-size for custom float size

Author: VonTum

src/codegen/system_verilog.rs

@@ -6,6 +6,7 @@ use ibig::IBig;
 use sus_proc_macro::get_builtin_type;
 
 use crate::alloc::zip_eq;
+use crate::config::config;
 use crate::latency::AbsLat;
 use crate::linker::{IsExtern, LinkInfo};
 use crate::prelude::*;
@@ -93,7 +94,10 @@ fn typ_to_declaration(mut typ: &ConcreteType, var_name: &str) -> String {
                     }
                 }
                 get_builtin_type!("bool") => return format!(" {var_name}{array_string}"),
-                get_builtin_type!("float") => return format!("[31:0] {var_name}{array_string}"),
+                get_builtin_type!("float") => {
+                    let float_msb = config().float_size - 1;
+                    return format!("[{float_msb}:0] {var_name}{array_string}");
+                }
                 _ => todo!("Structs"),
             },
             ConcreteType::Array(arr) => {
@@ -214,9 +218,8 @@ impl<'g> CodeGenerationContext<'g> {
                         matches!(cst, Value::Unset),
                         "TODO: Generative non-Unset floats"
                     );
-                    result
-                        .write_str("32'bxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx")
-                        .unwrap();
+                    let float_size = config().float_size;
+                    write!(result, "{float_size}'x").unwrap();
                 }
                 _ => todo!("Structs"),
             },

src/config.rs

@@ -60,6 +60,9 @@ pub struct ConfigStruct {
     pub enabled_debug_paths: HashSet<String>,
     pub early_exit: EarlyExitUpTo,
     pub no_redump: bool,
+
+    /// Temporary, just because flopoco forces weirdly-sized floats, have it provide a struct in the future instead
+    pub float_size: usize,
 }
 
 pub const VERSION_INFO: &str = concat!(
@@ -153,6 +156,13 @@ fn command_builder() -> Command {
                 }
             })
         )
+        .arg(Arg::new("float-size")
+            .long("float-size")
+            .hide(true)
+            .help("Set the size of floats - in bits. Makes no claims about the representation, that's up to the libraries")
+            .value_parser(clap::value_parser!(usize))
+            .default_value("32")
+        )
         // Debug stuff
         .arg(Arg::new("debug")
             .long("debug")
@@ -242,6 +252,7 @@ where
         files,
         sus_home_override,
         no_redump: matches.get_flag("no-redump"),
+        float_size: *matches.get_one("float-size").unwrap(),
     })
 }
 

src/typing/concrete_type.rs

@@ -2,6 +2,7 @@ use ibig::IBig;
 use ibig::UBig;
 use sus_proc_macro::get_builtin_type;
 
+use crate::config;
 use crate::linker::GlobalUUID;
 use crate::prelude::*;
 use crate::to_string::join_string_iter_formatter;
@@ -287,7 +288,7 @@ impl ConcreteType {
                 bounds.bitwidth()
             }
             get_builtin_type!("bool") => 1,
-            get_builtin_type!("float") => 32,
+            get_builtin_type!("float") => config().float_size as u64,
             _other => todo!("Other Named Structs are not implemented yet"),
         }
     }