Skip to content

Commit f4e613f

Browse files
committed
[GR-55152] Remove PGO context insensitive config.
PullRequest: graal/19440
2 parents c51b8f9 + adcedcb commit f4e613f

File tree

1 file changed

+1
-10
lines changed

1 file changed

+1
-10
lines changed

vm/mx.vm/mx_vm_benchmark.py

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,6 @@ def __init__(self, name, config_name, extra_java_args=None, extra_launcher_args=
524524
self.pgo_instrumentation = False
525525
self.pgo_exclude_conditional = False
526526
self.pgo_sampler_only = False
527-
self.pgo_context_sensitive = True
528527
self.is_gate = False
529528
self.is_quickbuild = False
530529
self.use_string_inlining = False
@@ -585,9 +584,6 @@ def config_name(self):
585584
if self.is_llvm is True:
586585
config += ["llvm"]
587586
is_pgo_set = False
588-
if self.pgo_context_sensitive is False:
589-
config += ["pgo-ctx-insens"]
590-
is_pgo_set = True
591587
if self.pgo_sampler_only is True:
592588
config += ["pgo-sampler"]
593589
is_pgo_set = True
@@ -643,7 +639,7 @@ def _configure_from_name(self, config_name):
643639

644640
# This defines the allowed config names for NativeImageVM. The ones registered will be available via --jvm-config
645641
rule = r'^(?P<native_architecture>native-architecture-)?(?P<string_inlining>string-inlining-)?(?P<otw>otw-)?(?P<compacting_gc>compacting-gc-)?(?P<gate>gate-)?(?P<upx>upx-)?(?P<quickbuild>quickbuild-)?(?P<gc>g1gc-)?' \
646-
r'(?P<llvm>llvm-)?(?P<pgo>pgo-|pgo-ctx-insens-|pgo-sampler-)?(?P<inliner>inline-)?' \
642+
r'(?P<llvm>llvm-)?(?P<pgo>pgo-|pgo-sampler-)?(?P<inliner>inline-)?' \
647643
r'(?P<analysis_context_sensitivity>insens-|allocsens-|1obj-|2obj1h-|3obj2h-|4obj3h-)?(?P<no_inlining_before_analysis>no-inline-)?(?P<jdk_profiles>jdk-profiles-collect-|adopted-jdk-pgo-)?' \
648644
r'(?P<profile_inference>profile-inference-feature-extraction-|profile-inference-pgo-|profile-inference-debug-)?(?P<sampler>safepoint-sampler-|async-sampler-)?(?P<optimization_level>O0-|O1-|O2-|O3-|Os-)?(default-)?(?P<edition>ce-|ee-)?$'
649645

@@ -698,10 +694,6 @@ def _configure_from_name(self, config_name):
698694
if pgo_mode == "pgo":
699695
mx.logv(f"'pgo' is enabled for {config_name}")
700696
self.pgo_instrumentation = True
701-
elif pgo_mode == "pgo-ctx-insens":
702-
mx.logv(f"'pgo-ctx-insens' is enabled for {config_name}")
703-
self.pgo_instrumentation = True
704-
self.pgo_context_sensitive = False
705697
elif pgo_mode == "pgo-sampler":
706698
self.pgo_instrumentation = True
707699
self.pgo_sampler_only = True
@@ -1256,7 +1248,6 @@ def run_stage_instrument_run(self):
12561248
def run_stage_image(self):
12571249
executable_name_args = ['-o', self.config.final_image_name]
12581250
pgo_args = [f"--pgo={self.config.profile_path}"]
1259-
pgo_args += svm_experimental_options(['-H:' + ('+' if self.pgo_context_sensitive else '-') + 'PGOContextSensitivityEnabled'])
12601251
if self.adopted_jdk_pgo:
12611252
# choose appropriate profiles
12621253
jdk_version = mx_sdk_vm.get_jdk_version_for_profiles()

0 commit comments

Comments
 (0)