Commit graph

57203 commits

Author SHA1 Message Date
Peter von der Ahé cc1a7b35d0 Update SDK wrt to void and invalid returns
Change-Id: Ia1a6b056351beb9faddc2c6e86280983a2395d45
Reviewed-on: https://dart-review.googlesource.com/65140
Commit-Queue: Peter von der Ahé <ahe@google.com>
Reviewed-by: Lasse R.H. Nielsen <lrn@google.com>
2018-07-17 13:37:47 +00:00
Brian Wilkerson 3dba24bda2 Use TypeVariableInConstExpression in place of Unspecified
Change-Id: Ib0f902979c33e97fa46660f0c0a4e98b092c274c
Reviewed-on: https://dart-review.googlesource.com/65240
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2018-07-17 13:23:18 +00:00
Brian Wilkerson 6927e30cac Use BreakOutsideOfLoop instead of Unspecified
Change-Id: Ib420f7d2bb3c47da36f8fa81b4c2c09d319b74e7
Reviewed-on: https://dart-review.googlesource.com/65205
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2018-07-17 13:23:09 +00:00
Dan Rubel b571e9391c Report missing semicolon on last consumed token
This moves the location on which a missing semicolon error is reported,
making it easier for a user to understand and fix,
and fixing some of the statement completion tests.

Change-Id: I11b9de57e2b3104238e1edb801ad1cd02e9a63a2
Reviewed-on: https://dart-review.googlesource.com/64900
Commit-Queue: Dan Rubel <danrubel@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2018-07-17 12:30:37 +00:00
William Hesse dcd58ff563 Update unittest to 0.11.x. Check out unittest from chromium.googlesource.
Change-Id: I35fbf7d18cf596c23fcdc9c702cc1a6d498a4691
Reviewed-on: https://dart-review.googlesource.com/65320
Reviewed-by: Lasse R.H. Nielsen <lrn@google.com>
Commit-Queue: William Hesse <whesse@google.com>
2018-07-17 12:18:48 +00:00
Bob Nystrom 42bbdec54e Update dartfmt to 1.1.3.
We also need to update the prebuilt SDK to keep it in sync with this
change so the presubmit doesn't get wonky. But, in this case, it should
be fine. The change affects a rare case that I don't think occurs in
the SDK: a multi-line string inside a string interpolation, inside
another string. If that does occur, it's easy to just change that code.

Change-Id: Icdfdfb0b2e1359005d88d983e9a3cf05b0e91e48
Reviewed-on: https://dart-review.googlesource.com/65280
Reviewed-by: Jenny Messerly <jmesserly@google.com>
2018-07-17 11:04:29 +00:00
Peter von der Ahé c137fe2fbe Update tests related to generalized void
Change-Id: Iac43d86759e0ae958940bbfd2ce53286762b7207
Reviewed-on: https://dart-review.googlesource.com/64381
Reviewed-by: Peter von der Ahé <ahe@google.com>
Commit-Queue: Peter von der Ahé <ahe@google.com>
2018-07-17 10:25:25 +00:00
Vyacheslav Egorov c910738825 [vm/embedder] Introduce dart_standalone_embedder_api.h
This enables other emdedders to reuse parts of the standalone embedder
implementation without depending too much on the implementations details.

As part of the change we also remove a dependency from DartUtils on
IsolateData - by removing dart:_builtin library caching. Lookup
of this library does not seem to be on hot path anymore in Kernel world.

