getAndAddInt() breaks AOT compilation for AArch64

[dean.long at oracle.com]

On 5/30/18 4:20 AM, Andrew Haley wrote:
> 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?
>

I believe a pull request is going to be needed to get it upstream into 
Graal, but I can include this patch in "JDK-8202670: Update Graal" to 
allow the update to proceed.

dl