Do not force overridden descriptors computation

Currently 'overriddenDescriptors' of substituted function is lazy
and in most cases it's unnecessary to compute it
(it's enough to use the same field from 'original')

compiler/backend/src/org/jetbrains/kotlin/codegen/state/typeMappingUtil.kt

@@ -77,7 +77,7 @@ fun getEffectiveVariance(parameterVariance: Variance, projectionKind: Variance):
 val CallableDescriptor?.isMethodWithDeclarationSiteWildcards: Boolean
     get() {
         if (this !is CallableMemberDescriptor) return false
-        return firstOverridden {
+        return original.firstOverridden(useOriginal = true) {
             METHODS_WITH_DECLARATION_SITE_WILDCARDS.contains(it.propertyIfAccessor.fqNameOrNull())
         } != null
     }

compiler/frontend.java/src/org/jetbrains/kotlin/load/java/sam/SamAdapterOverridabilityCondition.java

@@ -117,7 +117,7 @@ public class SamAdapterOverridabilityCondition implements ExternalOverridability
             return new SamAdapterInfo(samAdapter, ownerType);
         }
 
-        for (CallableMemberDescriptor overridden : samAdapter.getOverriddenDescriptors()) {
+        for (CallableMemberDescriptor overridden : samAdapter.getOriginal().getOverriddenDescriptors()) {
             ClassDescriptor containingClass = (ClassDescriptor) overridden.getContainingDeclaration();
 
             for (KotlinType immediateSupertype : TypeUtils.getImmediateSupertypes(ownerType)) {

compiler/frontend/src/org/jetbrains/kotlin/resolve/deprecationUtil.kt

@@ -99,7 +99,7 @@ private fun deprecationByOverridden(root: CallableMemberDescriptor): Deprecation
         visited.add(node)
 
         val deprecatedAnnotation = node.getDeprecationByAnnotation()
-        val overriddenDescriptors = node.overriddenDescriptors
+        val overriddenDescriptors = node.original.overriddenDescriptors
         when {
             deprecatedAnnotation != null -> {
                 deprecations.add(deprecatedAnnotation)

core/descriptor.loader.java/src/org/jetbrains/kotlin/load/java/specialBuiltinMembers.kt

@@ -142,6 +142,8 @@ object BuiltinMethodsWithSpecialGenericSignature {
 
     @JvmStatic
     fun CallableMemberDescriptor.getSpecialSignatureInfo(): SpecialSignatureInfo? {
+        if (name !in ERASED_VALUE_PARAMETERS_SHORT_NAMES) return null
+
         val builtinFqName = firstOverridden { it is FunctionDescriptor && it.hasErasedValueParametersInJava }?.fqNameOrNull()
                 ?: return null
 

core/descriptors/src/org/jetbrains/kotlin/descriptors/FunctionDescriptor.java

@@ -35,6 +35,12 @@ public interface FunctionDescriptor extends CallableMemberDescriptor {
     @Override
     FunctionDescriptor substitute(@NotNull TypeSubstitutor substitutor);
 
+    /**
+     * This method should be used with q Wgreat care, because if descriptor is substituted one, calling 'getOverriddenDescriptors'
+     * may force lazy computation, that's unnecessary in most cases.
+     * So, if 'getOriginal().getOverriddenDescriptors()' is enough for you, please use it instead.
+     * @return
+     */
     @Override
     @NotNull
     Collection<? extends FunctionDescriptor> getOverriddenDescriptors();

core/descriptors/src/org/jetbrains/kotlin/descriptors/impl/FunctionDescriptorImpl.java

@@ -196,7 +196,7 @@ public abstract class FunctionDescriptorImpl extends DeclarationDescriptorNonRoo
     public boolean isOperator() {
         if (isOperator) return true;
 
-        for (FunctionDescriptor descriptor : getOverriddenDescriptors()) {
+        for (FunctionDescriptor descriptor : getOriginal().getOverriddenDescriptors()) {
             if (descriptor.isOperator()) return true;
         }
 
@@ -207,7 +207,7 @@ public abstract class FunctionDescriptorImpl extends DeclarationDescriptorNonRoo
     public boolean isInfix() {
         if (isInfix) return true;
 
-        for (FunctionDescriptor descriptor : getOverriddenDescriptors()) {
+        for (FunctionDescriptor descriptor : getOriginal().getOverriddenDescriptors()) {
             if (descriptor.isInfix()) return true;
         }
 

core/descriptors/src/org/jetbrains/kotlin/resolve/DescriptorUtils.kt

@@ -210,14 +210,14 @@ val CallableMemberDescriptor.propertyIfAccessor: CallableMemberDescriptor
 fun CallableDescriptor.fqNameOrNull(): FqName? = fqNameUnsafe.check { it.isSafe }?.toSafe()
 
 fun CallableMemberDescriptor.firstOverridden(
+        useOriginal: Boolean = false,
         predicate: (CallableMemberDescriptor) -> Boolean
 ): CallableMemberDescriptor? {
     var result: CallableMemberDescriptor? = null
     return DFS.dfs(listOf(this),
-                   object : DFS.Neighbors<CallableMemberDescriptor> {
+                   { current ->
-                       override fun getNeighbors(current: CallableMemberDescriptor?): Iterable<CallableMemberDescriptor> {
+                       val descriptor = if (useOriginal) current?.original else current
-                           return current?.overriddenDescriptors ?: emptyList()
+                       descriptor?.overriddenDescriptors ?: emptyList()
-                       }
                    },
                    object : DFS.AbstractNodeHandler<CallableMemberDescriptor, CallableMemberDescriptor?>() {
                        override fun beforeChildren(current: CallableMemberDescriptor) = result == null