From f7bf277e899609a2ff1d97c64df0731dfd3c5c26 Mon Sep 17 00:00:00 2001 From: Hansung Kim Date: Fri, 28 Apr 2023 00:50:34 -0700 Subject: [PATCH] Fix unittest for CoalShiftQueue deq.valid change --- src/test/scala/coalescing/CoalescingUnitTest.scala | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/test/scala/coalescing/CoalescingUnitTest.scala b/src/test/scala/coalescing/CoalescingUnitTest.scala index b97e5a5..7469e59 100644 --- a/src/test/scala/coalescing/CoalescingUnitTest.scala +++ b/src/test/scala/coalescing/CoalescingUnitTest.scala @@ -357,12 +357,11 @@ class CoalShiftQueueTest extends AnyFlatSpec with ChiselScalatestTester { c.io.queue.enq.valid.poke(false.B) // invalidate should work for the head just being dequeued at the same - // cycle. However, it should not change deq.valid right away to avoid - // combinational cycles (see definition). + // cycle c.io.invalidate.valid.poke(true.B) c.io.invalidate.bits.poke(0x1.U) c.io.queue.deq.ready.poke(true.B) - c.io.queue.deq.valid.expect(true.B) + c.io.queue.deq.valid.expect(false.B) c.clock.step() // 0x12 should have been dequeued c.io.invalidate.valid.poke(false.B)