TR_RelocationRecordClassAddress is relocated incorrectly on platforms that use an ordered pair

[dsouzai]

In , the new constantpool address is acquired via

openj9/runtime/compiler/runtime/RelocationRecord.cpp

Lines 2217 to 2218 in 46a6d02

	uintptr_t oldValue = (uintptr_t) reloTarget->loadAddress(reloLocationHigh, reloLocationLow);
	uintptr_t newConstantPool = computeNewConstantPool(reloRuntime, reloTarget, oldValue);

However, because the oldValue read from the loaded code is an old J9Class pointer, the code that calculates the new constantpool pointer in

openj9/runtime/compiler/runtime/RelocationRecord.cpp

Line 1537 in 46a6d02

uintptr_t methodCP = oldValue - oldCPBase + (uintptr_t)J9_CP_FROM_METHOD((J9Method *)ramMethod);

is going to end up yielding an invalid pointer. I believe the reason there hasn't been any issues is because this path is likely disabled.

[github-actions]

Issue Number: 15696
Status: Closed
Actual Components: comp:jit, comp:jit:aot
Actual Assignees: No one :(
PR Assignees: luke-li-2003