Fix IsSafeToContained. (#14268)

When AliasSet adds a node, it should also add all its nodes that are
executed inside it (marked as contained).

diff --git a/src/jit/sideeffects.cpp b/src/jit/sideeffects.cpp
index e87abd3..931ee6f 100644 (file)
--- a/src/jit/sideeffects.cpp
+++ b/src/jit/sideeffects.cpp
@@ -263,6 +263,10 @@ void AliasSet::AddNode(Compiler* compiler, GenTree* node)
 
             m_lclVarReads.Add(compiler, lclNum);
         }
+        if (!operand->IsArgPlaceHolderNode() && operand->isContained())
+        {
+            AddNode(compiler, operand);
+        }
         return GenTree::VisitResult::Continue;
     });
 

diff --git a/tests/issues.targets b/tests/issues.targets
index af76950..fce3f8a 100644 (file)
--- a/tests/issues.targets
+++ b/tests/issues.targets
@@ -184,9 +184,6 @@
         <ExcludeList Include="$(XunitTestBinBase)\Loader\classloader\TypeGeneratorTests\TypeGeneratorTest683\Generated683\*">
             <Issue>6707</Issue>
         </ExcludeList>
-        <ExcludeList Include="$(XunitTestBinBase)\JIT\Regression\JitBlue\GitHub_13910\GitHub_13910\GitHub_13910.cmd">
-            <Issue>13910</Issue>
-        </ExcludeList>
     </ItemGroup>
 
     <!-- The following are x86 failures -->