fix caching in checkExpressionCached when checkMode is not normal

Author: gabritto

src/compiler/checker.ts

@@ -33663,11 +33663,11 @@ namespace ts {
         }
 
         function checkExpressionCached(node: Expression | QualifiedName, checkMode?: CheckMode): Type {
+            if (checkMode && checkMode !== CheckMode.Normal) {
+                return checkExpression(node, checkMode);
+            }
             const links = getNodeLinks(node);
             if (!links.resolvedType) {
-                if (checkMode && checkMode !== CheckMode.Normal) {
-                    return checkExpression(node, checkMode);
-                }
                 // When computing a type that we're going to cache, we need to ignore any ongoing control flow
                 // analysis because variables may have transient types in indeterminable states. Moving flowLoopStart
                 // to the top of the stack ensures all transient types are computed from a known point.

tests/baselines/reference/narrowingDestructuring.types

@@ -111,7 +111,7 @@ function func3<T extends { kind: "a", a: string } | { kind: "b", b: number }>(t:
 
         const { kind, ...r1 } = t;
 >kind : "a"
->r1 : { a: string; }
+>r1 : Omit<T, "kind">
 >t : { kind: "a"; a: string; }
 
         const r2 = (({ kind, ...rest }) => rest)(t);