Commit graph

71928 commits

Author SHA1 Message Date
Johnni Winther dc97454b63 [analyzer,cfe,dart2js] Add pretty printing to id testing
Adds support for pretty printing verbose annotations.
Adds support for force updating annotation texts useful for applying
the pretty printing to valid annotations.

Change-Id: I8fe9b6d2f8198bd5e160854053c8d964ef832a8d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/129710
Reviewed-by: Dmitry Stefantsov <dmitryas@google.com>
Commit-Queue: Johnni Winther <johnniwinther@google.com>
2020-04-14 13:24:09 +00:00
Johnni Winther 208ab625c1 [cfe] Avoid reporting missing implementation on internal fields for late lowering
Closes #41436

Change-Id: Ib6227a34ca188a379f8910a2911082caef280d79
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/142984
Reviewed-by: Jens Johansen <jensj@google.com>
Commit-Queue: Johnni Winther <johnniwinther@google.com>
2020-04-14 12:21:19 +00:00
Karl Klose d1dbc9a475 [infra] Make configuration name for run_abi_tests a parameter
Change-Id: I5b5a19b298412f01902e4efde89f85fc091bfc9a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/141243
Reviewed-by: Alexander Thomas <athom@google.com>
2020-04-14 10:57:33 +00:00
William Hesse 3e43a3dcad [infra] Remove unsatisfactory link to results feed from status output
The link placed in the results step from the recipe is better,
filtering on configurations rather than configuration groups.
So this link is replaced by the better link, which is also
placed more prominently.

Change-Id: I82f7bf808bc99087e5a1c4e20c3822376f93eec2
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/142151
Reviewed-by: Karl Klose <karlklose@google.com>
2020-04-14 06:11:37 +00:00
Sigmund Cherem 7baa21d97a [changelog] Add implements-Function breaking change to the changelog
Change-Id: I32a4800a73861922040e3f9b1130b702fafa1d56
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143332
Reviewed-by: Johnni Winther <johnniwinther@google.com>
Commit-Queue: Sigmund Cherem <sigmund@google.com>
2020-04-14 05:35:09 +00:00
Jaime Wren 5bf1223ea9 Set a longer default timeout for dartdev.dart tests so flakiness is removed from the bots. Reason in utils.dart:
/// A long [Timeout] is provided for tests that start a process on
/// `bin/dartdev.dart` as the command is not compiled ahead of time, and each
/// invocation requires the VM to compile the entire dependency graph.

Change-Id: Ieec2238b5fb7358af5e9924912eaf2580d5dbfb1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143181
Commit-Queue: Siva Annamalai <asiva@google.com>
Auto-Submit: Jaime Wren <jwren@google.com>
Reviewed-by: Siva Annamalai <asiva@google.com>
2020-04-14 04:19:29 +00:00
Joshua Litt 4aa896464a [dart2js] Add new nnbd strong bots.
Change-Id: I8a40fcc74bcf5f6161831f3c1da857955d624a63
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/142220
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2020-04-14 00:58:32 +00:00
pq 7fe16ba812 sort_directives quick fix
Fixes: https://github.com/dart-lang/sdk/issues/41464

Change-Id: I9c5804ef8ec872b8420feba3d9a0f0da5fb757d6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143303
Commit-Queue: Phil Quitslund <pquitslund@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2020-04-14 00:45:18 +00:00
Srujan Gaddam 53296536de [dart:html] Add back lib/html/input_element_attributes_test
Now that TextInputElement.value accepts a nullable value, this test
can be added back to test null behavior.

Change-Id: Ia17554314f25c5d09e113bb5c9c0a6b6f1cb1c03
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143190
Reviewed-by: Sigmund Cherem <sigmund@google.com>
Commit-Queue: Srujan Gaddam <srujzs@google.com>
2020-04-14 00:37:41 +00:00
Srujan Gaddam 2bd4858d40 [dart:html] Use TreatNullAs attribute in nullability
Closes https://github.com/dart-lang/sdk/issues/41419

Uses the external attribute TreatNullAs to allow nullable setters
and parameters.

