Skip to content

Commit 2e2459b

Browse files
[dv] Downgrade more errors when building vcs simv with newer compilers
6d9e632 previously downgraded 'implicit-function-declaration' error into warnings when building the vcs simv executable, which is an issue on newer compiler releases which have changed the default severity for several checks. int-conversion errors have also been observed for the same reason. An example is given below. Downgrade this error as well. (again, only while building the simv) // rmapats.c:466:36: error: passing argument 1 of // 'setChildClockWriteFuncAndPcode' makes integer from pointer without a cast // [-Wint-conversion] Co-authored-by: Andreas Kurth <[email protected]> Signed-off-by: Harry Callahan <[email protected]>
1 parent 7619f89 commit 2e2459b

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

hw/dv/tools/dvsim/vcs.hjson

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,15 @@
3838
// and final value of the selection input at the end of a simulation timestep.
3939
// See https://github.com/lowRISC/ibex/issues/845.
4040
"-xlrm uniq_prior_final",
41-
// Newer compiler versions have escalated this warning into an error by default, and
42-
// VCS generates code to build the simulation executable which fails this check.
41+
// Newer compiler versions have escalated these warnings into an error by default, and
42+
// VCS generates code to build the simulation executable which fails these checks.
4343
// De-escalate the error (only when building the simv itself.)
4444
// > rmapats.c:20:9: error: implicit declaration of function 'vcs_simpSetEBlkEvtID'
4545
// > [-Wimplicit-function-declaration]
46-
"-Xcflags='-Wno-error=implicit-function-declaration'",
46+
// > rmapats.c:466:36: error: passing argument 1 of 'setChildClockWriteFuncAndPcode' makes
47+
// > integer from pointer without a cast
48+
// > [-Wint-conversion]
49+
"-Xcflags='-Wno-error=implicit-function-declaration -Wno-error=int-conversion'",
4750
// Force DPI-C compilation in C99 mode. The -fno-extended-identifiers flag tells g++
4851
// not to worry about unicode. For some bizarre reason, the VCS DPI code contains
4952
// preprocessor macros with smart quotes, which causes GCC 10.2 and later to choke

0 commit comments

Comments
 (0)