Rename COMPlus_ to DOTNET_ for IntMultipliy.csproj test #77321
Conversation
ghost
added
the
area-CodeGen-coreclr
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsIt looks like we use
|
|
/azp run runtime-coreclr jitstress, runtime-coreclr libraries-jitstress |
|
Azure Pipelines successfully started running 2 pipeline(s). |
| <CLRTestEnvironmentVariable Include="COMPlus_TieredCompilation" Value="0" /> | ||
| <CLRTestEnvironmentVariable Include="COMPlus_JITMinOpts" Value="0" /> | ||
| <CLRTestEnvironmentVariable Include="DOTNET_TieredCompilation" Value="0" /> | ||
| <CLRTestEnvironmentVariable Include="DOTNET_JITMinOpts" Value="0" /> |
There was a problem hiding this comment.
Are these needed? it seems to be default for tests
There was a problem hiding this comment.
Yes, since it will force JTMinOpts to always be 0 even when the test CI leg sets JITMinOpts to 1. We haven't done enough work to try to exclude certain tests based on the environment variables they rely on.
There was a problem hiding this comment.
It's still a little odd to me, COMPlus_ is an accepted prefix to the environment variables, so why doesn't the old alias work to force min-opts?
There was a problem hiding this comment.
Because there are two of the same env variables at play? What happens if "DOTNET_JitMinOpts=1" and "COMPlus_JItMinOpts=0" are set at the same time?
There was a problem hiding this comment.
Ah I see, so the problem is happening due to both of them being set to different values. That makes sense.
It looks like we use
DOTNET_JITMinOptsnow instead ofCOMPlus_JITMinOptsso this should resolve: #77299