tactcomplabs / rev

RISC-V SST CPU Component
Other
17 stars 21 forks source link

Failed 'dep_check' test with cost of 'add' in RV32I larger than 1 #314

Open moonjohn-hpe opened 1 month ago

moonjohn-hpe commented 1 month ago

Describe the bug To test the cascade of dependent instructions in 'dep_check', the cost of 'add' in RV32I is modified to 2 by SetCost(2) according to the caption in 'dep_check.c'. However, the test is failed with 'Timeout' message.

To Reproduce All issues must be accompanied by:

Expected behavior The cost of 'add' in RV32I can be set to a value larger than 1 according to the caption in 'dep_check.c'. With an addition of 'SetCost(2)' in 'RV32I.h', it is supposed to run the test successfully and report eff less than 100%.

Trace If applicable, post a trace of the execution.

Additional context Add any other context about the problem here.

jleidel commented 1 month ago

@moonjohn-hpe what branch of Rev are you using?

moonjohn-hpe commented 1 month ago

It happens in both 'main' and 'devel' branch.