efabless / caravel_user_project

https://caravel-user-project.readthedocs.io
Apache License 2.0
180 stars 330 forks source link

clean up unused files #132

Open proppy opened 2 years ago

proppy commented 2 years ago

is https://github.com/efabless/caravel_user_project/blob/main/verilog/rtl/uprj_netlists.v still used?

proppy commented 1 year ago

similarly are files in https://github.com/efabless/caravel_user_project/tree/main/verilog/includes still used? (they don't seem to be using a consistent syntax)

marwaneltoukhy commented 1 year ago

similarly are files in https://github.com/efabless/caravel_user_project/tree/main/verilog/includes still used? (they don't seem to be using a consistent syntax)

Yes they are used in simulation, they have inconsistent syntax because GL and RTL simulations are done using iVerilog, while GL+SDF simulations are done using CVC. The tools use different syntax for including files.

proppy commented 1 year ago

@marwaneltoukhy is https://github.com/efabless/caravel_user_project/blob/main/verilog/rtl/uprj_netlists.v also used in simulation?

marwaneltoukhy commented 1 year ago

@proppy No it's not, at least not the default flow. Investigating if some users might be using it for any other reason, I don't want to remove a file that might end up breaking someone's flow.