Michael Platzer
Michael Platzer
Hi, when initializing a localparam integer array with a slice of a larger localparam integer array, then Verilator complains that the resulting localparam is not constant. These arrays are initialized...
Hi, when attempting to assign the values of an unpacked integer array parameter directly I get an internal error "InitArray type not assigned by AstPattern/Var visitor". Code example: ``` module...
Ensure that the controller FSM's XIF commit transaction tracking signals are reset every time a new instruction enters the EX stage (now ensured by simply resetting these signals anytime the...
When the XIF interface is enabled, the controller FSM uses the signals `commit_valid_q` and `commit_kill_q` to track whether an XIF instruction already received its commit transaction. These signals must be...
This PR adds a new operation `ADDS` to the `operation_e` enum, which implements a symmetric addition/subtraction, i.e., with both addends having the same source format. As discussed in #113, currently...
Currently the NONCOMP block sets the result bit when either operand of a "not equal" comparison is a quiet NaN but not for signaling NaNs. In order to treat quiet...
The NONCOMP block sets the result bit when either operand of a "not equal" comparison is a NaN: https://github.com/openhwgroup/cvfpu/blob/4aac6b3e87a30c8567dbe7401eba3274eea18afc/src/fpnew_noncomp.sv#L276-L279 However, it does so only for quiet NaNs, not for signaling...
Currently, the addend operand of the FMA shares its FP format with the result: https://github.com/openhwgroup/cvfpu/blob/4aac6b3e87a30c8567dbe7401eba3274eea18afc/src/fpnew_fma_multi.sv#L37-L38 This makes sense when considering the various fused multiply-accumulate operations of the RISC-V spec, which...
The FMA currently calculates the sum of the product and the addend first and, in case certain conditions are met, it then negates it. https://github.com/openhwgroup/cvfpu/blob/eba1bbebd84121a529a4696c5ce792da3013a8bd/src/fpnew_fma_multi.sv#L474-L475 However, two's complement negation requires...
This PR fixes #123 by correcting the assignment of floating-point status flags on `I2F` and `F2I` conversions, which is currently not compliant with IEEE 754-2008 as detailed in the linked...