AArch64: Fix String.hashCode() implementation

This commit fixes `inlineStringHashCode` so that `savedLengthReg` is copied to `lengthReg` when the reference count of`lengthNode` is > 1. Signed-off-by: Akira Saitoh <[email protected]>
eclipse-openj9 · Nov 22, 2023 · 38f5e41 · 38f5e41
1 parent d5e17dd
commit 38f5e41
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/runtime/compiler/aarch64/codegen/J9TreeEvaluator.cpp b/runtime/compiler/aarch64/codegen/J9TreeEvaluator.cpp
@@ -6097,6 +6097,12 @@ static TR::Register *inlineStringHashCode(TR::Node *node, bool isCompressed, TR:
    TR_ARM64ScratchRegisterManager *srm = cg->generateScratchRegisterManager(12);
    TR::Register *dataAddrReg = (arrayNode->getReferenceCount() > 1) ? srm->findOrCreateScratchRegister() : arrayReg;
    TR::Register *lengthReg = (lengthNode->getReferenceCount() > 1) ? srm->findOrCreateScratchRegister() : savedLengthReg;
+
+   if (lengthNode->getReferenceCount() > 1)
+      {
+      generateMovInstruction(cg, node, lengthReg, savedLengthReg, false);
+      }
+
    TR::Compilation *comp = cg->comp();
    if (comp->getOptions()->enableDebugCounters())
       {