Fixed assertion of ap_done
ap_done is now not asserted based on timeouts, rather only on memories_sent
being asserted. Avoids confusing behavior where hardware seems to abort
without a reason.
At some point we may want to revert this to enable
timeouts?
@@ -245,13 +245,7 @@ fn top_level(
// done signal
module.add_stmt(v::Parallel::Assign(
"ap_done".into(),
- v::Expr::new_logical_or(
- v::Expr::new_gt("counter", "timeout"),
- v::Expr::new_eq(
- "memories_sent",
- v::Expr::new_ulit_bin(memories.len() as u32, "1"),
- ),
- ),
+ "memories_sent".into(),
));
module
