-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial Graal native-image creation and Benchmarks #1194
base: develop
Are you sure you want to change the base?
Conversation
Generate changelog in
|
|
||
jdk(23) { | ||
distribution = 'graalvm-ce' | ||
jdkVersion = '23.0.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
> Task :setupJdks
+ Options
| Auto-detection: Disabled
| Auto-download: Disabled
+ Amazon Corretto JDK 11.0.25+9-LTS
| Location: /Users/crogoz/.gradle/gradle-jdks/amazon-corretto-11.0.25.9.1
| Language Version: 11
| Vendor: Amazon Corretto
| Architecture: aarch64
| Is JDK: true
| Detected by: Gradle property 'org.gradle.java.installations.paths'
+ Amazon Corretto JDK 17.0.13+11-LTS
| Location: /Users/crogoz/.gradle/gradle-jdks/amazon-corretto-17.0.13.11.1
| Language Version: 17
| Vendor: Amazon Corretto
| Architecture: aarch64
| Is JDK: true
| Detected by: Current JVM
+ Amazon Corretto JDK 21.0.5+11-LTS
| Location: /Users/crogoz/.gradle/gradle-jdks/amazon-corretto-21.0.5.11.1
| Language Version: 21
| Vendor: Amazon Corretto
| Architecture: aarch64
| Is JDK: true
| Detected by: Gradle property 'org.gradle.java.installations.paths'
+ GraalVM Community JDK 23.0.1+11-jvmci-b01
| Location: /Users/crogoz/.gradle/gradle-jdks/graalvm-ce-23.0.1
| Language Version: 23
| Vendor: GraalVM Community
| Architecture: aarch64
| Is JDK: true
| Detected by: Gradle property 'org.gradle.java.installations.paths'
gradle/gradle-jdks-functions.sh
Outdated
@@ -141,11 +141,11 @@ install_and_setup_jdks() { | |||
case "$distribution_url" in | |||
*.zip) | |||
distribution_name=${distribution_url##*/} | |||
curl -C - "$distribution_url" -o "$distribution_name" | |||
curl -L -C - "$distribution_url" -o "$distribution_name" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just ran ./gradlew setupJdks
- same below
@@ -0,0 +1,18 @@ | |||
[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these were generated via https://pl.ntr/2qm
fallback = false | ||
sharedLibrary = false | ||
configurationFileDirectories.from(file('src/main/resources/META-INF/native-image')) | ||
buildArgs.addAll("-H:ConfigurationFileDirectories=${projectDir}/src/main/resources/META-INF/native-image", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same args as in google-java-format: https://github.com/google/google-java-format/blob/eb0b7682f15ec6a2bd03d12b53bdf951dd924266/core/pom.xml#L282-L295
|
||
@Measurement(iterations = 2, time = 2) | ||
@Warmup(iterations = 2, time = 4) | ||
public class BenchmarkMultiFiles { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before this PR
After this PR
==COMMIT_MSG==
Graal native-image and Benchmarks
==COMMIT_MSG==
Possible downsides?