From eeb92da8a1743cd081909ebe1e371a3a8d0d9535 Mon Sep 17 00:00:00 2001
From: Hansung Kim <hansung_kim@berkeley.edu>
Date: Thu, 18 Jan 2024 01:07:49 -0800
Subject: [PATCH] Fix InFlightTable lookup assert

---
 src/main/scala/radiance/memory/Coalescing.scala | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/main/scala/radiance/memory/Coalescing.scala b/src/main/scala/radiance/memory/Coalescing.scala
index 0282eaa..02a27c3 100644
--- a/src/main/scala/radiance/memory/Coalescing.scala
+++ b/src/main/scala/radiance/memory/Coalescing.scala
@@ -1317,14 +1317,12 @@ class InFlightTable(
   // Lookup logic
   io.lookup.valid := table(io.lookupSourceId).valid
   io.lookup.bits := table(io.lookupSourceId).bits
-  // under normal circumstances, every lookup to the table should succeed
-  // as long as the request gets recorded earlier than the response
-  when(io.lookup.ready) {
-    assert(table(io.lookupSourceId).valid === true.B,
-      "table lookup with a valid sourceId failed")
-  }
   // Dequeue as soon as lookup succeeds
   when(io.lookup.fire) {
+    // every lookup to the table should succeed as the request should have
+    // gotten recorded earlier than the response
+    assert(table(io.lookupSourceId).valid === true.B,
+      "table lookup with a valid sourceId failed")
     table(io.lookupSourceId).valid := false.B
   }
   assert(