[Typechecker] Don't add empty identifiers to the set

Author: theblixguy

lib/Sema/MiscDiagnostics.cpp

@@ -277,6 +277,9 @@ static void diagSyntacticUseRestrictions(TypeChecker &TC, const Expr *E,
         names.reserve(tupleExpr->getNumElements());
 
         for (auto name : tupleExpr->getElementNames()) {
+          if (name.empty())
+            continue;
+
           if (names.count(name) == 1) {
             diagnose = true;
             break;

lib/Sema/TypeCheckType.cpp

@@ -3166,13 +3166,16 @@ Type TypeResolver::resolveTupleType(TupleTypeRepr *repr,
 
     auto eltName = repr->getElementName(i);
 
+    elements.emplace_back(ty, eltName, ParameterTypeFlags());
+
+    if (eltName.empty())
+      continue;
+
     if (seenEltNames.count(eltName) == 1) {
       foundDupLabel = true;
     }
 
     seenEltNames.insert(eltName);
-
-    elements.emplace_back(ty, eltName, ParameterTypeFlags());
   }
 
   if (hadError)