[vm/aot] Enable tree-shaking of write-only fields by default

Tree-shaking of write only fields is implemented in https://dart-review.googlesource.com/c/sdk/+/143286 This change just enables it by default. Flutter gallery in release mode armv7 app.so size -0.41% (gzip -0.24%, brotli -0.20%) armv8 app.so size -0.38% (gzip -0.23%, brotli -0.29%) Issue: #35310 Change-Id: Iaae3e893b4a6bf4d468d6cd05aba5f8ee8810afc Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/148543 Reviewed-by: Ryan Macnak <[email protected]> Commit-Queue: Alexander Markov <[email protected]>
dart-lang · May 21, 2020 · 9c50efc · 9c50efc
1 parent 7b3ce0d
commit 9c50efc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/frontend_server/lib/frontend_server.dart b/pkg/frontend_server/lib/frontend_server.dart
@@ -60,7 +60,7 @@ ArgParser argParser = ArgParser(allowTrailingOptions: true)
       defaultsTo: false)
   ..addFlag('tree-shake-write-only-fields',
       help: 'Enable tree shaking of fields which are only written in AOT mode.',
-      defaultsTo: false)
+      defaultsTo: true)
   ..addFlag('protobuf-tree-shaker',
       help: 'Enable protobuf tree shaker transformation in AOT mode.',
       defaultsTo: false)

diff --git a/pkg/vm/lib/kernel_front_end.dart b/pkg/vm/lib/kernel_front_end.dart
@@ -102,7 +102,7 @@ void declareCompilerOptions(ArgParser args) {
       defaultsTo: true);
   args.addFlag('tree-shake-write-only-fields',
       help: 'Enable tree shaking of fields which are only written in AOT mode.',
-      defaultsTo: false);
+      defaultsTo: true);
   args.addFlag('protobuf-tree-shaker',
       help: 'Enable protobuf tree shaker transformation in AOT mode.',
       defaultsTo: false);