Don't skip CoreRt tests since the specific version was updated.

Also added `TaskNoAllocate` in `MemoryDiagnoserSurvivedIsAccurate` test.

Author: timcassell

tests/BenchmarkDotNet.IntegrationTests/MemoryDiagnoserTests.cs

@@ -82,15 +82,17 @@ public class AccurateSurvived
             public byte[] bytes64;
             public Task<int> task;
 
-            [GlobalSetup(Targets = new string[] { nameof(EightBytesArrayNoAllocate), nameof(SixtyFourBytesArrayNoAllocate) })]
+            [GlobalSetup(Targets = new string[] { nameof(EightBytesArrayNoAllocate), nameof(SixtyFourBytesArrayNoAllocate), nameof(TaskNoAllocate) })]
             public void SetupNoAllocate()
             {
                 bytes8 = new byte[8];
                 bytes64 = new byte[64];
+                task = Task.FromResult(default(int));
             }
 
             [Benchmark] public byte[] EightBytesArrayNoAllocate() => bytes8;
             [Benchmark] public byte[] SixtyFourBytesArrayNoAllocate() => bytes64;
+            [Benchmark] public Task<int> TaskNoAllocate() => task;
 
 
             [Benchmark] public void EightBytesArraySurvive() => bytes8 = new byte[8];
@@ -118,6 +120,7 @@ public void MemoryDiagnoserSurvivedIsAccurate(IToolchain toolchain)
 
                 { nameof(AccurateSurvived.EightBytesArrayNoAllocate), 0 },
                 { nameof(AccurateSurvived.SixtyFourBytesArrayNoAllocate), 0 },
+                { nameof(AccurateSurvived.TaskNoAllocate), 0 },
 
                 { nameof(AccurateSurvived.EightBytesArraySurvive), 8 + objectAllocationOverhead + arraySizeOverhead },
                 { nameof(AccurateSurvived.SixtyFourBytesArraySurvive), 64 + objectAllocationOverhead + arraySizeOverhead },
@@ -401,9 +404,7 @@ private void AssertSurvived(IToolchain toolchain, Type benchmarkType, Dictionary
         {
             // Core has survived memory measurement problems.
             // See https://github.com/dotnet/runtime/issues/45446
-            if (toolchain is CsProjCoreToolchain
-                || (toolchain.IsInProcess && RuntimeInformation.IsNetCore)
-                || toolchain is CoreRtToolchain) // CoreRt actually does measure accurately in a normal benchmark run, but doesn't with the specific version used in these tests.
+            if (toolchain is CsProjCoreToolchain || (toolchain.IsInProcess && RuntimeInformation.IsNetCore))
                 return;
 
             var config = CreateConfig(toolchain, MemoryDiagnoser.WithSurvived);
@@ -413,6 +414,11 @@ private void AssertSurvived(IToolchain toolchain, Type benchmarkType, Dictionary
 
             foreach (var benchmarkSurvivedValidator in benchmarkSurvivedValidators)
             {
+                // CoreRT is missing some of the CoreCLR threading/task related perf improvements, so sizeof(Task<int>) calculated for CoreCLR < sizeof(Task<int>) on CoreRT
+                // see https://github.com/dotnet/corert/issues/5705
+                if (benchmarkSurvivedValidator.Key == nameof(AccurateSurvived.AllocateTaskSurvive) && toolchain is CoreRtToolchain)
+                    continue;
+
                 var survivedBenchmarks = benchmarks.BenchmarksCases.Where(benchmark => benchmark.Descriptor.WorkloadMethodDisplayInfo == benchmarkSurvivedValidator.Key).ToArray();
 
                 foreach (var benchmark in survivedBenchmarks)