Merge pull request #7959 from MarcusDenker/7955-ASTCache-can-cause-hu…

…ge-leaks

Do not add DoIts to the cache for now. fixes #7955

src/AST-Core/ASTCache.class.st

@@ -75,9 +75,12 @@ ASTCache class >> shutDown [
 
 { #category : #accessing }
 ASTCache >> at: aCompiledMethod [
-	^ self 
-		at: aCompiledMethod
-		ifAbsentPut: [ self class cacheMissStrategy getASTFor: aCompiledMethod ]
+	"for doit methods, the ast is stored in the method propery"
+	^ aCompiledMethod propertyAt: #ast ifAbsent: [ 
+		  self
+			  at: aCompiledMethod
+			  ifAbsentPut: [ 
+			  self class cacheMissStrategy getASTFor: aCompiledMethod ] ]
 ]
 
 { #category : #initialization }

src/OpalCompiler-Core/OpalCompiler.class.st

@@ -360,7 +360,8 @@ OpalCompiler >> compileDoit [
 	ast := RBParser parseMethod: ast formattedCode.
 	ast compilationContext: self compilationContext.
 	self doSemanticAnalysis.
-	ASTCache default at: cm  put: ast. 
+	"we store the ast in the doit method so that the debugger can read variables from the requestor"
+	cm propertyAt: #ast put: ast.
 	^cm
 ]