diff --git a/src/V3EmitCFunc.cpp b/src/V3EmitCFunc.cpp index e17d72150..e1b35f8ae 100644 --- a/src/V3EmitCFunc.cpp +++ b/src/V3EmitCFunc.cpp @@ -627,7 +627,7 @@ void EmitCFunc::emitVarReset(AstVar* varp) { } } else if (AstUnpackArrayDType* const adtypep = VN_CAST(dtypep, UnpackArrayDType)) { if (initarp->defaultp()) { - puts("for (int __Vi=0; __Vi<" + cvtToStr(adtypep->elementsConst())); + puts("for (int __Vi = 0; __Vi < " + cvtToStr(adtypep->elementsConst())); puts("; ++__Vi) {\n"); emitSetVarConstant(varNameProtected + "[__Vi]", VN_AS(initarp->defaultp(), Const)); puts("}\n"); @@ -677,7 +677,7 @@ string EmitCFunc::emitVarResetRecurse(const AstVar* varp, const string& varNameP UASSERT_OBJ(adtypep->hi() >= adtypep->lo(), varp, "Should have swapped msb & lsb earlier."); const string ivar = string("__Vi") + cvtToStr(depth); - const string pre = ("for (int " + ivar + "=" + cvtToStr(0) + "; " + ivar + "<" + const string pre = ("for (int " + ivar + " = " + cvtToStr(0) + "; " + ivar + " < " + cvtToStr(adtypep->elementsConst()) + "; ++" + ivar + ") {\n"); const string below = emitVarResetRecurse(varp, varNameProtected, adtypep->subDTypep(), depth + 1, suffix + "[" + ivar + "]"); diff --git a/src/V3EmitCFunc.h b/src/V3EmitCFunc.h index 28cd0b9e8..166dd60de 100644 --- a/src/V3EmitCFunc.h +++ b/src/V3EmitCFunc.h @@ -240,7 +240,7 @@ public: // "+" in the debug indicates a print from the model puts("VL_DEBUG_IF(VL_DBG_MSGF(\"+ "); - for (int i = 0; i < m_modp->level(); ++i) { puts(" "); } + for (int i = 0; i < m_modp->level(); ++i) puts(" "); puts(prefixNameProtect(m_modp)); puts(nodep->isLoose() ? "__" : "::"); puts(nodep->nameProtect() + "\\n\"); );\n"); diff --git a/src/V3EmitCImp.cpp b/src/V3EmitCImp.cpp index 0c0cdd379..05749c259 100644 --- a/src/V3EmitCImp.cpp +++ b/src/V3EmitCImp.cpp @@ -393,8 +393,8 @@ class EmitCImp final : EmitCFunc { UASSERT_OBJ(arrayp->hi() >= arrayp->lo(), varp, "Should have swapped msb & lsb earlier."); const string ivar = string("__Vi") + cvtToStr(vecnum); - puts("for (int __Vi" + cvtToStr(vecnum) + "=" + cvtToStr(0)); - puts("; " + ivar + "<" + cvtToStr(arrayp->elementsConst())); + puts("for (int __Vi" + cvtToStr(vecnum) + " = " + cvtToStr(0)); + puts("; " + ivar + " < " + cvtToStr(arrayp->elementsConst())); puts("; ++" + ivar + ") {\n"); elementp = arrayp->subDTypep()->skipRefp(); } @@ -407,8 +407,8 @@ class EmitCImp final : EmitCFunc { && !(basicp && basicp->keyword() == VBasicDTypeKwd::STRING)) { const int vecnum = vects++; const string ivar = string("__Vi") + cvtToStr(vecnum); - puts("for (int __Vi" + cvtToStr(vecnum) + "=" + cvtToStr(0)); - puts("; " + ivar + "<" + cvtToStr(elementp->widthWords())); + puts("for (int __Vi" + cvtToStr(vecnum) + " = " + cvtToStr(0)); + puts("; " + ivar + " < " + cvtToStr(elementp->widthWords())); puts("; ++" + ivar + ") {\n"); } puts("os" + op + varp->nameProtect()); diff --git a/src/V3EmitCModel.cpp b/src/V3EmitCModel.cpp index f3425caff..517d2618c 100644 --- a/src/V3EmitCModel.cpp +++ b/src/V3EmitCModel.cpp @@ -303,9 +303,9 @@ class EmitCModel final : public EmitCFunc { UASSERT_OBJ(arrayp->hi() >= arrayp->lo(), varp, "Should have swapped msb & lsb earlier."); const string ivar = std::string{"__Vi"} + cvtToStr(vecnum); - puts("for (int __Vi" + cvtToStr(vecnum) + "=" + puts("for (int __Vi" + cvtToStr(vecnum) + " = " + cvtToStr(arrayp->lo())); - puts("; " + ivar + "<=" + cvtToStr(arrayp->hi())); + puts("; " + ivar + " <= " + cvtToStr(arrayp->hi())); puts("; ++" + ivar + ") {\n"); } puts("sensitive << " + varp->nameProtect()); diff --git a/src/V3EmitCSyms.cpp b/src/V3EmitCSyms.cpp index 752fe77c6..6dcfab8b1 100644 --- a/src/V3EmitCSyms.cpp +++ b/src/V3EmitCSyms.cpp @@ -866,7 +866,7 @@ void EmitCSyms::emitSymImp() { if (v3Global.dpi()) { m_ofpBase->puts("// Setup export functions\n"); - m_ofpBase->puts("for (int __Vfinal=0; __Vfinal<2; __Vfinal++) {\n"); + m_ofpBase->puts("for (int __Vfinal = 0; __Vfinal < 2; ++__Vfinal) {\n"); for (auto it = m_scopeFuncs.begin(); it != m_scopeFuncs.end(); ++it) { AstScopeName* const scopep = it->second.m_scopep; AstCFunc* const funcp = it->second.m_cfuncp;