Enable async testing in CI (#121857)

With #121814 merged we should be able to start running these in the CI
for native AOT.

Author: MichalStrehovsky

eng/pipelines/coreclr/runtime-nativeaot-outerloop.yml

@@ -37,6 +37,7 @@ schedules:
 
 variables:
   - template: /eng/pipelines/common/variables.yml
+  - template: /eng/pipelines/helix-platforms.yml
 
 extends:
   template:  /eng/pipelines/common/templates/pipeline-with-resources.yml

eng/pipelines/runtime.yml

@@ -605,7 +605,7 @@ extends:
               - template: /eng/pipelines/coreclr/nativeaot-post-build-steps.yml
                 parameters:
                   creator: dotnet-bot
-                  testBuildArgs: 'nativeaot tree ";nativeaot;Loader;Interop;" /p:BuildNativeAotFrameworkObjects=true'
+                  testBuildArgs: 'nativeaot tree ";nativeaot;Loader;Interop;async;" /p:BuildNativeAotFrameworkObjects=true'
                   liveLibrariesBuildConfig: Release
                   testRunNamePrefixSuffix: NativeAOT_$(_BuildConfig)
             extraVariablesTemplates:

src/coreclr/tools/Common/Compiler/DependencyAnalysis/MethodReadOnlyDataNode.cs

@@ -26,7 +26,10 @@ public override bool ShouldSkipEmittingObjectNode(NodeFactory factory)
         }
 #endif
 
-        public override ObjectNodeSection GetSection(NodeFactory factory) => ObjectNodeSection.ReadOnlyDataSection;
+        // TODO: (async) This should stay RO everywhere: https://github.com/dotnet/runtime/issues/121871
+        public override ObjectNodeSection GetSection(NodeFactory factory)
+            => factory.Target.IsWindows ? ObjectNodeSection.ReadOnlyDataSection : ObjectNodeSection.DataSection;
+
         public override bool StaticDependenciesAreComputed => _data != null;
 
         public void AppendMangledName(NameMangler nameMangler, Utf8StringBuilder sb)

src/tests/async/Directory.Build.targets

@@ -1,12 +1,20 @@
 <Project>
   <PropertyGroup>
     <!-- disable building for configurations that do not yet support runtime async. -->
-    <DisableProjectBuild Condition="'$(RuntimeFlavor)' == 'mono' or '$(TestBuildMode)' == 'nativeaot' or '$(TargetArchitecture)' == 'wasm'">true</DisableProjectBuild>
+    <DisableProjectBuild Condition="'$(RuntimeFlavor)' == 'mono' or '$(TargetArchitecture)' == 'wasm'">true</DisableProjectBuild>
+  </PropertyGroup>
+
+  <PropertyGroup>
+    <!-- https://github.com/dotnet/runtime/issues/121872 -->
+    <DisableProjectBuild Condition="'$(TestBuildMode)' == 'nativeaot' and '$(TargetArchitecture)' == 'x86'">true</DisableProjectBuild>
+
+    <!-- https://github.com/dotnet/runtime/issues/121878 -->
+    <DisableProjectBuild Condition="'$(TestBuildMode)' == 'nativeaot' and '$(TargetArchitecture)' == 'arm64'">true</DisableProjectBuild>
   </PropertyGroup>
 
   <PropertyGroup>
     <!-- runtime async testing in main repo NYI -->
-    <DisableProjectBuild>true</DisableProjectBuild>
+    <DisableProjectBuild Condition="'$(TestBuildMode)' != 'nativeaot'">true</DisableProjectBuild>
   </PropertyGroup>
 
   <Import Project="$([MSBuild]::GetPathOfFileAbove(Directory.Build.targets, $(MSBuildThisFileDirectory)..))" />