From 9e3d27a0e52cd98a374fdfc68ffbd51705f1c6b1 Mon Sep 17 00:00:00 2001
From: hogoww <pierre_misse25@msn.com>
Date: Thu, 23 Dec 2021 17:10:09 +0100
Subject: [PATCH] Mutant #563, Reverting [ Replace #ifFalse: with #ifTrue: ] on
 method [ markWeaklingsAndMarkAndFireEphemerons ] KILLED by 1/10 test cases.

---
 .../VMMaker/SpurMemoryManager.class.st        | 32 ++++++++-----------
 1 file changed, 14 insertions(+), 18 deletions(-)

diff --git a/smalltalksrc/VMMaker/SpurMemoryManager.class.st b/smalltalksrc/VMMaker/SpurMemoryManager.class.st
index 72fded0259..d6cae225bb 100644
--- a/smalltalksrc/VMMaker/SpurMemoryManager.class.st
+++ b/smalltalksrc/VMMaker/SpurMemoryManager.class.st
@@ -8456,34 +8456,30 @@ SpurMemoryManager >> markStack: anOop [
 
 { #category : #'gc - global' }
 SpurMemoryManager >> markWeaklingsAndMarkAndFireEphemerons [
-
 	"After the initial scan-mark is complete ephemerons can be processed.
 	 Weaklings have accumulated on the weaklingStack, but more may be
 	 uncovered during ephemeron processing.  So trace the strong slots
 	 of the weaklings, and as ephemerons are processed ensure any newly
 	 reached weaklings are also traced."
-
-	<inline: false>
 	| numTracedWeaklings |
+	<inline: false>
 	numTracedWeaklings := 0.
-	[ 
-	coInterpreter markAndTraceUntracedReachableStackPages.
-	coInterpreter markAndTraceMachineCodeOfMarkedMethods.
-	"Make sure all reached weaklings have their strong slots traced before firing ephemerons..."
-	[ 
-	numTracedWeaklings := self markAndTraceWeaklingsFrom:
-		                      numTracedWeaklings.
-	(self sizeOfObjStack: weaklingStack) > numTracedWeaklings ] whileTrue.
-	self noUnscannedEphemerons ifTrue: [ 
-		coInterpreter
+	[coInterpreter markAndTraceUntracedReachableStackPages.
+	 coInterpreter markAndTraceMachineCodeOfMarkedMethods.
+	 "Make sure all reached weaklings have their strong slots traced before firing ephemerons..."
+	 [numTracedWeaklings := self markAndTraceWeaklingsFrom: numTracedWeaklings.
+	  (self sizeOfObjStack: weaklingStack) > numTracedWeaklings] whileTrue.
+	 self noUnscannedEphemerons ifTrue:
+		[coInterpreter
 			markAndTraceUntracedReachableStackPages;
-			markAndTraceMachineCodeOfMarkedMethods;
+	 		markAndTraceMachineCodeOfMarkedMethods;
 			freeUntracedStackPages;
 			freeUnmarkedMachineCode.
-		^ self ].
-	self markInactiveEphemerons ifTrue: [ 
-		self fireAllUnscannedEphemerons ].
-	self markAllUnscannedEphemerons ] repeat
+		 ^self].
+	 self markInactiveEphemerons ifFalse:
+		[self fireAllUnscannedEphemerons].
+	 self markAllUnscannedEphemerons]
+		repeat
 ]
 
 { #category : #'header format' }