[PropertyWrappers] Remove redundant calls to setInterfaceType(ErrorType::get()) as setInvalid automatically does that

Author: theblixguy

lib/Sema/TypeCheckPropertyWrapper.cpp

@@ -597,7 +597,6 @@ PropertyWrapperBackingPropertyTypeRequest::evaluate(
     var->diagnose(diag::property_wrapper_incompatible_property,
                   propertyType, rawType);
     var->setInvalid();
-    var->setInterfaceType(ErrorType::get(propertyType));
     if (auto nominalWrapper = rawType->getAnyNominal()) {
       nominalWrapper->diagnose(diag::property_wrapper_declared_here,
                                nominalWrapper->getFullName());

lib/Sema/TypeCheckStorage.cpp

@@ -2345,7 +2345,6 @@ PropertyWrapperBackingPropertyInfoRequest::evaluate(Evaluator &evaluator,
       var->diagnose(diag::property_wrapper_incompatible_property,
                     propertyType, wrapperType);
       var->setInvalid();
-      var->setInterfaceType(ErrorType::get(propertyType));
       if (auto nominalWrapper = wrapperType->getAnyNominal()) {
         nominalWrapper->diagnose(diag::property_wrapper_declared_here,
                                  nominalWrapper->getFullName());