Optimize external source maps without full cache

Author: JoostK

src/compiler/emitter.ts

@@ -861,6 +861,7 @@ namespace ts {
         let sourceMapsDisabled = true;
         let sourceMapGenerator: SourceMapGenerator | undefined;
         let sourceMapSource: SourceMapSource;
+        let mostRecentlyAddedSourceMapSource: SourceMapSource;
         let sourceMapSourceIndex = -1;
 
         // Comments
@@ -5339,7 +5340,7 @@ namespace ts {
                 const savedSourceMapSource = sourceMapSource;
                 setSourceMapSource(source);
                 emitPos(pos);
-                setSourceMapSource(savedSourceMapSource);
+                setSourceMapSource(savedSourceMapSource, /*addSource*/ false);
             }
             else {
                 emitPos(pos);
@@ -5379,13 +5380,17 @@ namespace ts {
             return tokenPos;
         }
 
-        function setSourceMapSource(source: SourceMapSource) {
+        function setSourceMapSource(source: SourceMapSource, addSource?: boolean) {
             if (sourceMapsDisabled) {
                 return;
             }
 
             sourceMapSource = source;
 
+            if (addSource === false || source === mostRecentlyAddedSourceMapSource) {
+                return;
+            }
+
             if (isJsonSourceMapSource(source)) {
                 return;
             }
@@ -5394,6 +5399,8 @@ namespace ts {
             if (printerOptions.inlineSources) {
                 sourceMapGenerator!.setSourceContent(sourceMapSourceIndex, source.text);
             }
+
+            mostRecentlyAddedSourceMapSource = source;
         }
 
         function isJsonSourceMapSource(sourceFile: SourceMapSource) {

src/compiler/sourcemap.ts

@@ -13,7 +13,6 @@ namespace ts {
         const rawSources: string[] = [];
         const sources: string[] = [];
         const sourceToSourceIndexMap = new Map<string, number>();
-        const fileNameToSourceIndexMap = new Map<string, number>();
         let sourcesContent: (string | null)[] | undefined;
 
         const names: string[] = [];
@@ -52,10 +51,6 @@ namespace ts {
 
         function addSource(fileName: string) {
             enter();
-            const sourceIndexByFileName = fileNameToSourceIndexMap.get(fileName);
-            if (sourceIndexByFileName !== undefined) {
-                return sourceIndexByFileName;
-            }
             const source = getRelativePathToDirectoryOrUrl(sourcesDirectoryPath,
                 fileName,
                 host.getCurrentDirectory(),
@@ -69,7 +64,6 @@ namespace ts {
                 rawSources.push(fileName);
                 sourceToSourceIndexMap.set(source, sourceIndex);
             }
-            fileNameToSourceIndexMap.set(fileName, sourceIndex);
             exit();
             return sourceIndex;
         }