Skip to content

Conversation

kroening
Copy link
Member

@kroening kroening commented Aug 8, 2024

The Hazard3 RISC-V CPU comes with SVA assertions. This script downloads the RTL, and runs ebmc to check them.

@kroening kroening force-pushed the Hazard3 branch 3 times, most recently from 9b63b41 to 09cd0b7 Compare August 13, 2024 00:02
@kroening kroening force-pushed the Hazard3 branch 2 times, most recently from a077f9b to 80fce09 Compare September 14, 2024 00:51
@kroening kroening force-pushed the Hazard3 branch 2 times, most recently from a0f912b to f739d81 Compare October 20, 2024 22:27
@kroening kroening marked this pull request as ready for review October 20, 2024 22:27

# expected elaboration-time constant, but got `hazard3_muldiv_seq.properties.i'
# $past, for loop
# ebmc -I hdl -D HAZARD3_ASSERTIONS --systemverilog --bound 0 hdl/arith/hazard3_muldiv_seq.v
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do those issues still persist?

# clone Hazard3 repo if not done yet
if [ ! -e Hazard3/.git ] ; then
git clone https://github.com/Wren6991/Hazard3
(cd Hazard3 ; git checkout v1.0)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about git clone —branch v1.0 —depth 1

-D HAZARD3_ASSERTIONS --bound 0 \
hdl/arith/hazard3_shift_barrel.v

# conflicting assignment types
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here and below: do those persist? If so, can we have KNOWNBUG tests for them?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes; one test added, the other one linked as a comment

The Hazard3 RISC-V CPU comes with SVA assertions.  This script downloads the
RTL, and runs ebmc to check them.
@kroening kroening merged commit 6d19481 into main Oct 21, 2024
9 checks passed
@kroening kroening deleted the Hazard3 branch October 21, 2024 15:36
Romy15200 pushed a commit to Romy15200/nws that referenced this pull request Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants