replica
/
firesim
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[assert] Fix a bug that would lead to double counting asserts in tests

This commit is contained in:
David Biancolin 2020-06-20 17:45:58 -07:00
parent 40b900c5ca
commit 7c7d3709b5
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sim/midas/src/main/scala/midas/widgets/Assert.scala
View File

@ -29,8 +29,8 @@ class AssertBridgeModule(assertMessages: Seq[String])(implicit p: Parameters) ex
val stallN = (!assertFire || resume)
val dummyPredicate = true.B
val tFireHelper = DecoupledHelper(hPort.toHost.hValid, stallN, dummyPredicate)
val targetFire = tFireHelper.fire() // FIXME: On next RC bump
val tFireHelper = DecoupledHelper(hPort.toHost.hValid, stallN)
val targetFire = tFireHelper.fire
hPort.toHost.hReady := tFireHelper.fire(hPort.toHost.hValid)
// We only sink tokens, so tie off the return channel
hPort.fromHost.hValid := true.B
@ -39,7 +39,7 @@ class AssertBridgeModule(assertMessages: Seq[String])(implicit p: Parameters) ex
}
genROReg(assertId, "id")
genROReg(assertFire, "fire")
genROReg(assertFire && hPort.toHost.hValid, "fire")
// FIXME: no hardcode
genROReg(cycles(31, 0), "cycle_low")
genROReg(cycles >> 32, "cycle_high")