Move [SkipLocalsInit] to shared .targets file

Author: Sergio0694

src/CommunityToolkit.Common/Properties/AssemblyInfo.cs

@@ -2,15 +2,9 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+#if !NETSTANDARD2_1_OR_GREATER
 using System.Runtime.CompilerServices;
 
-// We can suppress the .init flag for local variables for the entire module.
-// This doesn't affect the correctness of methods in this assembly, as none of them
-// are relying on the JIT ensuring that all local memory is zeroed out to work. Doing
-// this can provide some minor performance benefits, depending on the workload.
-[module: SkipLocalsInit]
-
 // We need to test the RuntimeHelpers polyfills on applicable runtimes
-#if !NETSTANDARD2_1_OR_GREATER
 [assembly: InternalsVisibleTo("CommunityToolkit.HighPerformance.UnitTests, publickey=002400000480000094000000060200000024000052534131000400000100010041753AF735AE6140C9508567666C51C6AB929806ADB0D210694B30AB142A060237BC741F9682E7D8D4310364B4BBA4EE89CC9D3D5CE7E5583587E8EA44DCA09977996582875E71FB54FA7B170798D853D5D8010B07219633BDB761D01AC924DA44576D6180CDCEAE537973982BB461C541541D58417A3794E34F45E6F2D129E2")]
 #endif

src/CommunityToolkit.Diagnostics/Properties/AssemblyInfo.cs

@@ -0,0 +1,27 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information.
+
+#if !NET6_0_OR_GREATER
+
+namespace System.Runtime.CompilerServices;
+
+/// <summary>
+/// Used to indicate to the compiler that the <c>.locals init</c> flag should not be set in method headers.
+/// </summary>
+/// <remarks>Internal copy from the BCL attribute.</remarks>
+[AttributeUsage(
+    AttributeTargets.Module |
+    AttributeTargets.Class |
+    AttributeTargets.Struct |
+    AttributeTargets.Interface |
+    AttributeTargets.Constructor |
+    AttributeTargets.Method |
+    AttributeTargets.Property |
+    AttributeTargets.Event,
+    Inherited = false)]
+internal sealed class SkipLocalsInitAttribute : Attribute
+{
+}
+
+#endif

src/CommunityToolkit.HighPerformance/Properties/AssemblyInfo.cs

@@ -33,6 +33,7 @@
   <ItemGroup>
     <Compile Include="$(MSBuildThisFileDirectory)Attributes\NotNullWhenAttribute.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)Attributes\NullabilityAttributesGenerator.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)Attributes\SkipLocalsInitAttribute.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)ComponentModel\INotifyPropertyChangedGenerator.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)ComponentModel\Models\AttributeInfo.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)ComponentModel\Models\INotifyPropertyChangedInfo.cs" />

src/CommunityToolkit.Mvvm.SourceGenerators/Attributes/SkipLocalsInitAttribute.cs

@@ -4,7 +4,4 @@
 
 using System.Runtime.CompilerServices;
 
-// See notes in CommunityToolkit.HighPerformance for more info
-[module: SkipLocalsInit]
-
 [assembly: InternalsVisibleTo("CommunityToolkit.Mvvm.Internals.UnitTests, PublicKey=002400000480000094000000060200000024000052534131000400000100010041753AF735AE6140C9508567666C51C6AB929806ADB0D210694B30AB142A060237BC741F9682E7D8D4310364B4BBA4EE89CC9D3D5CE7E5583587E8EA44DCA09977996582875E71FB54FA7B170798D853D5D8010B07219633BDB761D01AC924DA44576D6180CDCEAE537973982BB461C541541D58417A3794E34F45E6F2D129E2")]

src/CommunityToolkit.Mvvm.SourceGenerators/CommunityToolkit.Mvvm.SourceGenerators.projitems

@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="utf-8"?>
+<Project>
+  <Import Project="..\Directory.Build.targets" />
+
+  <!--
+    The following target has been ported from TerraFX.Interop.Windows.
+    See: https://github.com/terrafx/terrafx.interop.windows.
+    Using [module: SkipLocalsInit] suppresses the .init flag for local variables for the entire module.
+    This doesn't affect the correctness of methods in this assembly, as none of them are relying on the
+    JIT ensuring that all local memory is zeroed out to work. Doing this can provide some minor
+    performance benefits, depending on the workload.
+  -->
+  <PropertyGroup>
+    <GeneratedSkipLocalsInitFile Condition="'$(GeneratedSkipLocalsInitFile)' == ''">$(IntermediateOutputPath)$(MSBuildProjectName).SkipLocalsInit.g.cs</GeneratedSkipLocalsInitFile>
+    <GeneratedSkipLocalsInitFileLines>
+      <![CDATA[//------------------------------------------------------------------------------
+// <auto-generated>
+//     This code was generated by a tool.
+//
+//     Changes to this file may cause incorrect behavior and will be lost if
+//     the code is regenerated.
+// </auto-generated>
+//------------------------------------------------------------------------------
+
+[module: global::System.Runtime.CompilerServices.SkipLocalsInitAttribute]]]>
+    </GeneratedSkipLocalsInitFileLines>
+  </PropertyGroup>
+
+  <Target Name="GenerateSkipLocalsInit"
+          BeforeTargets="BeforeCompile;CoreCompile"
+          DependsOnTargets="PrepareForBuild"
+          Condition="'$(Language)' == 'C#'"
+          Inputs="$(MSBuildAllProjects)"
+          Outputs="$(GeneratedSkipLocalsInitFile)">
+
+    <!-- Write the file with the attribute -->
+    <WriteLinesToFile Lines="$(GeneratedSkipLocalsInitFileLines)" Overwrite="true" WriteOnlyWhenDifferent="true" File="$(GeneratedSkipLocalsInitFile)" />
+
+    <!-- Include the generated file in the list of files to compile -->
+    <ItemGroup>
+      <Compile Include="$(GeneratedSkipLocalsInitFile)" />
+    </ItemGroup>
+  </Target>
+</Project>