[AST] Temporarily turn off another assert

`scopeLifetimeParamIndices` sometimes ends up non-null but empty, which causes crashes in module deserialization, blocking cross-module work on nonescapable types.

rdar://147765187

Author: lorentey

include/swift/AST/LifetimeDependence.h

@@ -220,13 +220,16 @@ class LifetimeDependenceInfo {
         targetIndex(targetIndex) {
     assert(this->isImmortal() || inheritLifetimeParamIndices ||
            scopeLifetimeParamIndices);
-    // FIXME: This assert can trigger when Optional/Result support ~Escapable use (rdar://147765187)
+    // FIXME: These asserts can trigger when Optional/Result support ~Escapable use (rdar://147765187)
     // assert(!inheritLifetimeParamIndices ||
     //        !inheritLifetimeParamIndices->isEmpty());
+    // assert(!scopeLifetimeParamIndices || !scopeLifetimeParamIndices->isEmpty());
     if (inheritLifetimeParamIndices && inheritLifetimeParamIndices->isEmpty()) {
       inheritLifetimeParamIndices = nullptr;
     }
-    assert(!scopeLifetimeParamIndices || !scopeLifetimeParamIndices->isEmpty());
+    if (scopeLifetimeParamIndices && scopeLifetimeParamIndices->isEmpty()) {
+      scopeLifetimeParamIndices = nullptr;
+    }
     assert((!addressableParamIndices
             || !conditionallyAddressableParamIndices
             || conditionallyAddressableParamIndices->isDisjointWith(