Verilator compile issues about axi HOT 7 CLOSED

resp_t vs axi_pkg::resp_t conflict

I agree that using resp_t as parameter to hold the entire response struct is not ideal, since it could collide with axi_pkg::resp_t when the latter is imported into module namespace. However, with namespaces guarding from exactly such conflicts, I am against changing this to rsp_t right now. Can you check in which modules exactly the conflict happens?

For instance, there are wildcard imports in axi_cdc, axi_intf, axi_test, tb_axi_cdc, and tb_axi_atop_filter. Is one of those imports not bounded by a module scope? I would prefer to fix this problem this way.

Typedefs in axi_dw_upsizer

I agree with your solution. Please open a PR against the DWC branch.

from axi.

I had the problem also for example in axi_demux. There no import is happening.

from axi.

I see, but if one of the instances above is compiled before axi_demux and has a globally visible import, that could cause the problem, no?

from axi.

This was it, removing the global imports fixed the compile and simulation worked. See fixes in #70.

from axi.

Nice catch and fix! The compilation units in SV are really a pain.

from axi.

This can be closed now that #70 is merged, right?

from axi.

Yes #70 solved the compile issues, partially.
However Verilator does not like in the assertions in the interface AXI_BUS_DV in axi_interface.
Guarding them with

// pragma translate_off
`ifndef VERILATOR

`endif
// pragma translate_on

resolves the compile issue

from axi.