Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix bug: ‘string’ does not name a type; #37

Closed
wants to merge 1 commit into from
Closed

fix bug: ‘string’ does not name a type; #37

wants to merge 1 commit into from

Conversation

higuoxing
Copy link

string is provided by std::string

or the building process will be failed

string is provided by std::string
@higuoxing
Copy link
Author

Well, this is the whole information ...

In file included from <command-line>:0:0:
/home/me/Documents/WorkSpace/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/me/Documents/WorkSpace/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/me/Documents/WorkSpace/riscv-sodor/emulator/rv32_1stage/obj_dir'
make[1]: *** [/home/me/Documents/WorkSpace/riscv-sodor/emulator/common/Makefile.include:104: emulator] Error 2
make[1]: Leaving directory '/home/me/Documents/WorkSpace/riscv-sodor/emulator/rv32_1stage'
make: *** [Makefile:108: emulator/rv32_1stage/emulator] Error 2

@redpanda3
Copy link

Thank you. I have the same issue.

@higuoxing
Copy link
Author

Thank you. I have the same issue.

Hi, you could use librecores/riscv-sodor, these bugs are fixed there : )

@ucbjrl ucbjrl mentioned this pull request Oct 12, 2018
@ucbjrl ucbjrl mentioned this pull request Oct 30, 2018
@edwardcwang
Copy link
Member

Fixed by #42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants