Quantcast

Re: Package resolution when compiling jdk.aot against jdk.vm.ci

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Package resolution when compiling jdk.aot against jdk.vm.ci

Doug Simon @ Oracle
Adding in hotspot-compiler-dev as they are more knowledge on building the aot code.

> On 13 Feb 2017, at 12:49, Andrew Haley <[hidden email]> wrote:
>
> I have this patch:
>
> diff --git a/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/ELFMacroAssembler.java b/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/ELFMacroAssembler.java
> --- a/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/ELFMacroAssembler.java
> +++ b/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/ELFMacroAssembler.java
> @@ -24,8 +24,10 @@
> package jdk.tools.jaotc;
>
> import jdk.tools.jaotc.CompiledMethodInfo.StubInformation;
> +import jdk.tools.jaotc.aarch64.AArch64ELFMacroAssembler;
> import jdk.tools.jaotc.amd64.AMD64ELFMacroAssembler;
>
> +import jdk.vm.ci.aarch64.AArch64;
> import jdk.vm.ci.amd64.AMD64;
> import jdk.vm.ci.code.Architecture;
> import jdk.vm.ci.code.TargetDescription;
> @@ -36,6 +38,8 @@
>         Architecture architecture = target.arch;
>         if (architecture instanceof AMD64) {
>             return new AMD64ELFMacroAssembler(target);
> +        } else if (architecture instanceof AArch64) {
> +            return new AArch64ELFMacroAssembler(target);
>         } else {
>             throw new InternalError("Unsupported architecture " + architecture);
>         }
>
> but I get
>
> Compiling jdk.aot
> /home/aph/jdk10/hs/hotspot/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/ELFMacroAssembler.java:30: error: package jdk.vm.ci.aarch64 does not exist
> import jdk.vm.ci.aarch64.AArch64;
>                        ^
> I presume that there is something I should be doing to enable javac to
> see the jdk.vm.ci.aarch64 package.  hotspot/make/gensrc/Gensrc-jdk.vm.compiler.gmk
> already exports jdk.vm.ci.aarch64.
>
> Any ideas?
>
> Thanks,
>
> Andrew.

Loading...