peepopt.muldiv: Add a signedness check.

Fixes #2318.
This commit is contained in:
Marcelina Kościelnicka 2020-08-04 16:30:24 +02:00
parent c39ebe6ae0
commit e89cc9c02f
2 changed files with 16 additions and 1 deletions

View File

@ -1,16 +1,18 @@
pattern muldiv
state <SigSpec> t x y
state <bool> is_signed
match mul
select mul->type == $mul
select GetSize(port(mul, \A)) + GetSize(port(mul, \B)) <= GetSize(port(mul, \Y))
endmatch
code t x y
code t x y is_signed
t = port(mul, \Y);
x = port(mul, \A);
y = port(mul, \B);
is_signed = param(mul, \A_SIGNED).as_bool();
branch;
std::swap(x, y);
endcode
@ -19,6 +21,7 @@ match div
select div->type.in($div)
index <SigSpec> port(div, \A) === t
index <SigSpec> port(div, \B) === x
filter param(div, \A_SIGNED).as_bool() == is_signed
endmatch
code

12
tests/opt/bug2318.ys Normal file
View File

@ -0,0 +1,12 @@
read_verilog <<EOT
module t(input [3:0] A, input [3:0] B, output signed [3:0] Y);
wire [7:0] P = A * B;
wire signed [7:0] SP = P;
wire signed [3:0] SB = B;
assign Y = SP / SB;
endmodule
EOT
equiv_opt -assert peepopt