search

wyvernSemi / vproc

Virtual processor co-simulation element for Verilog, VHDL and SystemVerilog environments
http://www.anita-simulators.org.uk/wyvernsemi
GNU General Public License v3.0
46 stars 9 forks source link

Fixed Icarus compile #1

Closed john32979 closed 3 years ago

john32979 commented 3 years ago

Fixed makefile.ica and Icarus related compile errors to allow compile to finish. However, I'm unable to fix the warnings.

wyvernSemi commented 3 years ago

Hi John. I tested the changes you are requesting to pull, but they do not work unless I add an include path to find veriuser.h which, when a I did a clean install, was located in /usr/include/iverilog. How are you compiling without this include path?

john32979 commented 3 years ago

Hi Simon, my Linux distro is a Redhat Enterprise Linux clone that installs the header into /usr/include/veriuser.h. /usr/include is on the default search path for gcc. Are you using Ubuntu?

wyvernSemi commented 3 years ago

I am using Ubuntu. Perhaps if the ICADIR variable is restored, pointing to /usr/include/iverilog, and -I${ICADIR} added to the CFLAGS, then this will not interfere with the redhat installation, will work for Ubuntu, but can also be overridden on the command line (make ICADIR=<my iverilog path>) for other possible settings on different distros.

I am keen to pull down your change, as I am (coincidentally) about to update the code to support a VHDL version using the foreign language interface (FLI) for ModelSim, and would like your change merged before this is done. If you agree with the above, then either I can pull down you change and update or, if you'd rather, you can make the change in your branch, which would allow it to be tested on RedHat (which I do not have access to).

john32979 commented 3 years ago

Hi Simon, I made the change as you suggested.