Unifying compiler context

pkg/front_end/lib/src/fasta/source/source_loader.dart

Index: pkg/front_end/lib/src/fasta/source/source_loader.dart
diff --git a/pkg/front_end/lib/src/fasta/source/source_loader.dart b/pkg/front_end/lib/src/fasta/source/source_loader.dart
index d95702b3df962e033b8b60b65e859e10c675de5b..267f27f11078e4be3a56249219dc407aa7569ec8 100644
--- a/pkg/front_end/lib/src/fasta/source/source_loader.dart
+++ b/pkg/front_end/lib/src/fasta/source/source_loader.dart
@@ -80,7 +80,8 @@ class SourceLoader<L> extends Loader<L> {
 
   final Map<Uri, List<int>> sourceBytes = <Uri, List<int>>{};
 
-  final bool excludeSource = CompilerContext.current.options.excludeSource;
+  final bool excludeSource =
+      CompilerContext.current.options.excludeSourceInformation;

[ahe, 2017/07/18 16:54:37]

This option literally means "do not include source code in the generated dill
file".

This is not just about line number tables or source mapping. Perhaps you want to
rename the option?

[Siggi Cherem (dart-lang), 2017/07/18 22:50:44]

Good point. Renamed to embedSourceText

 
   // Used when building directly to kernel.
   ClassHierarchy hierarchy;