Change offset verification to check for multiple of 8 (0x7)

- the "value" field in AtomicLong 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 · c5bab58 · c5bab58
1 parent 96ba473
commit c5bab58
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;
  }