[graalvm-dev] Formal dependency on JVMCI

[Chris Seaton]

> On 30 Sep 2019, at 12:34, Doug Simon <doug.simon at oracle.com> wrote:
> 
> Hi Chris,
> 
> The source truth for the required JVMCI version is in JVMCIVersionCheck.java <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_blob_03217a31ee2f8e3fb2b75e9d7b14220d645e5955_compiler_src_org.graalvm.compiler.hotspot_src_org_graalvm_compiler_hotspot_JVMCIVersionCheck.java&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=-Z80nHT_pwcjrKSPnUteBRLeNgODjh3t4Kh8zNEZahQ&e= >.

Ok I’ll use that as my single source of truth, thanks.

How come it doesn’t match what’s used in CI, though? The third digit is the build, right? This shows b2 and b3 being used in the same commit.

https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_blob_1d9d33960b860a97fa3c488789f007f1229cc390_compiler_src_org.graalvm.compiler.hotspot_src_org_graalvm_compiler_hotspot_JVMCIVersionCheck.java&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=bD2dE7dwTk_mG1Ndh1t7t-fzNWTwRZubbKS4mkkzdoQ&e=  <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_blob_1d9d33960b860a97fa3c488789f007f1229cc390_compiler_src_org.graalvm.compiler.hotspot_src_org_graalvm_compiler_hotspot_JVMCIVersionCheck.java&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=bD2dE7dwTk_mG1Ndh1t7t-fzNWTwRZubbKS4mkkzdoQ&e= >
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_blob_1d9d33960b860a97fa3c488789f007f1229cc390_common.hocon-23L10&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=-UkHDNJFQfIfzcwAa7s_U6_XJjHzdzBAgMEcbuprr3Y&e=  <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_blob_1d9d33960b860a97fa3c488789f007f1229cc390_common.hocon-23L10&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=-UkHDNJFQfIfzcwAa7s_U6_XJjHzdzBAgMEcbuprr3Y&e= >
> Where else and in what format would you like that to be declared? Note that with JDK 11 about to be supported, a single JVMCI version number will denote 2 different base JDKs.

All other version dependency information is in the chain of suite.py files. JVMC is I think the only exception, so seems like it should be in there instead, and perhaps JVMCIVersionCheck.java built from it during compilation?

Ultimately, the fact that the JVMCI version is repeated tens of times all over the code base (lots of CI files, that Java file, etc), and that it’s being scraped with regular expressions from these files, gives me the idea that something about how it is being declared is not right.

> -Doug
> 
>> On 30 Sep 2019, at 12:48, Chris Seaton <chris.seaton at shopify.com <mailto:chris.seaton at shopify.com>> wrote:
>> 
>> Could we get some kind of declarative dependency on a JVMCI version from GraalVM and languages? As I understand it, Graal depends on a particular JVMCI version. But which version? I have to go scraping around to figure it out from clues like the CI configuration when I do my own builds.
>> 
>> Would it make sense to declare a clear dependency on a JVMCI version in our mx suites like we do on other components?
>> 
>> Examples of issues this causes includes the number of places it’s repeated and the various formats it’s in.
>> 
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_commit_d8f623a1957ee378c4d20cefabc0950878037620&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=tX8e3aX6CE746lg8MZV1LfsM21z6dj90oAqBqmH1gvw&e=  <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_graal_commit_d8f623a1957ee378c4d20cefabc0950878037620&d=DwMFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=5Fpxezl2ceVCO_hT1GeSBkUl-njbxmg2gJsRbxLRrWM&s=T6fWXJ_7oIW8j_KkbxC15oEaErQ4X6Yr5WUYSQKr1YU&e=>
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_truffleruby_commit_0808ffc9e65a98c38a252c87f917e34cff584b28&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=0s9BWWy1LB-abIbY6xRUtpj4IUjmz-Z-UQt2qvRubdQ&e=  <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_truffleruby_commit_0808ffc9e65a98c38a252c87f917e34cff584b28&d=DwMFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=5Fpxezl2ceVCO_hT1GeSBkUl-njbxmg2gJsRbxLRrWM&s=G6Pm_bwKdM6pZmGIE65bZS7t_9-Geuwr9mCflnmfgww&e=>
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_truffleruby_commit_e79e9d57d665bd15d77d80c0b53b547bd0eeff73&d=DwIFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=OHHkrQy-Zw_Y_f0n8VgainNtnTt-Nw3bz5R-lG9Mdno&s=FyRg-woBg3FhRW7yDz2v4a0jle1IytIeyu6cE6az5tE&e=  <https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oracle_truffleruby_commit_e79e9d57d665bd15d77d80c0b53b547bd0eeff73&d=DwMFaQ&c=RoP1YumCXCgaWHvlZYR8PZh8Bv7qIrMUB65eapI_JnE&r=P1aEbpQqX18a0ssutGCeizsWEVvvJcWvSeSxdI9X3-o&m=5Fpxezl2ceVCO_hT1GeSBkUl-njbxmg2gJsRbxLRrWM&s=ijCPXws2Igq6u0uayVUxHLoKXHNzG0soxwFdX_D6ziA&e=>
>> 
>> Regards,
>> 
>> Chris
>> _______________________________________________
>> GraalVM-Dev mailing list
>> GraalVM-Dev at oss.oracle.com <mailto:GraalVM-Dev at oss.oracle.com>
>> https://oss.oracle.com/mailman/listinfo/graalvm-dev
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://oss.oracle.com/pipermail/graalvm-dev/attachments/20190930/e3c4be0c/attachment-0001.html