Change-Id: Ia35b28886121828fad7a96f00bcbceff75e00ae8
Reviewed-on: https://dart-review.googlesource.com/64848
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Martin Kustermann <kustermann@google.com>
2018-07-17 09:04:05 +00:00
Johnni Winther fadd6c0698 Update status for ie11
Change-Id: Ic6679377a913ad176048bc798cdd7633f97eb2e6
Reviewed-on: https://dart-review.googlesource.com/65301
Reviewed-by: Johnni Winther <johnniwinther@google.com>
Commit-Queue: Johnni Winther <johnniwinther@google.com>
2018-07-17 08:28:15 +00:00
Konstantin Shcheglov 736a45babe Mark test_getResult_functionTypeFormalParameter_withTypeParameter as @assertFailingTest.
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: Ibe77736bfbb1ebf70412a9878496d480155346f8
Reviewed-on: https://dart-review.googlesource.com/65260
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2018-07-17 02:09:56 +00:00
Konstantin Shcheglov 061004e9b7 Store resolution for instance creation even when wrong number of arguments.
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: Id9526b2e3bfc731b4a27638b3190795357878811
Reviewed-on: https://dart-review.googlesource.com/65168
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2018-07-17 00:48:34 +00:00
Devon Carew e84f061db8 Build a 2nd, --no-preview-dart-2, analysis server snapshot.
Change-Id: I991aabef1ecacdcd01a67d5dcb536e7592f6b5cc
Reviewed-on: https://dart-review.googlesource.com/65020
Commit-Queue: Devon Carew <devoncarew@google.com>
Reviewed-by: Siva Annamalai <asiva@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2018-07-16 23:37:53 +00:00
Aart Bik 8ed9345ccd Revert "[vm/compiler] Improved type analysis for check class."
This reverts commit 5333dd7f0f.

Reason for revert:

vm-kernel-optcounter-threshold-linux-release-x64
started to timeout on buildbot, see:

https://github.com/dart-lang/sdk/issues/33870

Original change's description:
> [vm/compiler] Improved type analysis for check class.
> 
> Rationale:
> Improves the analysis if instance calls need checks
> (check class or check null) combined with CHA.
> 
> Bug: https://github.com/dart-lang/sdk/issues/33664
> 
> 
> Change-Id: I0a4761f8816bf5a5aaf5d17ca56d8bf4e3b79fc3
> Reviewed-on: https://dart-review.googlesource.com/64440
> Commit-Queue: Aart Bik <ajcbik@google.com>
> Reviewed-by: Vyacheslav Egorov <vegorov@google.com>
> Reviewed-by: Alexander Markov <alexmarkov@google.com>

TBR=vegorov@google.com,alexmarkov@google.com,asiva@google.com,ajcbik@google.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: https://github.com/dart-lang/sdk/issues/33664
Change-Id: Iac1e630e55ffbbb0b635c28bbfd5211c2ae6751d
Reviewed-on: https://dart-review.googlesource.com/65220
Reviewed-by: Aart Bik <ajcbik@google.com>
Commit-Queue: Aart Bik <ajcbik@google.com>
2018-07-16 22:58:53 +00:00
Paul Berry 6858559c48 Add support for Function syntax to analyzer/CFE integration.
Fixes #33846.

Change-Id: I5b8836e55d2d3d450e368cd1e234dc01e2630269
Reviewed-on: https://dart-review.googlesource.com/65166
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2018-07-16 21:50:42 +00:00
Shai Barack 7a5e49e4e4 [component] rename libs fuchsia::sys -> component
CP-79 #comment

TEST: n/a

