Merge pull request #6946 from BruceForstall/Fix3597

Change RyuJIT/x86 GC encoder to handle LSRA flexibility

diff --git a/src/jit/gcencode.cpp b/src/jit/gcencode.cpp
index fbd51bb..a9f4a90 100644 (file)
--- a/src/jit/gcencode.cpp
+++ b/src/jit/gcencode.cpp
@@ -1777,7 +1777,11 @@ size_t GCInfo::gcMakeRegPtrTable(BYTE* dest, int mask, const InfoHdr& header, un
                      */
 
                     /* Has this argument been enregistered? */
+#ifndef LEGACY_BACKEND
+                    if (!varDsc->lvOnFrame)
+#else  // LEGACY_BACKEND
                     if (varDsc->lvRegister)
+#endif // LEGACY_BACKEND
                     {
                         /* if a CEE_JMP has been used, then we need to report all the arguments
                            even if they are enregistered, since we will be using this value

diff --git a/tests/issues.targets b/tests/issues.targets
index ce73216..869dfbd 100644 (file)
--- a/tests/issues.targets
+++ b/tests/issues.targets
@@ -266,9 +266,6 @@
     <!-- The following x86 failures only occur with RyuJIT/x86 -->
 
     <ItemGroup Condition="'$(XunitTestBinBase)' != '' and '$(BuildArch)' == 'x86'">
-        <ExcludeList Include="$(XunitTestBinBase)\JIT\Performance\CodeQuality\Serialization\Serialize\Serialize.cmd">
-             <Issue>3597</Issue>
-        </ExcludeList>
         <ExcludeList Include="$(XunitTestBinBase)\JIT\Methodical\MDArray\basics\stringarr_cs_do\stringarr_cs_do.cmd">
              <Issue>4844</Issue>
         </ExcludeList>