From 52c14fb8d01ddd293a0710642aa1e7ef3a438687 Mon Sep 17 00:00:00 2001 From: Andy Ayers Date: Thu, 22 Feb 2024 19:59:01 -0800 Subject: [PATCH] JIT: test the new cse policy in jit-experimental (#98777) 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 28a421c105512..a063c2127ecc7 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 78f2d73e49f6f..f6f452199d627 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 7d0ba71411fab..e82110af21a8f 100644 --- a/src/tests/Common/testenvironment.proj +++ b/src/tests/Common/testenvironment.proj @@ -80,7 +80,7 @@ RunningIlasmRoundTrip; DOTNET_JitSynthesizeCounts; DOTNET_JitCheckSynthesizedCounts; - DOTNET_JitEnableCrossBlockLocalAssertionProp + DOTNET_JitRLCSEGreedy; @@ -223,7 +223,6 @@ - @@ -239,6 +238,7 @@ +