Use noteUseOfOpaqueTypeDescriptor instead of introducing and using markOpaqueTypeDescriptorEmitted

Author: aschwaighofer

lib/IRGen/GenDecl.cpp

@@ -1277,23 +1277,6 @@ void IRGenerator::noteUseOfFieldDescriptor(NominalTypeDecl *type) {
   LazyFieldDescriptors.push_back(type);
 }
 
-void IRGenerator::markOpaqueTypeDescriptorEmitted(OpaqueTypeDecl *opaque) {
-  if (!opaque)
-    return;
-
-  auto insertResult = LazyOpaqueTypes.try_emplace(opaque);
-  auto &entry = insertResult.first->second;
-  entry.IsDescriptorUsed = true;
-  entry.IsDescriptorEmitted = true;
-  LazyOpaqueTypeDescriptors.erase(
-    std::remove_if(LazyOpaqueTypeDescriptors.begin(),
-                   LazyOpaqueTypeDescriptors.end(),
-                   [&](OpaqueTypeDecl *entry) -> bool {
-                     return entry == opaque;
-                   }),
-    LazyOpaqueTypeDescriptors.end());
-}
-
 void IRGenerator::noteUseOfOpaqueTypeDescriptor(OpaqueTypeDecl *opaque) {
   if (!opaque)
     return;

lib/IRGen/GenStruct.cpp

@@ -885,9 +885,10 @@ void IRGenModule::maybeEmitOpaqueTypeDecl(OpaqueTypeDecl *opaque) {
     // then emit all opaque type descriptors and make them runtime-discoverable
     // so that remote ast/mirror can recover them.
     addRuntimeResolvableType(opaque);
-    emitOpaqueTypeDecl(opaque);
     if (IRGen.hasLazyMetadata(opaque))
-      IRGen.markOpaqueTypeDescriptorEmitted(opaque);
+      IRGen.noteUseOfOpaqueTypeDescriptor(opaque);
+    else
+      emitOpaqueTypeDecl(opaque);
   } else if (!IRGen.hasLazyMetadata(opaque)) {
     emitOpaqueTypeDecl(opaque);
   }

lib/IRGen/IRGenModule.h

@@ -391,7 +391,6 @@ class IRGenerator {
   }
 
   void noteUseOfOpaqueTypeDescriptor(OpaqueTypeDecl *opaque);
-  void markOpaqueTypeDescriptorEmitted(OpaqueTypeDecl *opaque);
 
   void noteUseOfFieldDescriptor(NominalTypeDecl *type);