From d2e85ccd2b930c7c16b8382b8ec417706b707f1b Mon Sep 17 00:00:00 2001 From: Ryszard Rozak Date: Tue, 14 Nov 2023 12:39:03 +0100 Subject: [PATCH] Don't set accesses of arguments in V3Task Signed-off-by: Ryszard Rozak --- src/V3Task.cpp | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/V3Task.cpp b/src/V3Task.cpp index 6e5dcb5246..966ce42fdd 100644 --- a/src/V3Task.cpp +++ b/src/V3Task.cpp @@ -30,7 +30,6 @@ #include "V3Const.h" #include "V3EmitCBase.h" #include "V3Graph.h" -#include "V3LinkLValue.h" #include @@ -503,7 +502,6 @@ class TaskVisitor final : public VNVisitor { } } else if (portp->isInoutish()) { // if (debug() >= 9) pinp->dumpTree("-pinrsize- "); - V3LinkLValue::linkLValueSet(pinp); AstVarScope* const newvscp = createVarScope(portp, namePrefix + "__" + portp->shortName()); @@ -524,12 +522,6 @@ class TaskVisitor final : public VNVisitor { beginp->addNext(postassp); // if (debug() >= 9) beginp->dumpTreeAndNext(cout, "-pinrsize-out- "); } else if (portp->isWritable()) { - // Make output variables - // Correct lvalue; we didn't know when we linked - // This is slightly scary; are we sure no decisions were made - // before here based on this not being a lvalue? - // Doesn't seem so; V3Unknown uses it earlier, but works ok. - V3LinkLValue::linkLValueSet(pinp); // Even if it's referencing a varref, we still make a temporary // Else task(x,x,x) might produce incorrect results AstVarScope* const newvscp