Skip to content

Commit 672df39

Browse files
author
Marjana Gligorijević
committed
[GR-38667] Tests for Analysis Context Sensitivity
PullRequest: graal/11764
2 parents 3487e67 + 6376934 commit 672df39

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

vm/mx.vm/mx_vm_benchmark.py

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ def _configure_from_name(self, config_name):
226226

227227
# This defines the allowed config names for NativeImageVM. The ones registered will be available via --jvm-config
228228
rule = r'^(?P<native_architecture>native-architecture-)?(?P<string_inlining>string-inlining-)?(?P<gate>gate-)?(?P<quickbuild>quickbuild-)?(?P<gc>g1gc-)?(?P<llvm>llvm-)?(?P<pgo>pgo-|pgo-hotspot-|pgo-ctx-insens-)?(?P<inliner>aot-inline-|iterative-|inline-explored-)?' \
229-
r'(?P<analysis_context_sensitivity>insens-|allocsens-|1obj-)?(?P<no_inlining_before_analysis>no-inline-)?(?P<edition>ce-|ee-)?$'
229+
r'(?P<analysis_context_sensitivity>insens-|allocsens-|1obj-|2obj1h-|3obj2h-|4obj3h-)?(?P<no_inlining_before_analysis>no-inline-)?(?P<edition>ce-|ee-)?$'
230230

231231
mx.logv("== Registering configuration: {}".format(config_name))
232232
match_name = "{}-".format(config_name) # adding trailing dash to simplify the regex
@@ -309,15 +309,12 @@ def _configure_from_name(self, config_name):
309309

310310
if matching.group("analysis_context_sensitivity") is not None:
311311
context_sensitivity = matching.group("analysis_context_sensitivity")[:-1]
312-
if context_sensitivity == "insens":
313-
mx.logv("analysis context sensitivity 'insens' is enabled for {}".format(config_name))
314-
self.analysis_context_sensitivity = "insens"
315-
elif context_sensitivity == "allocsens":
316-
mx.logv("analysis context sensitivity 'allocsens' is enabled for {}".format(config_name))
317-
self.analysis_context_sensitivity = "allocsens"
318-
elif context_sensitivity == "1obj":
319-
mx.logv("analysis context sensitivity '1obj' is enabled for {}".format(config_name))
320-
self.analysis_context_sensitivity = "_1obj"
312+
if context_sensitivity in ["insens", "allocsens"]:
313+
mx.logv("analysis context sensitivity {} is enabled for {}".format(context_sensitivity, config_name))
314+
self.analysis_context_sensitivity = context_sensitivity
315+
elif context_sensitivity in ["1obj", "2obj1h", "3obj2h", "4obj3h"]:
316+
mx.logv("analysis context sensitivity {} is enabled for {}".format(context_sensitivity, config_name))
317+
self.analysis_context_sensitivity = "_{}".format(context_sensitivity)
321318
else:
322319
mx.abort("Unknown analysis context sensitivity: {}".format(context_sensitivity))
323320

@@ -1273,11 +1270,14 @@ def register_graalvm_vms():
12731270
_gu_vm_registry.add_vm(GuVm(host_vm_name, 'default', [], []), _suite, 10)
12741271

12751272
# Inlining before analysis is done by default
1276-
analysis_context_sensitivity = ['insens', 'allocsens', '1obj']
1273+
analysis_context_sensitivity = ['insens', 'allocsens', '1obj', '2obj1h', '3obj2h', '4obj3h']
12771274
analysis_context_sensitivity_no_inline = ['{}-{}'.format(analysis_component, 'no-inline') for analysis_component in analysis_context_sensitivity]
1275+
pgo_aot_inline_context_sensitivity = ['{}-{}'.format('pgo-aot-inline', analysis_component) for analysis_component in analysis_context_sensitivity]
1276+
pgo_aot_inline_context_sensitivity += ['{}-{}'.format('pgo-aot-inline', analysis_component) for analysis_component in analysis_context_sensitivity_no_inline]
1277+
12781278
for short_name, config_suffix in [('niee', 'ee'), ('ni', 'ce')]:
12791279
if any(component.short_name == short_name for component in mx_sdk_vm_impl.registered_graalvm_components(stage1=False)):
1280-
for main_config in ['default', 'gate', 'llvm', 'native-architecture'] + analysis_context_sensitivity + analysis_context_sensitivity_no_inline:
1280+
for main_config in ['default', 'gate', 'llvm', 'native-architecture'] + analysis_context_sensitivity + analysis_context_sensitivity_no_inline + pgo_aot_inline_context_sensitivity:
12811281
final_config_name = '{}-{}'.format(main_config, config_suffix)
12821282
mx_benchmark.add_java_vm(NativeImageVM('native-image', final_config_name), _suite, 10)
12831283
break

0 commit comments

Comments
 (0)