Change-Id: I387e7be3546909721e29c455dd512b8ee9bb099d
Reviewed-on: https://dart-review.googlesource.com/65169
Reviewed-by: Zach Anderson <zra@google.com>
Reviewed-by: Adam Barth <abarth@google.com>
Commit-Queue: Zach Anderson <zra@google.com>
2018-07-16 21:48:42 +00:00
Brian Wilkerson dfc2adf547 Map the error code for InheritedMembersConflict
Change-Id: I89394fcf6fe47261f196a4c7c265a172f4302060
Reviewed-on: https://dart-review.googlesource.com/64940
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
2018-07-16 21:28:03 +00:00
Brian Wilkerson 00c8cf84c5 Convert Unspecified to NonConst[Constructor|Factory]
Change-Id: If71220e1ee7b8a57daea228191c9fafef22b6bc2
Reviewed-on: https://dart-review.googlesource.com/64920
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
2018-07-16 21:22:32 +00:00
Leaf Petersen 773be5181c Update crc32_test.dart with Dart 2 constant name
Change-Id: I5a6a1bd7e027bab5e6833a5a3369aec469c3c0f0
Reviewed-on: https://dart-review.googlesource.com/65167
Reviewed-by: Bob Nystrom <rnystrom@google.com>
Commit-Queue: Leaf Petersen <leafp@google.com>
2018-07-16 21:15:31 +00:00
Ben Konyi 9aa9d88fe4 [ VM / Windows ] Fixed potential null dereference in ThreadLocalData::RunDestructors.
ThreadLocalData::RunDestructors is called even if
ThreadLocalData::InitOnce has not been run, which would result in a null
pointer dereference of thread_locals_. Check for this case, then return
if thread_locals_ is NULL.
Change-Id: Ic0ab12bf643aa112c72d144662f2a32c3f0888c5
Reviewed-on: https://dart-review.googlesource.com/65171
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Commit-Queue: Ben Konyi <bkonyi@google.com>
2018-07-16 21:08:21 +00:00
Régis Crelier 9be82b92ab [VM interpreter] Support inline type check of implicit setter value.
Change-Id: I2c357a638308f15d83d284a85623b550395a6f4e
Reviewed-on: https://dart-review.googlesource.com/64825
Commit-Queue: Régis Crelier <regis@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>
2018-07-16 20:33:41 +00:00
Alexander Aprelev 3c5de5d4e1 [vm] Support http and https schemes in dart imports.
Bug: https://github.com/dart-lang/sdk/issues/33388
Change-Id: I2bc6d45a3a953c56a68954def11bcc3cf237b6a6
Reviewed-on: https://dart-review.googlesource.com/64960
Commit-Queue: Alexander Aprelev <aam@google.com>
Reviewed-by: Zach Anderson <zra@google.com>
Reviewed-by: Siva Annamalai <asiva@google.com>
2018-07-16 20:02:32 +00:00
Konstantin Shcheglov 34c5058e35 Create new ProcessedOptions for every new file compiled.
With https://dart-review.googlesource.com/c/sdk/+/65082 CFE starts
requiring inputs in ProcessedOptions. We don't use it in Analyzer,
but this will help us avoid failing of all out tests.

R=brianwilkerson@google.com, paulberry@google.com

Bug: https://github.com/dart-lang/sdk/issues/33868
Change-Id: I5e1596b80b4c1940848ee774fe32700b778b2c0e
Reviewed-on: https://dart-review.googlesource.com/65164
Reviewed-by: Paul Berry <paulberry@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 20:00:52 +00:00
Jenny Messerly 875abcea39 Remove experimental closure support from dartdevc
Change-Id: Id171cbfd220c4b504f13183ffdcad581c44fb41a
Reviewed-on: https://dart-review.googlesource.com/64826
Commit-Queue: Jenny Messerly <jmesserly@google.com>
Reviewed-by: Bob Nystrom <rnystrom@google.com>
2018-07-16 19:32:44 +00:00
Jenny Messerly 2420134b23 Remove obsolete tools and tests from package:dev_compiler
Change-Id: I28fcd5832d89a0531961daac8f94a97a8b5ec68c
Reviewed-on: https://dart-review.googlesource.com/64824
Reviewed-by: Bob Nystrom <rnystrom@google.com>
Commit-Queue: Jenny Messerly <jmesserly@google.com>
2018-07-16 19:28:01 +00:00
Konstantin Shcheglov a473a0da2f Store resolution for invalid member references.
Change-Id: I7ce556bc4819a77e082d81bb96e54b9cedc68d78
Reviewed-on: https://dart-review.googlesource.com/65021
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 19:15:01 +00:00
Konstantin Shcheglov 8b01effa5b Store unresolved prefix resolution.
Change-Id: I03558469978ec6866d10c57f7a725e72fa6b4ec8
Reviewed-on: https://dart-review.googlesource.com/65163
Reviewed-by: Paul Berry <paulberry@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 19:05:41 +00:00
Paul Berry 7b964090b9 Adapt ResolutionApplier.visitFormalParameterList to handle function-typed formal parameters.
Fixes #33845.

