Drop a Needless Reset of An Inherited Type

For enums with broken raw representable bounds, we should aim to make derivation tolerant of the error state rather than work around it in TypeCheckDeclPrimary. This is the last user of the MutableArrayRef form of the inheritance clauses. Those will be removed in the subsequent commit.

Author: CodaFi

lib/Sema/TypeCheckDeclPrimary.cpp

@@ -1915,7 +1915,6 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
         if (!rawTy->is<ErrorType>()) {
           DE.diagnose(ED->getInherited().front().getSourceRange().Start,
                       diag::raw_type_not_literal_convertible, rawTy);
-          ED->getInherited().front().setType(ErrorType::get(getASTContext()));
         }
       }