Change-Id: I5182ac6dcfdcf78e84f204079a782aebbf128a78
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143185
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2020-04-14 00:37:41 +00:00
Srujan Gaddam 09e47b6177 [dart:html] Copy test resources from lib_2 to lib
Some none Dart files were not copied over in the migration. Tests
are refactored to use the lib version over lib_2.

Change-Id: I21053d81770c4f83b01f27af99cec11f08577c7b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143330
Reviewed-by: Sigmund Cherem <sigmund@google.com>
Reviewed-by: Nicholas Shahan <nshahan@google.com>
Commit-Queue: Srujan Gaddam <srujzs@google.com>
2020-04-13 23:56:08 +00:00
Liam Appelbe b65704e817 [nnbd] Fix lib/async/future_test in strong mode
Change-Id: Idaa06e9824c81009a0c719b9c1f8f698af104309
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143306
Commit-Queue: Liam Appelbe <liama@google.com>
Reviewed-by: Régis Crelier <regis@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>
2020-04-13 23:43:58 +00:00
Riley Porter 17f5dcb336 [tests] Fix CompileTimeErrors in migrated tests.
Fixes:
lib/math/rectangle_test
lib/convert/utf8_test
lib/convert/utf8_encode_test
lib/convert/streamed_conversion_utf8_encode_test
lib/convert/streamed_conversion_utf8_decode_test
lib/convert/streamed_conversion_json_utf8_encode_test
lib/convert/streamed_conversion_json_utf8_decode_test
lib/convert/json_lib_test
lib/convert/encoding_test
lib/convert/chunked_conversion_utf8_test
lib/convert/chunked_conversion_utf85_test
lib/convert/chunked_conversion_utf84_test
lib/convert/base64_test/none
Change-Id: I9d540e2bc908ac2ac74e42fc1b2eb4c729371c96
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143183
Commit-Queue: Riley Porter <rileyporter@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>
2020-04-13 22:42:51 +00:00
Ben Konyi b73bd49ae8 [ Service ] Fix issue where invoke was crashing when passed non-instance arguments
Fixes https://github.com/dart-lang/sdk/issues/41300

Change-Id: Ia4297c9be41db1ae59647f31784b9dd2097f3baf
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143307
Commit-Queue: Ben Konyi <bkonyi@google.com>
Reviewed-by: Ryan Macnak <rmacnak@google.com>
2020-04-13 22:23:49 +00:00
Brian Wilkerson b68334052a Support macro expansion of Dartdoc in code completion
Change-Id: Idbde9584da82606949cce79849088c6432d0602a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143322
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2020-04-13 21:37:09 +00:00
Ben Konyi dc9f500960 Reland "[ VM / DartDev ] Fixed issue where trying to find the directory of the Dart executable would fail if we encountered a relative symlink while trying to determine the absolute path."
This reland contains fixes related to: https://github.com/dart-lang/sdk/issues/41329

This reverts commit 05e43cff92.

Fixes: https://github.com/dart-lang/sdk/issues/41329
Change-Id: I1abef994cf6ffbac7e5c6bddd6de15f4f53535ab
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/142440
Commit-Queue: Ben Konyi <bkonyi@google.com>
Reviewed-by: Siva Annamalai <asiva@google.com>
2020-04-13 21:11:52 +00:00
Ben Konyi 20eaa47ac2 [ VM ] Updated help message for --write-service-info to --write-service-info=file_uri to reduce ambiguity with Windows' paths
Fixes https://github.com/dart-lang/sdk/issues/41109

Change-Id: I26f5c6ba262ca2e0cb8c63fa4b892f014825177f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143308
Reviewed-by: Ryan Macnak <rmacnak@google.com>
2020-04-13 20:39:58 +00:00
Ryan Macnak d49544423e [observatory] Provide missing type arguments to RenderingScheduler.
Needed in Dart 3.

Change-Id: Ib73286d14e02b110d78059e76d5dbf92d0ba3467
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143187
Reviewed-by: Ben Konyi <bkonyi@google.com>
2020-04-13 20:34:14 +00:00
Konstantin Shcheglov dd033a6ab7 Report HintCode.NULLABLE_TYPE_IN_CATCH_CLAUSE.
Change-Id: I747fa9992243f811b4a98320a4ebd169f35daec0
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143326
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 20:21:42 +00:00
Sam Rawlins e1822e9626 Migrator: Add new fix kinds: replaceVar, addType
Fixes #41410

