Address final review feedback: update docs, add dependency, sort properties

Co-authored-by: rolfbjarne <249268+rolfbjarne@users.noreply.github.com>

Author: Copilot

docs/building-apps/build-properties.md

@@ -287,12 +287,11 @@ This computed instruction set is then passed to crossgen2 via the `--instruction
 
 Set this property to `false` to disable automatic instruction set computation and use crossgen2's default behavior.
 
-Default: `true` (when `PublishReadyToRun` is enabled)
+Default: `true`
 
 Example:
 ```xml
 <PropertyGroup>
-  <PublishReadyToRun>true</PublishReadyToRun>
   <ComputeInstructionSetForReadyToRun>false</ComputeInstructionSetForReadyToRun>
 </PropertyGroup>
 ```

dotnet/targets/Microsoft.Sdk.R2R.targets

@@ -9,6 +9,7 @@
 	-->
 	<Target Name="_ComputeInstructionSetForCrossgen2"
 		Condition="'$(PublishReadyToRun)' == 'true' And '$(ComputeInstructionSetForReadyToRun)' != 'false'"
+		DependsOnTargets="_DetectSdkLocations"
 		BeforeTargets="ResolveReadyToRunCompilers">
 		<ComputeInstructionSet
 			RuntimeIdentifier="$(RuntimeIdentifier)"

msbuild/Xamarin.MacDev.Tasks/Tasks/ComputeInstructionSet.cs

@@ -16,10 +16,10 @@ public class ComputeInstructionSet : XamarinTask {
 		#region Inputs
 
 		[Required]
-		public string SupportedOSPlatformVersion { get; set; } = "";
+		public string RuntimeIdentifier { get; set; } = "";
 
 		[Required]
-		public string RuntimeIdentifier { get; set; } = "";
+		public string SupportedOSPlatformVersion { get; set; } = "";
 
 		#endregion