Skip to content

Commit

Permalink
Revert isIndexableDataAddrPresent to UDATA
Browse files Browse the repository at this point in the history
Since #20819 did not
resolved perf regression, and introduced DDR incompatibility since the
last release/split, isIndexableDataAddrPresent is reverted back to type
UDATA that was the type at the last split.

Performance regression will be dealt later in a separate change.

Signed-off-by: Aleksandar Micic <[email protected]>
  • Loading branch information
Aleksandar Micic authored and Aleksandar Micic committed Jan 14, 2025
1 parent 30f423a commit 56513fb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion debugtools/DDR_VM/src/com/ibm/j9ddr/AuxFieldInfo29.dat
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ J9JavaVM.hiddenLockwordFieldShape = required
J9JavaVM.identityHashData = required
J9JavaVM.impdep1PC = U8*
J9JavaVM.initialMethods = required
J9JavaVM.isIndexableDataAddrPresent = U_32
J9JavaVM.isIndexableDataAddrPresent = UDATA
J9JavaVM.intReflectClass = required
J9JavaVM.j2seVersion = required
J9JavaVM.j9ras = required
Expand Down
4 changes: 2 additions & 2 deletions runtime/oti/j9nonbuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -5540,7 +5540,7 @@ typedef struct J9VMThread {
UDATA contiguousIndexableHeaderSize;
UDATA discontiguousIndexableHeaderSize;
#if defined(J9VM_ENV_DATA64)
U_32 isIndexableDataAddrPresent;
UDATA isIndexableDataAddrPresent;
U_32 isVirtualLargeObjectHeapEnabled;
#endif /* defined(J9VM_ENV_DATA64) */
void* gpInfo;
Expand Down Expand Up @@ -6105,7 +6105,7 @@ typedef struct J9JavaVM {
UDATA contiguousIndexableHeaderSize;
UDATA discontiguousIndexableHeaderSize;
#if defined(J9VM_ENV_DATA64)
U_32 isIndexableDataAddrPresent;
UDATA isIndexableDataAddrPresent;
U_32 isVirtualLargeObjectHeapEnabled;
U_32 isIndexableDualHeaderShapeEnabled;
#endif /* defined(J9VM_ENV_DATA64) */
Expand Down

0 comments on commit 56513fb

Please sign in to comment.