Change-Id: I127d309e84423c24eb2563063af433187b9bf057
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143222
Reviewed-by: Mike Fairhurst <mfairhurst@google.com>
Commit-Queue: Samuel Rawlins <srawlins@google.com>
2020-04-13 20:11:32 +00:00
Konstantin Shcheglov f5b3a0abb0 Save state in FileResolver
Change-Id: I926346c66a5438b3dc97e638ccc34869457f7968
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143014
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Keerti Parthasarathy <keertip@google.com>
2020-04-13 19:03:32 +00:00
Konstantin Shcheglov 21f42b4305 Fix covariant/return_type_test
Change-Id: Ie71d0579ed52af187dacc8f642425866da6a2d2f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143246
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 18:18:22 +00:00
Konstantin Shcheglov 7ed155ff49 Update analyzer expected error range for language/operator/invalid_operators_test.
Change-Id: I1acbbfd8dd173f90a49231577c67678d94040c8a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143247
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 18:00:37 +00:00
Konstantin Shcheglov ec3dd52462 The type 'dynamic' is defined in dart:core.
This makes fixes to analyzer and update the test for updated spec.

Change-Id: Ie5222bcf9056d5cbb7f18437dec04d5af21ff314
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143245
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 17:55:36 +00:00
Konstantin Shcheglov 3d19be30b5 Report errors for enum constants conflics.
This should fix:
language/enum/syntax_test/05
language/enum/syntax_test/06

Change-Id: Id9cfc9ebd3a39d7234ed7e898196b8d30f417877
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143244
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2020-04-13 17:52:32 +00:00
Konstantin Shcheglov 22d08d8e26 Exclude _fe_analyzer_shared/test/constants/data
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: I9fd93402a84137106d8c3c0172ed91a5ae7e75f7
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143241
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 17:47:32 +00:00
Konstantin Shcheglov 115fb42a01 Fix warnings in nnbd_migration.
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: I56fd93eec4a300ed869c0208e2b1544d18fa183b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143242
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 17:35:56 +00:00
Konstantin Shcheglov c57fe977f0 Make exception and stack trace variables final.
This should fix:
language/exception/try_catch_on_syntax_test/10

Bug: https://github.com/dart-lang/sdk/issues/41460
Change-Id: Id300deeeab5d25c5ab8062c8b8f5a887dc8c318b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143243
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 17:30:52 +00:00
Sam Rawlins 638523c3e1 analyzer: Fix addtional language test
which was broken by https://dart-review.googlesource.com/c/sdk/+/143248

Change-Id: Id14c6583a2838f85d21da56575aa2ecaf538160a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143300
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Samuel Rawlins <srawlins@google.com>
2020-04-13 17:27:51 +00:00
Brian Wilkerson 286501b0f1 Compute the completion location while computing the OpType
The completion location will be used later to help compute relevance
based on the element kind.

Change-Id: I3927bb0137340a81fc90efcbc9c3ecb25ad63542
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143240
Reviewed-by: Jaime Wren <jwren@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2020-04-13 17:06:01 +00:00
Konstantin Shcheglov 5d703a5c9c Fix computing annotation values when referencing opt-in from opt-out.
Bug: https://buganizer.corp.google.com/issues/153801444
Change-Id: If3297aebd9f72b5d774af5b4570ac9f76638da8b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143250
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 16:42:31 +00:00
Konstantin Shcheglov f290ae0696 Issue 40554. Apply NORM and TOP_MERGE only when one of the interfaces is not syntactically equal.
Bug: https://github.com/dart-lang/sdk/issues/40554
Change-Id: Ib96873a825a5a33142655231881562e278b586d1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143249
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 16:42:13 +00:00
Brian Wilkerson 2aecf30d2b Add some missed children to code metrics
Change-Id: Ibfed9bfc32b291427247bd4750e57f0eaae62e4d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143253
Reviewed-by: Jaime Wren <jwren@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2020-04-13 16:08:53 +00:00
Sam Rawlins c9710e5059 analyzer: Do not report on a bad type of return expression if function has illegal return type
Fixes #38188 and fixes #41439

