8310540: G1: Verification should use raw oop decode functions

Thomas Schatzl · Thomas Schatzl · commit 9397094692ca · 2023-06-26T07:49:51.000Z
Reviewed-by: ayang, kbarrett
diff --git a/src/hotspot/share/gc/g1/heapRegion.cpp b/src/hotspot/share/gc/g1/heapRegion.cpp
@@ -618,7 +618,7 @@ class G1VerifyLiveAndRemSetClosure : public BasicOopIterateClosure {
     if (CompressedOops::is_null(heap_oop)) {
       return;
     }
-    oop obj = CompressedOops::decode_not_null(heap_oop);
+    oop obj = CompressedOops::decode_raw_not_null(heap_oop);
 
     LiveChecker<T> live_check(this, _containing_obj, p, obj, _vo);
     if (live_check.failed()) {

Original file line number	Diff line number	Diff line change
`@@ -618,7 +618,7 @@ class G1VerifyLiveAndRemSetClosure : public BasicOopIterateClosure {`
`618`	`618`	`if (CompressedOops::is_null(heap_oop)) {`
`619`	`619`	`return;`
`620`	`620`	`}`
`621`		`- oop obj = CompressedOops::decode_not_null(heap_oop);`
	`621`	`+ oop obj = CompressedOops::decode_raw_not_null(heap_oop);`
`622`	`622`
`623`	`623`	`LiveChecker<T> live_check(this, _containing_obj, p, obj, _vo);`
`624`	`624`	`if (live_check.failed()) {`