improved portability of logic conversion

- indirect converted reg continuous assignments through wires
- fix typeof for implicitly typed ports
- fix typeof for sized implicitly typed params

src/Convert/Logic.hs

@@ -106,21 +106,17 @@ traverseModuleItem ports scopes =
         fixModuleItem :: ModuleItem -> ModuleItem
         -- rewrite bad continuous assignments to use procedural assignments
         fixModuleItem (Assign AssignOptionNone lhs expr) =
-            if not (isReg lhs) then
-                Assign AssignOptionNone lhs expr
-            else if isConstant expr then
-                Initial $ Asgn AsgnOpEq Nothing lhs expr
-            else
-                AlwaysC AlwaysComb $ Asgn AsgnOpEq Nothing lhs expr
+            if not (isReg lhs)
+                then Assign AssignOptionNone lhs expr
+                else
+                    Generate $ map GenModuleItem
+                    [ MIPackageItem (Decl (Variable Local t x [] Nil))
+                    , Assign AssignOptionNone (LHSIdent x) expr
+                    , AlwaysC AlwaysComb $ Asgn AsgnOpEq Nothing lhs (Ident x)
+                    ]
             where
-                -- only handles expressions which are trivially constant for now
-                isConstant :: Expr -> Bool
-                isConstant Number{} = True
-                isConstant (Repeat _ es) = all isConstant es
-                isConstant (Concat   es) = all isConstant es
-                isConstant (BinOp _ e1 e2) = isConstant e1 && isConstant e2
-                isConstant (UniOp _ e) = isConstant e
-                isConstant _ = False
+                t = TypeOf expr
+                x = "sv2v_tmp_" ++ shortHash (lhs, expr)
         -- rewrite port bindings to use temporary nets where necessary
         fixModuleItem (Instance moduleName params instanceName rs bindings) =
             if null newItems

src/Convert/TypeOf.hs

@@ -37,11 +37,11 @@ traverseDeclM decl = do
     item <- traverseModuleItemM (MIPackageItem $ Decl decl)
     let MIPackageItem (Decl decl') = item
     case decl' of
-        Variable Local (Implicit sg rs) ident [] Nil -> do
+        Variable _ (Implicit sg rs) ident a _ ->
             -- implicit types, which are commonly found in function return
             -- types, are recast as logics to avoid outputting bare ranges
-            insertElem ident $ IntegerVector TLogic sg rs
-            return decl'
+            insertElem ident t' >> return decl'
+            where t' = injectRanges (IntegerVector TLogic sg rs) a
         Variable d t ident a e -> do
             let t' = injectRanges t a
             insertElem ident t'
@@ -52,6 +52,9 @@ traverseDeclM decl = do
             insertElem ident UnknownType >> return decl'
         Param _ UnknownType ident e ->
             typeof e >>= insertElem ident >> return decl'
+        Param _ (Implicit sg rs) ident _ ->
+            insertElem ident t' >> return decl'
+            where t' = IntegerVector TLogic sg rs
         Param _ t ident _ ->
             insertElem ident t >> return decl'
         ParamType{} -> return decl'

test/basic/logic_cond.sv

 module Example(inp, out);
     parameter ENABLED = 1;
+    localparam [0:0] DEFAULT = 1'b0;
     input logic inp;
     output logic out;
     if (ENABLED)
         always_comb out = inp;
     else
-        assign out = '0;
+        assign out = DEFAULT;
 endmodule

test/basic/logic_cond.v

 module Example(inp, out);
     parameter ENABLED = 1;
+    localparam [0:0] DEFAULT = 1'b0;
     input wire inp;
     output reg out;
     generate
         if (ENABLED)
             always @* out = inp;
         else
-            initial out = 0;
+            initial out = DEFAULT;
     endgenerate
 endmodule

test/basic/typeof.sv

@@ -77,12 +77,16 @@ module top;
 
     localparam X = 5'b10110;
     localparam Y = X + 6'b00001;
+    localparam [7:0] Z = 234;
     initial begin
         type(X) tX = X;
         type(Y) tY = Y;
+        type(Z) tZ = Z;
         $display("%b %d %d %d", X, X, $left(X), $right(X));
         $display("%b %d %d %d", Y, Y, $left(Y), $right(Y));
+        $display("%b %d %d %d", Z, Z, $left(Z), $right(Z));
         $display("%b %d %d %d", tX, tX, $left(tX), $right(tX));
         $display("%b %d %d %d", tY, tY, $left(tY), $right(tY));
+        $display("%b %d %d %d", tZ, tZ, $left(tZ), $right(tZ));
     end
 endmodule

test/basic/typeof.v

@@ -94,14 +94,19 @@ module top;
 
     localparam X = 5'b10110;
     localparam Y = X + 6'b00001;
+    localparam [7:0] Z = 234;
     initial begin : block5
         reg [4:0] tX;
         reg [5:0] tY;
+        reg [7:0] tZ;
         tX = X;
         tY = Y;
+        tZ = Z;
         $display("%b %d %d %d", X, X, 4, 0);
         $display("%b %d %d %d", Y, Y, 5, 0);
+        $display("%b %d %d %d", Z, Z, 7, 0);
         $display("%b %d %d %d", tX, tX, 4, 0);
         $display("%b %d %d %d", tY, tY, 5, 0);
+        $display("%b %d %d %d", tZ, tZ, 7, 0);
     end
 endmodule

test/basic/typeof_port.sv

+module Example(inp, out);
+    input inp;
+    output out;
+    type(inp) data;
+    assign data = ~inp;
+    assign out = data;
+endmodule

test/basic/typeof_port.v

+module Example(inp, out);
+    input inp;
+    output out;
+    wire data;
+    assign data = ~inp;
+    assign out = data;
+endmodule

test/basic/typeof_port_tb.v

+module top;
+    reg inp;
+    wire out;
+    Example e(inp, out);
+    initial begin
+        $monitor("%0d %b %b", $time, inp, out);
+    end
+endmodule