Change-Id: Id6339529afba331750858fb34771bed627ece0be
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143248
Commit-Queue: Samuel Rawlins <srawlins@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2020-04-13 03:42:35 +00:00
Alexander Aprelev 8ef508ba36 Revert "[vm/isolates] Introduce fast isolate spawn in AOT."
This reverts commit 61c0960a8b as it broke
vm-kernel-win-release-ia32 buildbot test https://ci.chromium.org/b/8883214567628884960

Change-Id: I6377fd622a3c7d808f46fd9c541c64d343ecc269
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143260
Reviewed-by: Alexander Aprelev <aam@google.com>
2020-04-12 15:59:41 +00:00
Alexander Aprelev 61c0960a8b [vm/isolates] Introduce fast isolate spawn in AOT.
Speed up is achieved by sharing most of the dart code, object store
and class table between isolates in single isolate group. So
instead of bootstrapping isolate from the snapshot, isolate is
initialized by setting pointers to existing data structures already
set up for first isolate, and only few isolate-specific structures (moved
to newly introducted isolate_object_store) are created.

To allow for safe cross-isolate switchable call site, type test cache
mutations additional synchronization via RunWithStoppedMutators(that
relies on safepoints) was added.
Besides switchable call sites, no other mutation to the dart code is
done in AOT, which allows such sharing.

Bug: https://github.com/dart-lang/sdk/issues/37835
Bug: https://github.com/dart-lang/sdk/issues/36097
Change-Id: I64c86525f4ef9cb30567a49a106bfe700355942b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/136780
Commit-Queue: Alexander Aprelev <aam@google.com>
Reviewed-by: Ryan Macnak <rmacnak@google.com>
2020-04-12 14:57:09 +00:00
Konstantin Shcheglov 23a8788f6a Issue 41378. Update on-catch and throw for the spec changes.
Now it is not an error to use a potentially nullable type in on-catch.

But it is an error for throw an expression of type that is not
assignable to Object.

Bug: https://github.com/dart-lang/sdk/issues/41378
Change-Id: If0a198ae618cd01d03f45a46aa1efa6179df76ee
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143220
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2020-04-11 20:08:03 +00:00
Konstantin Shcheglov 89b0f67261 Implement InterfaceType.asInstanceOf() using allSupertypes.
Change-Id: I2e67f68dd924d1abc9718f5e822980692ce47408
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143221
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2020-04-11 18:39:53 +00:00
Martin Kustermann e2b6cc93b6 [vm/async] Update async transformation to take NNBD mode into account
The removal of unsafeCast causes flutter galley size regressions:

  - armv7: -0.43%
  - armv8: -0.21%

which we should look into recovering by having better inlining heuristics.

Issue https://github.com/dart-lang/sdk/issues/41373
Closes https://github.com/dart-lang/sdk/issues/41307

Change-Id: I7c0ba4c9598e13f01e807a607c83d50cf03e9abe
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/142501
Commit-Queue: Martin Kustermann <kustermann@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>
2020-04-11 08:56:52 +00:00
Zichang Guo 17294842e9 Reland "[dart:io] Making OSError implements Exception"
This is a reland of ae392b8e00

https://github.com/flutter/flutter/pull/53258 will be landed first to unblock the roll.

Original change's description:
> [dart:io] Making OSError implements Exception
>
> Bug: https://github.com/dart-lang/sdk/issues/40934
> Change-Id: Icff457333bd570d81cdf9c3136ee4fe1cadfaae8
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/139406
> Reviewed-by: Zach Anderson <zra@google.com>
> Reviewed-by: Siva Annamalai <asiva@google.com>
> Commit-Queue: Zichang Guo <zichangguo@google.com>

Bug: https://github.com/dart-lang/sdk/issues/40934
Change-Id: Ibc17c0a5717c2005898ce279d4d16edbdbbf44fc
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/140910
Commit-Queue: Zichang Guo <zichangguo@google.com>
Reviewed-by: Siva Annamalai <asiva@google.com>
Reviewed-by: Jonas Termansen <sortie@google.com>
Reviewed-by: Zach Anderson <zra@google.com>
2020-04-11 01:29:22 +00:00
Joshua Litt c8b940141c [dart2js] fix for native_wrapping_function3_frog_test in nnbd.
Change-Id: I40811ff96000cec05668e1bc89a95a9853ff0227
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143015
Commit-Queue: Joshua Litt <joshualitt@google.com>
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2020-04-11 00:59:22 +00:00
Sam Rawlins 5ddea4b8f6 analyzer: Fix analyzer crash when spreading a mixin
Fixes #41424

