Merge pull request #70120 from hamishknight/remove-assert-5.10

Author: hamishknight

lib/SIL/IR/SILProfiler.cpp

@@ -781,7 +781,8 @@ struct CoverageMapping : public ASTWalker {
   /// Subtract \c Expr from \c Node's counter.
   void subtractFromCounter(ASTNode Node, CounterExpr Expr) {
     auto Counter = getCounter(Node);
-    assert(!Counter.isZero() && "Cannot create a negative counter");
+    // FIXME: This assertion ought to be restored (rdar://100470244)
+    // assert(!Counter.isZero() && "Cannot create a negative counter");
     assignCounter(Node,
                   CounterExpr::Sub(Counter, std::move(Expr), CounterBuilder));
   }

test/Profiler/rdar118896974.swift

@@ -0,0 +1,10 @@
+// RUN: %target-swift-frontend -profile-generate -profile-coverage-mapping -emit-ir %s
+
+// rdar://118896974 - Make sure we don't crash.
+func foo() {
+  do {
+    return
+  } catch {
+    return
+  }
+}