Formatting fixes
authorSahand Kashani <sahand.kashani@gmail.com>
Mon, 23 Nov 2020 09:43:59 +0000 (10:43 +0100)
committerSahand Kashani <sahand.kashani@gmail.com>
Mon, 23 Nov 2020 09:55:09 +0000 (10:55 +0100)
backends/firrtl/firrtl.cc

index 5c2807ef49e755fc1e1cad744dc679cfb1768290..44c3397da3780aeaf84e72298bc75eb51d4a1e92 100644 (file)
@@ -178,11 +178,8 @@ std::string dump_const(const RTLIL::Const &data)
                else
                {
                        // If value is larger than 32 bits, then emit a binary representation of
-                       // the number. We have to do this as firrtl number literals don't support
-                       // specifying their width, therefore a binary literal is the only way to
-                       // guarantee the parameter widths match that provided on the RHS of a
-                       // verilog parameter assignment. There is a caveat to this approach
-                       // though:
+                       // the number as integers are not large enough to contain the result.
+                       // There is a caveat to this approach though:
                        //
                        // Note that parameter may be defined as having a fixed width as follows:
                        //
@@ -194,12 +191,12 @@ std::string dump_const(const RTLIL::Const &data)
                        // precision, then yosys considers the value you used as an int and
                        // assigns it a width of 32 bits regardless of the type of the parameter.
                        //
-                       //              defparam <inst_name> .test_signed = 49;                                         (width = 32, though should be 27 based on definition)
-                       //              defparam <inst_name> .test_unsigned = 40'd35;                   (width = 40, though should be 27 based on definition)
-                       //              defparam <inst_name> .test_signed_large = 40'd12;       (width = 40)
+                       //     defparam <inst_name> .test_signed = 49;           (width = 32, though should be 27 based on definition)
+                       //     defparam <inst_name> .test_unsigned = 40'd35;     (width = 40, though should be 27 based on definition)
+                       //     defparam <inst_name> .test_signed_large = 40'd12; (width = 40)
                        //
                        // We therefore may lose the precision of the original verilog literal if
-                       // it was written without it's bitwidth specifier.
+                       // it was written without its bitwidth specifier.
 
                        // Emit binary prefix for string.
                        res_str += "\"b";
@@ -329,7 +326,7 @@ void emit_extmodule(RTLIL::Cell *cell, RTLIL::Module *mod_instance, std::ostream
  * and emit parameterized extmodules with a unique name for each of them. The
  * name that's given to the extmodule is
  *
- *             <blackbox_name>_<instance_name>
+ *     <blackbox_name>_<instance_name>
  *
  * Beware that it is therefore necessary for users to replace "parameterized"
  * instances in the RTLIL sense with these custom extmodules for the firrtl to