From: Sergey Andreenko <seandree@microsoft.com>
Date: Tue, 3 Oct 2017 20:38:02 +0000 (-0700)
Subject: Fix IsSafeToContained. (#14268)
X-Git-Tag: accepted/tizen/base/20180629.140029~996
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=32bb7e16d5b741c12ed2388d44b5d736ae0ab240;p=platform%2Fupstream%2Fcoreclr.git

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
--- 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
--- 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 -->