From 9130174cbef365c84b48002d6899f3ea9ec55df4 Mon Sep 17 00:00:00 2001
From: Joseph Tremoulet <jotrem@microsoft.com>
Date: Tue, 22 Nov 2016 08:59:29 -0800
Subject: Remove bogus "fast path" in unroller branch update

BasicBlocks whose `bbJumpKind` doesn't make use of `bbJumpDest` may have
arbitrary garbage in their `bbJumpDest` field, so remove the code from
loop unrolling that was expecting such blocks to have null `bbJumpDest`s,
and instead always defer to the `optCopyBlkDest`/`optRedirectBlock`
helpers that check the jump kind before checking the jump dest.

Fixes #8231.
---
 .../Regression/JitBlue/GitHub_8231/GitHub_8231.cs  | 62 ++++++++++++++++++++++
 .../JitBlue/GitHub_8231/GitHub_8231.csproj         | 39 ++++++++++++++
 2 files changed, 101 insertions(+)
 create mode 100644 tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.cs
 create mode 100644 tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.csproj

(limited to 'tests')

diff --git a/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.cs b/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.cs
new file mode 100644
index 0000000000..4d2ec7628a
--- /dev/null
+++ b/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.cs
@@ -0,0 +1,62 @@
+// 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.
+
+using System;
+using System.Numerics;
+using System.Runtime.CompilerServices;
+
+namespace N
+{
+    public static class C
+    {
+        // This is a regression test for a failure in loop unrolling when
+        // the unrolled loop contains a switch statement.
+        [MethodImpl(MethodImplOptions.NoInlining)]
+        static int Test()
+        {
+            int s = 0;
+
+            // Loop to some Vector<T>.Count to trigger unrolling.
+            for (int i = 0; i < Vector<int>.Count; i++)
+            {
+                // Loop contains switch; the bug was that the clones
+                // of the switch were all sharing its BBswtDesc instead
+                // of getting their own, so updates to their jump targets
+                // were incorrectly shared.
+                switch (i)
+                {
+                    case 1: s += 4; break;
+                    case 2: s += 2; break;
+                    case 3: s += i; break;
+                }
+            }
+
+            return s;
+        }
+
+        public static int Main(string[] args)
+        {
+            int result = Test();
+
+            // Expected result is a function of Vector<int>.Count.
+            int expected;
+            switch(Vector<int>.Count)
+            {
+                case 1:
+                    expected = 4;
+                    break;
+                case 2:
+                    expected = 6;
+                    break;
+                default:
+                    expected = 9;
+                    break;
+            }
+
+            // Return 100 on success (result == expected), other
+            // values on failure.
+            return 100 + result - expected;
+        }
+   }
+}
diff --git a/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.csproj b/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.csproj
new file mode 100644
index 0000000000..844638fe1b
--- /dev/null
+++ b/tests/src/JIT/Regression/JitBlue/GitHub_8231/GitHub_8231.csproj
@@ -0,0 +1,39 @@
+<?xml version="1.0" encoding="utf-8"?>
+<Project ToolsVersion="12.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), dir.props))\dir.props" />
+  <PropertyGroup>
+    <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
+    <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
+    <SchemaVersion>2.0</SchemaVersion>
+    <ProjectGuid>{2649FAFE-07BF-4F93-8120-BA9A69285ABB}</ProjectGuid>
+    <OutputType>Exe</OutputType>
+    <AppDesignerFolder>Properties</AppDesignerFolder>
+    <FileAlignment>512</FileAlignment>
+    <ProjectTypeGuids>{786C830F-07A1-408B-BD7F-6EE04809D6DB};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>
+    <ReferencePath>$(ProgramFiles)\Common Files\microsoft shared\VSTT\11.0\UITestExtensionPackages</ReferencePath>
+    <SolutionDir Condition="$(SolutionDir) == '' Or $(SolutionDir) == '*Undefined*'">..\..\</SolutionDir>
+  </PropertyGroup>
+  <!-- Default configurations to help VS understand the configurations -->
+  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
+  </PropertyGroup>
+  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
+  </PropertyGroup>
+  <ItemGroup>
+    <CodeAnalysisDependentAssemblyPaths Condition=" '$(VS100COMNTOOLS)' != '' " Include="$(VS100COMNTOOLS)..\IDE\PrivateAssemblies">
+      <Visible>False</Visible>
+    </CodeAnalysisDependentAssemblyPaths>
+  </ItemGroup>
+  <ItemGroup>
+    <Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
+  </ItemGroup>
+  <ItemGroup>
+    <Compile Include="GitHub_8231.cs" />
+  </ItemGroup>
+  <PropertyGroup>
+    <ProjectJson>$(JitPackagesConfigFileDirectory)benchmark\project.json</ProjectJson>
+    <ProjectLockJson>$(JitPackagesConfigFileDirectory)benchmark\project.lock.json</ProjectLockJson>
+  </PropertyGroup>
+  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), dir.targets))\dir.targets" />
+  <PropertyGroup Condition=" '$(MsBuildProjectDirOverride)' != '' ">
+  </PropertyGroup>
+</Project>
-- 
cgit v1.2.3