Remove experimental flag to cache for-of length checks.

Author: CyrusNajmabadi

Jakefile

@@ -254,7 +254,7 @@ function compileFile(outFile, sources, prereqs, prefixes, useBuiltCompiler, noOu
             options += " --stripInternal"
         }
 
-        options += " --cacheDownlevelForOfLength --preserveNewLines";
+        options += " --preserveNewLines";
 
         var cmd = host + " " + dir + compilerFilename + " " + options + " ";
         cmd = cmd + sources.join(" ");

src/compiler/commandLineParser.ts

@@ -146,12 +146,6 @@ module ts {
             description: Diagnostics.Preserve_new_lines_when_emitting_code,
             experimental: true
         },
-        {
-            name: "cacheDownlevelForOfLength",
-            type: "boolean",
-            description: "Cache length access when downlevel emitting for-of statements",
-            experimental: true,
-        },
         {
             name: "target",
             shortName: "t",

src/compiler/emitter.ts

@@ -2168,8 +2168,6 @@ module ts {
                 let counter = createTempVariable(TempFlags._i);
                 let rhsReference = rhsIsIdentifier ? <Identifier>node.expression : createTempVariable(TempFlags.Auto);
 
-                var cachedLength = compilerOptions.cacheDownlevelForOfLength ? createTempVariable(TempFlags._n) : undefined;
-
                 // This is the let keyword for the counter and rhsReference. The let keyword for
                 // the LHS will be emitted inside the body.
                 emitStart(node.expression);
@@ -2190,28 +2188,15 @@ module ts {
                     emitEnd(node.expression);
                 }
 
-                if (cachedLength) {
-                    write(", ");
-                    emitNodeWithoutSourceMap(cachedLength);
-                    write(" = ");
-                    emitNodeWithoutSourceMap(rhsReference);
-                    write(".length");
-                }
-
                 write("; ");
 
                 // _i < _a.length;
                 emitStart(node.initializer);
                 emitNodeWithoutSourceMap(counter);
                 write(" < ");
 
-                if (cachedLength) {
-                    emitNodeWithoutSourceMap(cachedLength);
-                }
-                else {
-                    emitNodeWithoutSourceMap(rhsReference);
-                    write(".length");
-                }
+                emitNodeWithoutSourceMap(rhsReference);
+                write(".length");
 
                 emitEnd(node.initializer);
                 write("; ");

src/compiler/types.ts

@@ -1589,7 +1589,6 @@ module ts {
         watch?: boolean;
         /* @internal */ stripInternal?: boolean;
         /* @internal */ preserveNewLines?: boolean;
-        /* @internal */ cacheDownlevelForOfLength?: boolean;
         [option: string]: string | number | boolean;
     }