luke
/
verilator
mirror of https://github.com/verilator/verilator.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix internal clang warnings.

This commit is contained in:
Wilson Snyder 2017-05-24 23:51:15 -04:00
parent 6b04901c4d
commit 447d803800
4 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
include/verilated.cpp
View File

@ -1135,7 +1135,7 @@ IData VL_VALUEPLUSARGS_INW(int rbits, const string& ld, WDataOutP rwp) {
_VL_CLEAN_INPLACE_W(rbits,rwp); _VL_CLEAN_INPLACE_W(rbits,rwp);
return 1; return 1;
} }
IData VL_VALUEPLUSARGS_INN(int rbits, const string& ld, string& rdr) { IData VL_VALUEPLUSARGS_INN(int, const string& ld, string& rdr) {
string prefix; string prefix;
bool inPct = false; bool inPct = false;
bool done = false; bool done = false;

2
include/verilated.h
View File

@ -1077,7 +1077,9 @@ static inline QData VL_MULS_QQQ(int,int lbits,int, QData lhs,QData rhs) {
static inline WDataOutP VL_MULS_WWW(int,int lbits,int, WDataOutP owp,WDataInP lwp,WDataInP rwp){ static inline WDataOutP VL_MULS_WWW(int,int lbits,int, WDataOutP owp,WDataInP lwp,WDataInP rwp){
int words = VL_WORDS_I(lbits); int words = VL_WORDS_I(lbits);
// cppcheck-suppress variableScope
IData lwstore[VL_MULS_MAX_WORDS]; // Fixed size, as MSVC++ doesn't allow [words] here IData lwstore[VL_MULS_MAX_WORDS]; // Fixed size, as MSVC++ doesn't allow [words] here
// cppcheck-suppress variableScope
IData rwstore[VL_MULS_MAX_WORDS]; IData rwstore[VL_MULS_MAX_WORDS];
WDataInP lwusp = lwp; WDataInP lwusp = lwp;
WDataInP rwusp = rwp; WDataInP rwusp = rwp;

9
src/V3CCtors.cpp
View File

@ -75,7 +75,8 @@ public:
m_numStmts += 1; m_numStmts += 1;
} }
V3CCtorsVisitor(AstNodeModule* nodep, string basename, string argsp="", string callargsp="") { V3CCtorsVisitor(AstNodeModule* nodep, string basename, string argsp="", string callargsp="",
const string& stmt="") {
m_basename = basename; m_basename = basename;
m_argsp = argsp; m_argsp = argsp;
m_callargsp = callargsp; m_callargsp = callargsp;
@ -87,6 +88,9 @@ public:
m_tlFuncp->isStatic(false); m_tlFuncp->isStatic(false);
m_tlFuncp->slow(true); m_tlFuncp->slow(true);
m_tlFuncp->argTypes(m_argsp); m_tlFuncp->argTypes(m_argsp);
if (stmt != "") {
m_tlFuncp->addStmtsp(new AstCStmt(nodep->fileline(), stmt));
}
m_funcp = m_tlFuncp; m_funcp = m_tlFuncp;
m_modp->addStmtp(m_tlFuncp); m_modp->addStmtp(m_tlFuncp);
} }
@ -100,7 +104,8 @@ void V3CCtors::cctorsAll() {
// Process each module in turn // Process each module in turn
V3CCtorsVisitor var_reset (modp, "_ctor_var_reset"); V3CCtorsVisitor var_reset (modp, "_ctor_var_reset");
V3CCtorsVisitor configure_coverage (modp, "_configure_coverage", V3CCtorsVisitor configure_coverage (modp, "_configure_coverage",
EmitCBaseVisitor::symClassVar()+ ", bool first", "vlSymsp, first"); EmitCBaseVisitor::symClassVar()+ ", bool first", "vlSymsp, first",
"if (0 && vlSymsp && first) {} // Prevent unused\n");
for (AstNode* np = modp->stmtsp(); np; np = np->nextp()) { for (AstNode* np = modp->stmtsp(); np; np = np->nextp()) {
AstVar* varp = np->castVar(); AstVar* varp = np->castVar();

2
src/V3Options.cpp
View File

@ -560,7 +560,7 @@ string V3Options::version() {
void V3Options::throwSigsegv() { void V3Options::throwSigsegv() {
// cppcheck-suppress nullPointer // cppcheck-suppress nullPointer
char* zp=NULL; *zp=0; char* zp=NULL; *zp=0; // Intentional core dump, ignore warnings here
} }
//###################################################################### //######################################################################