From 901f1f0cafd0bbe79e394fae834a65895472f789 Mon Sep 17 00:00:00 2001 From: Andy Ayers Date: Wed, 21 Feb 2024 14:16:55 -0800 Subject: [PATCH] JIT: test the new cse policy in jit-experimental Add some minimal CI testing for this new CSE heuristic. Remove testing of cross-block assertion prop, since it's now on by default. Contributes to #92915. --- eng/pipelines/common/templates/runtimes/run-test-job.yml | 2 +- eng/pipelines/libraries/run-test-job.yml | 2 +- src/tests/Common/testenvironment.proj | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/eng/pipelines/common/templates/runtimes/run-test-job.yml b/eng/pipelines/common/templates/runtimes/run-test-job.yml index a2e13dca489deb..75a2881f5fa2c0 100644 --- a/eng/pipelines/common/templates/runtimes/run-test-job.yml +++ b/eng/pipelines/common/templates/runtimes/run-test-job.yml @@ -585,7 +585,7 @@ jobs: - jitobjectstackallocation - jitphysicalpromotion_only - jitphysicalpromotion_full - - jitcrossblocklocalassertionprop + - jitrlcse ${{ if in(parameters.testGroup, 'jit-cfg') }}: scenarios: - jitcfg diff --git a/eng/pipelines/libraries/run-test-job.yml b/eng/pipelines/libraries/run-test-job.yml index 5c68b4377ee143..8027b6387ac31a 100644 --- a/eng/pipelines/libraries/run-test-job.yml +++ b/eng/pipelines/libraries/run-test-job.yml @@ -242,7 +242,7 @@ jobs: - jitosr_stress_random - syntheticpgo - syntheticpgo_blend - - jitcrossblocklocalassertionprop + - jitrlcse - ${{ if eq(parameters.SuperPmiCollect, true) }}: - template: /eng/pipelines/libraries/superpmi-postprocess-step.yml diff --git a/src/tests/Common/testenvironment.proj b/src/tests/Common/testenvironment.proj index d1bc1d1a94c427..a2f8dfd4e641cd 100644 --- a/src/tests/Common/testenvironment.proj +++ b/src/tests/Common/testenvironment.proj @@ -81,7 +81,7 @@ RunningIlasmRoundTrip; DOTNET_JitSynthesizeCounts; DOTNET_JitCheckSynthesizedCounts; - DOTNET_JitEnableCrossBlockLocalAssertionProp + DOTNET_JitRLCSEGreedy; @@ -224,7 +224,6 @@ - @@ -240,6 +239,7 @@ +