[perf] Speed up DeadCode rules with lazy docblock/walk and instanceof guard before scope type lookup

rules/DeadCode/Rector/BinaryOp/RemoveRedundantTypeCheckRector.php

@@ -104,12 +104,12 @@ private function refactorBooleanOr(BooleanOr $booleanOr): ?Expr
             return null;
         }
 
-        // the docblock type can be wider than the real value
-        if ($this->exprAnalyzer->isNonTypedFromParam($nullComparedExpr)) {
+        if (! $booleanOr->right instanceof BooleanNot) {
             return null;
         }
 
-        if (! $booleanOr->right instanceof BooleanNot) {
+        // the docblock type can be wider than the real value
+        if ($this->exprAnalyzer->isNonTypedFromParam($nullComparedExpr)) {
             return null;
         }
 
@@ -144,6 +144,10 @@ private function refactorBooleanAnd(BooleanAnd $booleanAnd): ?Expr
             return null;
         }
 
+        if (! $booleanAnd->right instanceof FuncCall) {
+            return null;
+        }
+
         // the docblock type can be wider than the real value
         if ($this->exprAnalyzer->isNonTypedFromParam($booleanAnd->left)) {
             return null;

rules/DeadCode/Rector/ClassMethod/RemoveUnusedPromotedPropertyRector.php

@@ -111,7 +111,7 @@ public function refactor(Node $node): ?Node
         }
 
         $hasChanged = false;
-        $phpDocInfo = $this->phpDocInfoFactory->createFromNode($constructClassMethod);
+        $phpDocInfo = null;
 
         foreach ($constructClassMethod->params as $key => $param) {
             // only private local scope; removing public property might be dangerous
@@ -140,6 +140,7 @@ public function refactor(Node $node): ?Node
                 continue;
             }
 
+            $phpDocInfo ??= $this->phpDocInfoFactory->createFromNode($constructClassMethod);
             if ($phpDocInfo instanceof PhpDocInfo) {
                 $paramTagValueNode = $phpDocInfo->getParamTagValueByName($paramName);
                 if ($paramTagValueNode instanceof ParamTagValueNode) {

rules/DeadCode/Rector/Property/RemoveDefaultValueFromAssignedPropertyRector.php

@@ -86,12 +86,8 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        // early return can skip the assign, so the default value is still needed
-        if ($this->hasEarlyReturn($node, $constructClassMethod)) {
-            return null;
-        }
-
         $hasChanged = false;
+        $hasEarlyReturn = null;
         $isFinal = $node->isFinal() || FeatureFlags::treatClassesAsFinal($node);
 
         foreach ($node->getProperties() as $property) {
@@ -128,6 +124,12 @@ public function refactor(Node $node): ?Node
                     continue;
                 }
 
+                // early return can skip the assign, so the default value is still needed
+                $hasEarlyReturn ??= $this->hasEarlyReturn($node, $constructClassMethod);
+                if ($hasEarlyReturn) {
+                    return null;
+                }
+
                 $propertyProperty->default = null;
                 $hasChanged = true;
             }