Change-Id: I6de918d644e9a22b1a9f9cd384b972047078a472
Reviewed-on: https://dart-review.googlesource.com/65162
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2018-07-16 18:07:48 +00:00
Brian Wilkerson 69a15d2006 Convert Unspecified to ConstructorNotSync
Change-Id: Iabfad9797912fdb13dc86c22de6593d2c51b6716
Reviewed-on: https://dart-review.googlesource.com/64923
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
2018-07-16 17:29:49 +00:00
Leaf Petersen bcad45f350 Update DEPS to post CONSTANT versions of packages
Change-Id: I0f09586610e4096ed6b1031541c90934ed8a2a59
Reviewed-on: https://dart-review.googlesource.com/65161
Commit-Queue: Leaf Petersen <leafp@google.com>
Reviewed-by: Bob Nystrom <rnystrom@google.com>
2018-07-16 17:18:30 +00:00
Ryan Macnak a0f517834e [observatory] Progress toward static mode compatibility.
Bug: https://github.com/dart-lang/sdk/issues/32503
Change-Id: Ic4390dce8874aab4aaa87fa4153aef668f28738e
Reviewed-on: https://dart-review.googlesource.com/64982
Reviewed-by: Zach Anderson <zra@google.com>
2018-07-16 17:16:33 +00:00
Brian Wilkerson e8d715db84 Convert some uses of Unspecified to NotAConstantExpression
Change-Id: I54df956afc9cdbc8f30d8dd8c3dd0d299907b97d
Reviewed-on: https://dart-review.googlesource.com/64880
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
2018-07-16 17:12:00 +00:00
Devon Carew 0d52cb6a62 Write data to the driver cache asynchronously.
This is perhaps a 3.6% performance improvement for the analysis server.

Change-Id: Ic820177dded1fe230e25ee19b979de8008a9b76b
Reviewed-on: https://dart-review.googlesource.com/65032
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 16:20:49 +00:00
Lasse R.H. Nielsen 91ce347309 Reapply "Create _nullFuture and _falseFuture in the root zone."
Originally landed by https://dart-review.googlesource.com/c/sdk/+/49509
Reverted because an internal test is fragile and changes behavior when the bug is fixed.

Change-Id: I8516082e5741547c46aa521a91826846dc101303
Reviewed-on: https://dart-review.googlesource.com/63743
Reviewed-by: Leaf Petersen <leafp@google.com>
Commit-Queue: Leaf Petersen <leafp@google.com>
2018-07-16 16:19:15 +00:00
Konstantin Shcheglov ce682638d9 Tests for invalid Map literal type arguments resolution.
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: I4e1e78b9c5497b1399a8a5ca3f600cdbbe86cf23
Reviewed-on: https://dart-review.googlesource.com/65029
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2018-07-16 15:59:45 +00:00
Adam Barth c9ba6a9709 [fuchsia] Migrate back to SyncPtr
Now that SyncPtr has the new semantics, we can migrate off the temporary
Sync2Ptr.

Change-Id: Ibb9e2160900603282e80d7f9c3f0bbb4cc8f9534
Reviewed-on: https://dart-review.googlesource.com/65160
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Adam Barth <abarth@google.com>
2018-07-16 15:48:55 +00:00
Konstantin Shcheglov 946be233f6 Use special error templates for for-in loops.
R=brianwilkerson@google.com, paulberry@google.com

Change-Id: I7006468db607cf53d202b37c01913fb20bd722e7
Reviewed-on: https://dart-review.googlesource.com/65034
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 15:18:06 +00:00
Dmitry Stefantsov 31bed29d6e [fasta] Correct override check for methods with covariant parameters
Bug: #32613

Change-Id: Id69e793afd2c933294299d796a28b66db6b5de17
Reviewed-on: https://dart-review.googlesource.com/64840
Commit-Queue: Dmitry Stefantsov <dmitryas@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Reviewed-by: Dmitry Stefantsov <dmitryas@google.com>
Reviewed-by: Peter von der Ahé <ahe@google.com>
2018-07-16 15:00:05 +00:00
Dmitry Stefantsov 48a258a4a5 [Fasta] Additional override checks
Checks override on fields and for methods with type parameters.

Bug: 32613
Change-Id: Ia32d1080959bc706fd72a075fb6ba724fa93fd58
Reviewed-on: https://dart-review.googlesource.com/64542
Reviewed-by: Dmitry Stefantsov <dmitryas@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
Reviewed-by: Lasse R.H. Nielsen <lrn@google.com>
2018-07-16 15:00:05 +00:00
Devon Carew ec5adbf663 Cache the package:path context.current field in toUri() calls.
Change-Id: I9a3c2e0b587d3821340b275af88ce3cff5ec1bcf
Reviewed-on: https://dart-review.googlesource.com/65030
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 14:40:35 +00:00
Dmitry Stefantsov 7d6cd21706 CFE and member conflict rules
See https://github.com/dart-lang/sdk/issues/32613
and https://github.com/dart-lang/sdk/issues/33235#issue-326617285

