[Pkg] The Trunk: Compiler-dtl.303.mcz

commits at source.squeak.org commits at source.squeak.org
Wed Jul 8 20:53:28 UTC 2015


Chris Muller uploaded a new version of Compiler to project The Trunk:
http://source.squeak.org/trunk/Compiler-dtl.303.mcz

==================== Summary ====================

Name: Compiler-dtl.303
Author: dtl
Time: 7 July 2015, 7:27:15.092 pm
UUID: 762ba9fa-1e47-4c91-af95-d673d8c6706d
Ancestors: Compiler-topa.302

Revert two methods from Compiler-eem.300 because they work on a stack interpreter but not on a context interpreter (VM crash).

=============== Diff against Compiler-topa.302 ===============

Item was changed:
  ----- Method: BlockNode>>emitCodeForEvaluatedClosureValue:encoder: (in category 'code generation (closures)') -----
  emitCodeForEvaluatedClosureValue: stack encoder: encoder
  	| position |
  	position := stack position.
  	stack position: arguments size + copiedValues size.
+ 	temporaries size timesRepeat:
+ 		[NodeNil emitCodeForValue: stack encoder: encoder].
- 	encoder genPushNClosureTemps: temporaries size.
  	self
  		reindexingLocalsDo: [self emitCodeForEvaluatedValue: stack encoder: encoder]
  		encoder: encoder.
  	self returns ifFalse:
  		[encoder genReturnTopToCaller.
  		 pc := encoder methodStreamPosition].
  	stack position: position!

Item was changed:
  ----- Method: BlockNode>>sizeCodeForEvaluatedClosureValue: (in category 'code generation (closures)') -----
  sizeCodeForEvaluatedClosureValue: encoder
  	"The closure value primitives push the arguments and the copied values.
  	 The compiler guarantees that any copied values come before all local temps.
  	 So on closure activation we only need to push nils for the remaining temporaries."
+ 	^temporaries size * (NodeNil sizeCodeForValue: encoder)
- 	^(encoder sizePushNClosureTemps: temporaries size)
  	+ (self
  		reindexingLocalsDo: [self sizeCodeForEvaluatedValue: encoder]
  		encoder: nil "don't store temps yet")
  	+ (self returns ifTrue: [0] ifFalse: [encoder sizeReturnTopToCaller])!



More information about the Packages mailing list