RFR (M): 8149415: [AArch64] implement JVMCI CodeInstaller
tobias.hartmann at oracle.com
Thu Feb 11 13:10:21 UTC 2016
On 11.02.2016 01:34, Christian Thalinger wrote:
> Damn. A last-minute change broke the build:
> 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.
>> 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.
>>> On 2/9/16 2:39 PM, Christian Thalinger wrote:
>>>> 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.
More information about the hotspot-compiler-dev