JulianKemmerer / PipelineC

A C-like hardware description language (HDL) adding high level synthesis(HLS)-like automatic pipelining as a language construct/compiler feature.
https://github.com/JulianKemmerer/PipelineC/wiki
GNU General Public License v3.0
586 stars 47 forks source link

Clean and standardize PipelineC repo. #126

Open bartokon opened 1 year ago

bartokon commented 1 year ago

Repo readability is really low. I suggest to move all .h files to some kind of headers folder and iee, io, axi folders to merge them. Do we really need one extra layer-directory for dvi/dvi_pmod.c? fixed/q0_23.h?

JulianKemmerer commented 1 year ago

Making a headers directory sounds fine to me And then move dirs+files like you said headers/axi/axis.h for ex. dvi/ dir was to match vga/ which did end up having a few files in it fixed/ is supposed to be filled in with various fixed point type implementations (if they dont get autogenerated otherwise :-/)

JulianKemmerer commented 1 year ago

I might prefer include as dir name over headers ex. include/axi/axis.h

ghost commented 1 year ago

Can we roll this out?

JulianKemmerer commented 1 year ago

I think it needs some clarification still @Datavenia

Is the goal to move directories with .h files into include/<name>/<file>.h ?

I am not in a rush to get to this

But if you put together a PR with the change I can try and test it after the big cleanup PR thats also pending probably