Change-Id: I2093b02d325f1593f36c4752dd665601d6d0422c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143141
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Samuel Rawlins <srawlins@google.com>
2020-04-10 23:50:12 +00:00
Paul Berry 3a8cfa3547 Migration: don't generate traces for casts that don't involve nullability.
Change-Id: I9dca37cb5542d27485b28a012d32eb0e05f45044
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143016
Reviewed-by: Samuel Rawlins <srawlins@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2020-04-10 23:32:03 +00:00
pq b41f7dd1d9 bump to linter 0.1.115
Change-Id: Ib20ee6049ce1155a5ba36664f607b09a502f0340
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143186
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Phil Quitslund <pquitslund@google.com>
2020-04-10 23:25:12 +00:00
Robert Nystrom 19313e32e8 Clean up the test runner's output reporting.
- Get rid of the redundant failure summary which no one seems to like.
- Remove "diff" mode since we don't use status files any more. Its
  job was to print the results in the form of a status file diff.
- Add "--report-failures" to print the list of failing tests at the end
  of a run.
- Clean up newline handling around the compact
  "[00:01 | 100% | +   33 | -    3]" output lines. Since those don't
  print a trailing newline, it would effectively eat the preceding
  newline of other output.
- Clean up some other code while I was in there.

Change-Id: Iaa5e1e196892e653bbc5eca5eebf0f8f3c4f3ae0
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143101
Commit-Queue: Bob Nystrom <rnystrom@google.com>
Auto-Submit: Bob Nystrom <rnystrom@google.com>
Reviewed-by: Nicholas Shahan <nshahan@google.com>
2020-04-10 23:05:42 +00:00
Stephen Adams 9e2c8ef000 [dart2js] Add --dump-ssa option
The dart2js option --dump-ssa=pattern outputs the SSA CFGs for methods
with names matching the RegExp 'pattern' to a '.cfg' file alongside
the javascript output, e.g. 'main.js.cfg'.

To dump all CFGs, use --dump-ssa=.

Change-Id: I8227232d9950bf3d30df0c87709b0c24e8944759
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143082
Commit-Queue: Stephen Adams <sra@google.com>
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2020-04-10 21:02:14 +00:00
Ben Konyi 1d6ee4a50e [ package:vm_service ] Remove package:test timeouts for package:vm_service tests
We'll let test.py kill the tests based on its own timeout settings.

Change-Id: I2dab10e16d936b4c7cfffcab3e78370c00733cb5
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143180
Reviewed-by: Siva Annamalai <asiva@google.com>
2020-04-10 20:56:16 +00:00
Alexander Markov bbbabcdacd [vm/nnbd] Take non-nullability of types into account in the VM
This change assumes that NNBD strong mode is sound.

Fixes https://github.com/dart-lang/sdk/issues/41420

Change-Id: I5f20fff05196abd4ae183a747865d3dec5a6ccda
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143021
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Reviewed-by: Régis Crelier <regis@google.com>
Commit-Queue: Alexander Markov <alexmarkov@google.com>
2020-04-10 20:08:14 +00:00
Alexander Markov 9b1205496a [vm/compiler] Remove redundant InitStaticField instructions
Flutter gallery AOT snapshot size in release mode
armv7 -0.13%
armv8 -0.16%

Fixes https://github.com/dart-lang/sdk/issues/41418

Change-Id: Ida22dbb8c6a51135eb7c3c72ce5113e8dabc2bc9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/143005
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Commit-Queue: Alexander Markov <alexmarkov@google.com>
2020-04-10 20:07:14 +00:00
Leaf Petersen a608616e6f Handle CFE and analyzer crashes when updating static errors.
Change-Id: I3a9b5d3d25f8e0a5331f72e0a83b439c679540a4
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/115008
Reviewed-by: Bob Nystrom <rnystrom@google.com>
2020-04-10 19:56:48 +00:00