Bug: #32613, #33235, #33237
Change-Id: I0d1432185b6811137e31135ac2c7f58c4de2de6c
Reviewed-on: https://dart-review.googlesource.com/64500
Commit-Queue: Dmitry Stefantsov <dmitryas@google.com>
Reviewed-by: Dmitry Stefantsov <dmitryas@google.com>
Reviewed-by: Erik Ernst <eernst@google.com>
2018-07-16 14:09:01 +00:00
Devon Carew 5b119ed796 Remove an as expression.
Change-Id: Iba2cc074da5fd052071a09a6bc4d985c0219d991
Reviewed-on: https://dart-review.googlesource.com/65031
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2018-07-16 13:58:28 +00:00
pq d533cef4b8 bump meta sdk constraint
Change-Id: I70dd4c24970f8b7a1c3bb10cfce3c289a4177193
Reviewed-on: https://dart-review.googlesource.com/65024
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Phil Quitslund <pquitslund@google.com>
2018-07-16 13:32:20 +00:00
William Hesse 5eea3942e6 [CQ] Remove pkg-linux-release from the CQ trybots until it is fixed
Change-Id: I17ee0bfc26af1d060fc6f129419239e4a790034a
Reviewed-on: https://dart-review.googlesource.com/65120
Reviewed-by: Jonas Termansen <sortie@google.com>
2018-07-16 13:23:16 +00:00
Johnni Winther 2d10906fff Update safari status
Change-Id: I655406526af5eca99580772e87c0db9335c0188f
Reviewed-on: https://dart-review.googlesource.com/65100
Reviewed-by: Johnni Winther <johnniwinther@google.com>
Reviewed-by: William Hesse <whesse@google.com>
Commit-Queue: Johnni Winther <johnniwinther@google.com>
2018-07-16 12:46:06 +00:00
Lasse R.H. Nielsen 76db06cc24 Fix assert in Future code to allow a Future<Future<int>>.
Fix the `lib_2/async/future_test` test which was otherwise failing everywhere.
Updated status files.

Change-Id: I4e277dd7d47692b92b3303bfdd7cfbde120a8f87
Reviewed-on: https://dart-review.googlesource.com/65080
Commit-Queue: Lasse R.H. Nielsen <lrn@google.com>
Reviewed-by: Johnni Winther <johnniwinther@google.com>
2018-07-16 11:27:36 +00:00
Dmitry Stefantsov 0277432f28 [fasta] Fix inference for const redirecting factories invocations
Fixes #33813.

Bug: http://dartbug.com/33813
Change-Id: I53708666f6c5d55760a8bc23f042565ba073812f
Reviewed-on: https://dart-review.googlesource.com/64841
Reviewed-by: Jens Johansen <jensj@google.com>
2018-07-16 11:24:36 +00:00
Johnni Winther 6acee7cdb7 Fix tracking of native classes needed for rti.
Closes #32286
Closes #33690

Change-Id: Ic62c145ca7bb5257d71c0d062b111b183258b7d0
Reviewed-on: https://dart-review.googlesource.com/64343
Commit-Queue: Johnni Winther <johnniwinther@google.com>
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2018-07-16 10:09:06 +00:00
Johnni Winther 117f651f7b Emit classes used in type variable replacements
Change-Id: Iad3bf28790172e364d83a96003fb7ef96f25747e
Reviewed-on: https://dart-review.googlesource.com/64322
Reviewed-by: Sigmund Cherem <sigmund@google.com>
2018-07-16 10:09:06 +00:00
asiva eb8be07021 Adjust status file to account for test passing on windows platform sure why).
Change-Id: I9e656b378d99e6a766dfe7a946562ce0d2311077
Reviewed-on: https://dart-review.googlesource.com/65035
Reviewed-by: Siva Annamalai <asiva@google.com>
2018-07-16 06:18:22 +00:00
asiva dc084713d4 Adjust status file to include app-jitk in the settings for the named pipe test.
Change-Id: I39f78ca04fe5f1454d7f54c2f4b839ca2263d55e
Reviewed-on: https://dart-review.googlesource.com/65033
Reviewed-by: Siva Annamalai <asiva@google.com>
2018-07-16 05:33:48 +00:00