RFR: 8177845: Need a mechanism to load Graal
mandy.chung at oracle.com
Thu Apr 27 22:04:41 UTC 2017
> On Apr 27, 2017, at 2:30 PM, Doug Simon <doug.simon at oracle.com> wrote:
>> On 27 Apr 2017, at 23:24, Mandy Chung <mandy.chung at oracle.com> wrote:
>>> On Apr 27, 2017, at 7:47 AM, Doug Simon <doug.simon at oracle.com> wrote:
>>>  http://cr.openjdk.java.net/~dnsimon/8177845.02
>> I reviewed the top repo and jdk repo change. For hotspot change,
>> I reviewed jdk.vm.ci.services/* and jdk.internal.vm.ci module-info.java.
>> Reading the original jdk.internal.vm.ci module-info.java ,
>> jdk.aot only accesses a small set of packages. e.g. it does not
>> access jdk.vm.ci.aarch64, jdk.vm.ci.code.stack, jdk.vm.ci.common, etc.
> I know but I agree with Vladimir that there's no harm in exporting everything to jdk.aot to simplify continued development of this tool as support for other platforms is added. As I learnt putting this patch together, missing these exports when they're actually needed leads to very confusing compiler error messages.
Indeed the compiler error message is very confusing and ought to be improved (JDK-8179359).
I have no issue to export these packages to jdk.aot, as you agree to do.
More information about the hotspot-dev