Skip to content

Commit

Permalink
Deprecate inlineBeforeAnalysis option
Browse files Browse the repository at this point in the history
`inlineBeforeAnalysis` is always enabled starting from GraalVM
21.3 (which is the minimum supported version)
  • Loading branch information
zakkak committed Sep 8, 2022
1 parent 5677c2f commit edc490d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,10 @@ public class NativeConfig {

/**
* If {@code -H:+InlineBeforeAnalysis} flag will be added to the native-image run
*
* @deprecated inlineBeforeAnalysis is always enabled starting from GraalVM 21.3.
*/
@Deprecated
@ConfigItem(defaultValue = "true")
public boolean inlineBeforeAnalysis;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -798,12 +798,7 @@ public NativeImageInvokerInfo build() {
if (!protocols.isEmpty()) {
nativeImageArgs.add("-H:EnableURLProtocols=" + String.join(",", protocols));
}
if (inlineBeforeAnalysis) {
if (graalVMVersion.isOlderThan(GraalVM.Version.VERSION_21_3)) {
// Enabled by default in GraalVM >= 21.3
nativeImageArgs.add("-H:+InlineBeforeAnalysis");
}
} else {
if (!inlineBeforeAnalysis) {
nativeImageArgs.add("-H:-InlineBeforeAnalysis");
}
if (!noPIE.isEmpty()) {
Expand Down

0 comments on commit edc490d

Please sign in to comment.