[GR-50383] Backport to 23.0: HeapImpl.dirtyAllReferencesOf must be no-op for epsilon GC.

medoussboug · medoussboug · commit 2cb5ba395c9f · 2023-12-10T13:49:18.000Z
PullRequest: graal/16268
diff --git a/substratevm/src/com.oracle.svm.core.genscavenge/src/com/oracle/svm/core/genscavenge/HeapImpl.java b/substratevm/src/com.oracle.svm.core.genscavenge/src/com/oracle/svm/core/genscavenge/HeapImpl.java
@@ -689,7 +689,7 @@ public long getThreadAllocatedMemory(IsolateThread thread) {
     @Override
     @Uninterruptible(reason = "Ensure that no GC can occur between modification of the object and this call.", callerMustBe = true)
     public void dirtyAllReferencesOf(Object obj) {
-        if (obj != null) {
+        if (SubstrateOptions.useRememberedSet() && obj != null) {
             ForcedSerialPostWriteBarrier.force(OffsetAddressNode.address(obj, 0), false);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -689,7 +689,7 @@ public long getThreadAllocatedMemory(IsolateThread thread) {`
`689`	`689`	`@Override`
`690`	`690`	`@Uninterruptible(reason = "Ensure that no GC can occur between modification of the object and this call.", callerMustBe = true)`
`691`	`691`	`public void dirtyAllReferencesOf(Object obj) {`
`692`		`- if (obj != null) {`
	`692`	`+ if (SubstrateOptions.useRememberedSet() && obj != null) {`
`693`	`693`	`ForcedSerialPostWriteBarrier.force(OffsetAddressNode.address(obj, 0), false);`
`694`	`694`	`}`
`695`	`695`	`}`