You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It feels odd to have the memory.read_port(domain="sync", synchronous=False) combination of arguments to be valid, and in fact that seems like it'd cause some hard to find bugs.
The text was updated successfully, but these errors were encountered:
@sbourdeauducq Because of the way Module is implemented, trying to do m.d.comb += stmt or m.d["comb"] += stmt will always add those statements to the combinatorial domain, even if there is a synchronous domain called comb.
So I wouldn't say it's valid, and further, we probably need to disallow either creating a ClockDomain("comb") or adding it to any module.
It feels odd to have the
memory.read_port(domain="sync", synchronous=False)
combination of arguments to be valid, and in fact that seems like it'd cause some hard to find bugs.The text was updated successfully, but these errors were encountered: