Fix the open-existential remover to not die on the OVEs in collection upcasts.

Author: rjmccall

lib/Sema/CSDiag.cpp

@@ -3150,6 +3150,17 @@ static void eraseOpenedExistentials(Expr *&expr) {
                "didn't see this OVE in a containing OpenExistentialExpr?");
         return { true, value->second };
       }
+
+      // Handle collection upcasts specially so that we don't blow up on
+      // their embedded OVEs.
+      if (auto CDE = dyn_cast<CollectionUpcastConversionExpr>(expr)) {
+        if (auto result = CDE->getSubExpr()->walk(*this)) {
+          CDE->setSubExpr(result);
+          return { false, CDE };
+        } else {
+          return { true, CDE };
+        }
+      }
 
       return { true, expr };
     }

validation-test/Sema/type_checker_crashers/rdar27815848.swift renamed to validation-test/Sema/type_checker_crashers_fixed/rdar27815848.swift

@@ -1,4 +1,4 @@
-// RUN: not --crash %target-swift-frontend %s -parse
+// RUN: not %target-swift-frontend %s -parse
 
 class C {}
 class D : C {}