8185505: AArch64: Port AOT (original) (raw)

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Apr 17 16:53:49 UTC 2018

On 4/16/18 2:03 AM, Andrew Haley wrote:

On 04/16/2018 09:53 AM, Ningsheng Jian wrote:

I can see both the jdk patch and Graal PR contain changes of AArch64Assembler.java and AArch64MacroAssembler.java, but the changes looks somewhat different. How will they be synced? The changes to Graal in JDK do no more than allow OpenJDK to build. They are not called by anything, and will disappear at the next Graal import.

I noticed that in make/hotspot/lib/JvmFeatures.gmk line ~144, there's:

_JVMEXCLUDEFILES += _ compiledICaotx8664.cpp Do you want to add compiledICaotaarch64.cpp to that list? I don't really know what this does, so I have no idea.

It was done when we did not support/build AOT on Win and MacOS. It also work when AOT is disabled in build: configure --disable-aot We have to exclude to AOT files to avoid build failure because they do not have #ifdef INCLUDE_AOT.

I think compiledIC_aot_aarch64.cpp should be added to exclude list too. And you can test you code with --disable-aot

Vladimir

I also found that immutablePIC and its getters/setters are in the INCLUDEAOT block, but some of their uses are not:

src/hotspot/cpu/aarch64/compiledICaarch64.cpp: 61 if (cbuf.immutablePIC()) { src/hotspot/share/jvmci/jvmciCodeInstaller.cpp: 594 buffer.setimmutablePIC(immutablepiccompilation); 628 buffer.setimmutablePIC(immutablepiccompilation); Thank you. For the sake of consistency I will change it.

More information about the hotspot-dev mailing list