Change offset check for multiple of 8 (0x7)

- the "value" field in AtomicLlong is of type "long", so it should be aligned to 8-byte boundaries to be used in the atomics intrinsics. Changed offset verification to check for a multiple of 8 instead ofa multiple of 4. Signed-off-by: Matthew Hall <[email protected]>
eclipse-openj9 · Oct 17, 2024 · 7aa7c9c · 7aa7c9c
1 parent 96ba473
commit 7aa7c9c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/runtime/compiler/z/codegen/J9CodeGenerator.cpp b/runtime/compiler/z/codegen/J9CodeGenerator.cpp
@@ -3267,8 +3267,8 @@ J9::Z::CodeGenerator::checkFieldAlignmentForAtomicLong()
    int32_t fieldNameLen = 5;
    const char * fieldSig = "J";
    int32_t fieldSigLen = 1;
-   int32_t intOrBoolOffset = self()->fe()->getObjectHeaderSizeInBytes() + self()->fej9()->getInstanceFieldOffset(classBlock, fieldName, fieldNameLen, fieldSig, fieldSigLen);
-   return (intOrBoolOffset & 0x3) == 0;
+   int32_t longOffset = self()->fe()->getObjectHeaderSizeInBytes() + self()->fej9()->getInstanceFieldOffset(classBlock, fieldName, fieldNameLen, fieldSig, fieldSigLen);
+   return (longOffset & 0x7) == 0;
    }