Merge pull request #4731 from MarcusDenker/OpalSemanticCleanup

OpalSemanticCleanup

Author: MarcusDenker

src/OpalCompiler-Core/OCASTClosureAnalyzer.class.st

@@ -68,6 +68,6 @@ OCASTClosureAnalyzer >> visitVariableNode: aVariableNode [
 	aVariableNode adaptToSemanticNode.
 	aVariableNode isTemp ifFalse: [^self].
 	var := self lookupAndFixBinding: aVariableNode.
-	var isTempVectorTemp ifTrue: [scope addCopyingTempToAllScopesUpToDefVector: var vectorName].
+	var isTempVectorTemp ifTrue: [scope addCopyingTempToAllScopesUpToDefVectorNamed: var vectorName].
 	var isCopying ifTrue: [scope addCopyingTempToAllScopesUpToDefTemp: var].
 ]

src/OpalCompiler-Core/OCAbstractMethodScope.class.st

@@ -14,15 +14,6 @@ Class {
 	#category : #'OpalCompiler-Core-Semantics'
 }
 
-{ #category : #'temp vars - copying' }
-OCAbstractMethodScope >> addCopiedTempToAllScopesUpToDefVector: aVariable [
-		
-	(self hasCopyingTempNamed: aVariable name)
-		ifFalse: [self addCopyingTemp: aVariable ].
-	self tempVectorName = aVariable name ifTrue: [^ self].
-	^ self outerScope addCopiedTempToAllScopesUpToDefVector: aVariable.
-]
-
 { #category : #'temp vars - copying' }
 OCAbstractMethodScope >> addCopyingTemp: aTempVar [
 	copiedVars add: (OCCopyingTempVariable new
@@ -54,11 +45,11 @@ OCAbstractMethodScope >> addCopyingTempToAllScopesUpToDefTemp: aVar [
 ]
 
 { #category : #'temp vars - copying' }
-OCAbstractMethodScope >> addCopyingTempToAllScopesUpToDefVector: aName [
+OCAbstractMethodScope >> addCopyingTempToAllScopesUpToDefVectorNamed: aName [
 
 	(self hasCopyingTempNamed: aName) ifFalse: [self addCopyingTempNamed: aName].
 	self tempVectorName = aName ifTrue: [^ self].
-	^ self outerScope addCopyingTempToAllScopesUpToDefVector: aName.
+	^ self outerScope addCopyingTempToAllScopesUpToDefVectorNamed: aName.
 ]
 
 { #category : #'temp vars' }