Skip to content

Commit

Permalink
finish merge main
Browse files Browse the repository at this point in the history
  • Loading branch information
jjaworska committed Jan 20, 2025
1 parent 0879cfc commit dbb7244
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import cacophony.controlflow.generation.generateLayoutOfVirtualRegisters
import cacophony.semantic.analysis.AnalyzedFunction
import cacophony.semantic.analysis.EscapeAnalysisResult
import cacophony.semantic.analysis.VariablesMap
import cacophony.semantic.syntaxtree.Definition
import cacophony.semantic.syntaxtree.FunctionalExpression
import kotlin.math.max

Expand Down Expand Up @@ -60,6 +61,11 @@ abstract class CallableHandlerImpl(
)
}

override fun getVariableFromDefinition(varDef: Definition): Variable =
variablesMap.definitions.getOrElse(varDef) {
throw IllegalArgumentException("Variable $varDef have not been defined inside function $function")
}

override fun getAnalyzedFunction() = analyzedFunction

protected fun allocateVariables() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import cacophony.controlflow.generation.*
import cacophony.semantic.analysis.AnalyzedFunction
import cacophony.semantic.analysis.EscapeAnalysisResult
import cacophony.semantic.analysis.VariablesMap
import cacophony.semantic.syntaxtree.Definition
import cacophony.semantic.syntaxtree.Definition.FunctionDefinition

class FunctionHandlerImpl(
Expand Down Expand Up @@ -84,11 +83,6 @@ class FunctionHandlerImpl(
callerFunction.generateAccessToFramePointer(ancestorFunctionHandlers.first())
}

override fun getVariableFromDefinition(varDef: Definition): Variable =
variablesMap.definitions.getOrElse(varDef) {
throw IllegalArgumentException("Variable $varDef have not been defined inside function $function")
}

override fun getFlattenedArguments(): List<CFGNode> =
function.arguments
.map { variablesMap.definitions[it]!! }
Expand Down

0 comments on commit dbb7244

Please sign in to comment.