ucb-bar / riscv-sodor

educational microarchitectures for risc-v isa
Other
688 stars 155 forks source link

Build error verilator #45

Open Mixermachine opened 6 years ago

Mixermachine commented 6 years ago

Hy there,

I completed the git clone and RISC-V front-end install to /usr/local/ After

./configure --with-riscv=/usr/local
make

I get

g++  -I.  -MMD -I/usr/local/share/verilator/include -I/usr/local/share/verilator/include/vltstd -DVL_PRINTF=printf -DVM_COVERAGE=0 -DVM_SC=0 -DVM_TRACE=0 -faligned-new -Wno-bool-operation -Wno-sign-compare -Wno-uninitialized -Wno-unused-but-set-variable -Wno-unused-parameter -Wno-unused-variable -Wno-shadow     -O1 -std=c++11  -g -I/home/e6430/risc-v/riscv-sodor/emulator/common -I/usr/local/include -I/home/e6430/risc-v/riscv-sodor/riscv-fesvr/fesvr  -L/usr/local/lib -Wl,-rpath,/usr/local/lib -L. -lpthread -DVERILATOR -include /home/e6430/risc-v/riscv-sodor/emulator/common/verilator.h   -c -o emulator.o /home/e6430/risc-v/riscv-sodor/emulator/common/emulator.cpp
In file included from <command-line>:
/home/e6430/risc-v/riscv-sodor/emulator/common/verilator.h:15:19: error: ‘string’ does not name a type; did you mean ‘stdin’?
   bool open(const string& name) override {
                   ^~~~~~
                   stdin
/home/e6430/risc-v/riscv-sodor/emulator/common/verilator.h:15:8: error: ‘bool VerilatedVcdFILE::open(const int&)’ marked ‘override’, but does not override
   bool open(const string& name) override {
        ^~~~
make[2]: *** [VTop.mk:63: emulator.o] Error 1
make[2]: Leaving directory '/home/e6430/risc-v/riscv-sodor/emulator/rv32_1stage/obj_dir'
make[1]: *** [/home/e6430/risc-v/riscv-sodor/emulator/common/Makefile.include:104: emulator] Error 2
make[1]: Leaving directory '/home/e6430/risc-v/riscv-sodor/emulator/rv32_1stage'
make: *** [Makefile:108: emulator/rv32_1stage/emulator] Error 2

My setup: Kubuntu 18.10

verilator --version
Verilator 3.922 2018-03-17 rev UNKNOWN_REV

Has anybody seen this error before? Might be the Verilator version?

Mixermachine commented 6 years ago

Verilator was installed via sudo apt install pkg-config verilator

ucbjrl commented 6 years ago

I believe this is a duplicate of issue #37. It should be addressed by PR #42. I'll close it when the latter PR is accepted.