getAndAddInt() breaks AOT compilation for AArch64

[Andrew Haley]

On 05/30/2018 10:43 AM, Andrew Haley wrote:
> On 05/30/2018 10:42 AM, Zhongwei Yao wrote:
>> And what shall I do if the patch is OK? Shall I create a pull request
>> in Graal for it? Or should we just wait for Daniel's full fix for it?
> 
> Let me do it.

This is all we need for correctness.

--- /local/graal/graal/compiler/src/org.graalvm.compiler.core.aarch64/src/org/graalvm/compiler/core/aarch64/AArch64LIRGenerator.java	2018-05-29 11:10:55.653614259 +0100
+++ src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.core.aarch64/src/org/graalvm/compiler/core/aarch64/AArch64LIRGenerator.java	2018-05-29 17:04:52.622429877 +0100
@@ -152,7 +152,7 @@
         Variable scratch1 = newVariable(kind);
         Variable scratch2 = newVariable(kind);

-        append(new AtomicReadAndAddOp((AArch64Kind) kind.getPlatformKind(), asAllocatable(result), asAllocatable(address), delta, asAllocatable(scratch1), asAllocatable(scratch2)));
+        append(new AtomicReadAndAddOp((AArch64Kind) kind.getPlatformKind(), asAllocatable(result), asAllocatable(address), asAllocatable(delta), asAllocatable(scratch1), asAllocatable(scratch2)));
         return result;
     }

I guess I don't have to create a pull request for this?

-- 
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671