RFR (M): 8149415: [AArch64] implement JVMCI CodeInstaller

[Tobias Hartmann]

On 11.02.2016 01:34, Christian Thalinger wrote:
> Damn.  A last-minute change broke the build:
> 
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/diff/b038c3bea5a4/src/cpu/aarch64/vm/jvmciCodeInstaller_aarch64.cpp#l1.61
> 
> But since JPRT doesn’t build open AArch64 we didn’t catch it.  Maybe we should pass the thread to CodeInstaller::pd_patch_DataSectionReference as well to have the option to throw an exception.  I will send a review tomorrow.  Sorry for the inconvenience.

Okay, I'll not push this to main then.

Best,
Tobias

> 
>> On Feb 10, 2016, at 9:58 AM, Christian Thalinger <christian.thalinger at oracle.com <mailto:christian.thalinger at oracle.com>> wrote:
>>
>>
>>> On Feb 10, 2016, at 8:54 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com <mailto:vladimir.kozlov at oracle.com>> wrote:
>>>
>>> Why you need #ifdef _LP64 in jvmciCodeInstaller_aarch64.cpp? It is only 64 bit VM.
>>
>> Haha, yeah, good point.  Copy-paste :-)  I’ll remove it.
>>
>>> Otherwise looks good.
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 2/9/16 2:39 PM, Christian Thalinger wrote:
>>>> https://bugs.openjdk.java.net/browse/JDK-8149415
>>>> http://cr.openjdk.java.net/~twisti/8149415/webrev.01/
>>>>
>>>> This is a first cut of the CodeInstaller implementation for AArch64.  One missing part is compressed oops support but
>>>> it’s not a big issue because Graal doesn’t support it yet either.
>>>>
>>>> The patch also includes changes to AMD64 and SPARC’s RegisterConfig to use an array for the reserved registers.  The
>>>> code before was too hard to understand.
>>>>
>>
>