Find a file
Stephen Adams 75d4047276 [dart2js] Dead code eliminator should remove dead phis
The bad codegen reported in #48383 was due to an unused phi confusing
the expression-tree construction algorithm. It was trying to generate

    data = cond ? callWithSideEffects() : null;

but `data` is unused, so somehow we generated only

    cond;

We should not have unused phis at this point so I put a safety check
in the above code to not try to build an unused conditional, and
changed the dead-code eliminator to eliminate dead phis.

Now we get

    if (cond)
      callWithSideEffects();

I compared the output for some large apps and there were about a dozen
changes. All looked like improvements, e.g. not assigning to an unused
variable. There was some code missing that is now present, but luckily
for the large apps, it was all code that did not have real-world side
effects.


Fixed: 48383
Change-Id: Id7b32cfa0cbfb47a4d9eff174ad9ae52da99f6a4
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/232781
Reviewed-by: Sigmund Cherem <sigmund@google.com>
Commit-Queue: Stephen Adams <sra@google.com>
2022-02-14 20:15:30 +00:00
.dart_tool [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
.github [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
benchmarks [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
build [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
docs [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
pkg [dart2js] Dead code eliminator should remove dead phis 2022-02-14 20:15:30 +00:00
runtime [gardening] Skip part of Isolate.spawnUri() which depends on --enable-assertions in PRODUCT mode 2022-02-14 18:37:54 +00:00
samples [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
samples-dev [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
samples_2 [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
sdk [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
tests [dart2js] Dead code eliminator should remove dead phis 2022-02-14 20:15:30 +00:00
third_party [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
tools [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
utils [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
.clang-format
.gitattributes [web] rename suite dart2js -> web. 2021-02-04 23:11:32 +00:00
.gitconfig
.gitignore [vm] Remove traces of LLVM backend 2021-03-09 20:37:20 +00:00
.gn [infra] Migrate scripts to python3 2021-04-15 10:10:20 +00:00
.mailmap
.packages [ DDS ] Move package:dds/vm_service_extensions.dart into package:dds_service_extensions 2022-01-10 21:11:33 +00:00
.style.yapf Mass format python with yapf 2019-08-05 20:34:31 +00:00
.vpython
AUTHORS [vm] Support RISC-V. 2022-01-20 00:57:57 +00:00
BUILD.gn Revert "[vm] Add analyze_snapshot tool for AOT snapshot inspection" 2021-12-16 18:32:20 +00:00
CHANGELOG.md Revert "Add the ability to customize socket creation." 2022-02-09 21:48:34 +00:00
codereview.settings [infra] Default presubmits to use python3 2021-08-24 08:20:47 +00:00
compile_output.txt Add Identifier class and replace most TypeAnnotation uses with it. 2022-01-27 18:03:12 +00:00
CONTRIBUTING.md Move configuration to separate section to make it easier to discover. 2022-02-08 20:36:46 +00:00
DEPS [co19] Roll co19 to a38d7c5685e64499cfbdbfe6548fbd5b63b57f15 2022-02-14 12:13:23 +00:00
LICENSE Update LICENSE 2021-04-08 07:57:10 +00:00
OWNERS [infra] Add OWNERS to the Dart SDK 2022-02-14 14:06:34 +00:00
PATENT_GRANT Rename patent file 2019-08-16 12:03:17 +00:00
PRESUBMIT.py Remove references to dartfmt in various comments and docs. 2021-10-01 00:56:14 +00:00
README.dart-sdk [sdk] - Update README file in sdk. 2021-10-25 16:08:40 +00:00
README.md master => main 2021-09-10 08:02:49 +00:00
sdk_args.gni Allow configuring the destination of the SDK in the build output directory 2021-06-29 18:45:29 +00:00
SECURITY.md Add link to published advisories 2022-02-08 19:45:07 +00:00
WATCHLISTS [dart2js] More shared_messages.dart cleanup 2021-09-22 19:20:54 +00:00

Dart

A client-optimized language for fast apps on any platform

Dart is:

  • Optimized for UI: Develop with a programming language specialized around the needs of user interface creation.

  • Productive: Make changes iteratively: use hot reload to see the result instantly in your running app.

  • Fast on all platforms: Compile to ARM & x64 machine code for mobile, desktop, and backend. Or compile to JavaScript for the web.

Dart's flexible compiler technology lets you run Dart code in different ways, depending on your target platform and goals:

  • Dart Native: For programs targeting devices (mobile, desktop, server, and more), Dart Native includes both a Dart VM with JIT (just-in-time) compilation and an AOT (ahead-of-time) compiler for producing machine code.

  • Dart Web: For programs targeting the web, Dart Web includes both a development time compiler (dartdevc) and a production time compiler (dart2js).

Dart platforms illustration

License & patents

Dart is free and open source.

See LICENSE and PATENT_GRANT.

Using Dart

Visit dart.dev to learn more about the language, tools, and to find codelabs.

Browse pub.dev for more packages and libraries contributed by the community and the Dart team.

Our API reference documentation is published at api.dart.dev, based on the stable release. (We also publish docs from our beta and dev channels, as well as from the primary development branch).

Building Dart

If you want to build Dart yourself, here is a guide to getting the source, preparing your machine to build the SDK, and building.

There are more documents on our wiki.

Contributing to Dart

The easiest way to contribute to Dart is to file issues.

You can also contribute patches, as described in Contributing.