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

VCS Manifest File Path Error #2573

Open
deepeshsahoo opened this issue Jan 11, 2025 · 1 comment
Open

VCS Manifest File Path Error #2573

deepeshsahoo opened this issue Jan 11, 2025 · 1 comment

Comments

@deepeshsahoo
Copy link

Bug Title

Issue with manifest file in vcsify with make vcs-* options

Type

  • Compile error

Steps to Reproduce

  1. Branch : Master
  2. Command line
    a. cd $core-v-verif/cv32e40p/sim/core
    b. make vcs-run

Additional context

Command in makefile
$(VCS) +vc -sverilog -race=all -ignore unique_checks -full64
-timescale=1ns/1ps
-assert svaext
-CC "-I$(VCS_HOME)/include -O3 -march=native" $(VCS_FLAGS)
-f $(MAKE_DIR)/$(CV_CORE_MANIFEST)
$(TBSRC_PKG) $(TBSRC)
The CV_CORE_MANIFEST already includes the right path, hence MAKE_DIR isn't needed.

image

@MikeOpenHWGroup
Copy link
Member

Thanks for your interest in CORE-V-VERIF @deepeshsahoo. You are right. The immediate fix for this is:

@@ -531,7 +531,7 @@ vcsify: CV_CORE_pkg tbsrc_pkg tbsrc
                -timescale=1ns/1ps \
                -assert svaext \
                -CC "-I$(VCS_HOME)/include -O3 -march=native" $(VCS_FLAGS) \
-               -f $(MAKE_DIR)/$(CV_CORE_MANIFEST) \
+               -f $(CV_CORE_MANIFEST) \
                $(TBSRC_PKG) $(TBSRC)

I currently do not have easy access to VCS, so if you could try this and submit a PR that would be appreciated. If this would ebe your first time submitting a PR to an OpenHW repo, please review CONTRIBUTING.

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

No branches or pull requests

2 participants