Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Remove quotes around VcsCFlags #574

Merged

Conversation

stevenmburns
Copy link

This is the real fix for the issue addressed in PR #572
The reason for the nullptr not being found compiler error was that the -std=c++11 flag was not being seen by an older compiler (gcc 4.8.5). What was actually being run was the command line:

g++ -w  -pipe -fPIC "-ICS_HOME/include -I/nfs/site/disks/scl.work.50/ash/users/smburns/chisel-examples/test_run_dir/Alu_should_pass_AluSpec_tests -fPIC -std=c++11" -O -I/p/hdk/rtl/cad/x86-64_linux44/synopsys/vcsmx/Q-2020.03/include    -c ../vpi.cpp

The double quotes around the command line options in the middle meant that they were not being treated separately, and silently then -std=c+11 options was being ignored.
The proposed fix corrects this issue, and we can leave the nullptr usages in the vpi.h file.
Also, the VcsCFlags annotation will to work, allowing new C compiler flags to be added as necessary by the user.

Copy link
Collaborator

@ekiwi ekiwi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! I think this might have been a copy-paste error from Verilator.

@ekiwi ekiwi added this to the 0.5.x milestone Dec 6, 2022
@ekiwi ekiwi added the Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI. label Dec 6, 2022
@mergify mergify bot merged commit 24e4ec4 into ucb-bar:main Dec 6, 2022
mergify bot pushed a commit that referenced this pull request Dec 6, 2022
* Remove quotes around VcsCFlags

* Revert back to using nullptr

* Remove added blank line

(cherry picked from commit 24e4ec4)
@mergify mergify bot added the Backported This PR has been backported to marked stable branch label Dec 6, 2022
ekiwi pushed a commit that referenced this pull request Dec 7, 2022
* Remove quotes around VcsCFlags

* Revert back to using nullptr

* Remove added blank line

(cherry picked from commit 24e4ec4)

Co-authored-by: Steve Burns <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Backported This PR has been backported to marked stable branch Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants