Commit graph

65376 commits

Author SHA1 Message Date
Liam Appelbe d1daa375d1 Support the output-directory flag in run_abi_tests.py
Bug: https://github.com/dart-lang/sdk/issues/37630
Change-Id: I99428a01d3ebf17a3feefcd1abcbc00a8a49b108
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111220
Commit-Queue: Liam Appelbe <liama@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
2019-07-30 18:28:11 +00:00
Paul Berry 8c699b96e5 Migration: fix some erroneous comments in nullability node test.
These comments were copy/pasted from other tests and weren't properly
updated.

Change-Id: Ib1a0279184bb8dbaeb3601676d24301728a812a8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111302
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2019-07-30 18:13:30 +00:00
Paul Berry fc3d23de80 Migration: change graph propagation variables into fields.
This should make it easier to expand the graph propagation algorithm
in follow-up CLs.

Change-Id: I5540d37716d5d617d768ea62d9ab1f0fcdca47c6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111301
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 18:13:30 +00:00
danrubel 3f22f1f5df do not include double-to-int fix by default
This changes the default fixes applied by dartfix so that it no longer
converts doubles to ints by default. The fix can still be applied
by specifying '--fix double-to-int' on the command line.

Change-Id: I751a42a9c6ae6589259e0b7ac5c769fd4efc4313
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111341
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Dan Rubel <danrubel@google.com>
2019-07-30 18:05:50 +00:00
danrubel 8dc24d0919 report dartfix error for unknown fix
Change-Id: I0ebc73423d46ad5c07f4755847fe2ec68020428f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111340
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2019-07-30 18:05:50 +00:00
Brian Wilkerson 1fe2793d37 Improve tests per previous review
Change-Id: I1cabd87a4ef082683725dd0099644efc98871ae3
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111307
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2019-07-30 17:59:50 +00:00
Alexander Aprelev 00ec685089 [gardening] Fix flutter hhh apply.sh so it clones dependencies added in patch files
Change-Id: Ib63d2cbaececda336c805651a38b88bc086117b9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111343
Reviewed-by: Alexander Aprelev <aam@google.com>
2019-07-30 17:43:35 +00:00
Jaime Wren 871c66cad7 Remove the experimental flag on edit.getPostfixCompletion in the Analysis Server spec
Change-Id: Ia3561dd6c80385e939a555c6d55c1460bda55835
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111308
Reviewed-by: Jaime Wren <jwren@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Jaime Wren <jwren@google.com>
2019-07-30 17:38:11 +00:00
Nate Bosch 001a77a966 Fix typo in bitwise and doc
Fixes #37637

Change-Id: I5bdb245db8dc9947d83e889440c09affab576139
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111263
Reviewed-by: Kevin Moore <kevmoo@google.com>
Commit-Queue: Nate Bosch <nbosch@google.com>
2019-07-30 17:30:20 +00:00
Alexander Aprelev 6e48c00c46 [gardening] Fix hhh patch apply.sh so it skips newly added, non-existent dependency folders
Change-Id: Ia2e4639294b5cfb0a33c672c4eb8beea16ce677f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111121
Reviewed-by: Alexander Aprelev <aam@google.com>
2019-07-30 17:12:49 +00:00
Konstantin Shcheglov dddd58b3d3 Report an error when extension declares a member with the same name as declared in Object.
R=brianwilkerson@google.com

Change-Id: Ic344d3c3dfe291e7f10494897160a2ed4f6c108e
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111140
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2019-07-30 16:59:30 +00:00
Alexander Aprelev 0b9668d3c5 [gardening] Follow-up to 038250d99e - fix patch file
Change-Id: Idd6736f578034d4ed60e673aba8b9f75c73f7e56
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111306
Reviewed-by: Alexander Aprelev <aam@google.com>
2019-07-30 16:47:58 +00:00
Brian Wilkerson 645c985201 Resolve references to static members of extensions
Change-Id: Ic25906ab6a9755c4d0e3b1f63b165718c8c0d1a8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111182
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 16:35:40 +00:00
Alexander Markov 713c2227b2 [bots] Enable co19 tests on dartkb bots
Issue: https://github.com/dart-lang/sdk/issues/36425
Change-Id: I1de33cdb846a301ff6f621d2546a0507fa7bb345
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110553
Commit-Queue: Alexander Markov <alexmarkov@google.com>
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
2019-07-30 16:32:30 +00:00
Michael Thomsen 821efa642c Update error messages in kernel_loader.cc
Closes #37660
https://github.com/dart-lang/sdk/pull/37660

GitOrigin-RevId: 102bdbc71402d22b42713d848f38615e4d2456ad
Change-Id: I438092c270aa673f90bb0b6d2c6015c6977bc20b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110980
Commit-Queue: Michael Thomsen <mit@google.com>
Reviewed-by: Vyacheslav Egorov <vegorov@google.com>
2019-07-30 16:20:50 +00:00
Alexander Aprelev 038250d99e [gardening] Fix flutter hhh with patch for 5d5a0c0164
Change-Id: I332c00726ea58b045b2ded59a0c500b11e41c8f8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111304
Reviewed-by: Alexander Aprelev <aam@google.com>
2019-07-30 16:11:55 +00:00
Konstantin Shcheglov b011a80478 Extract ImportsVerifier and company.
These classes are not used anywhere outside of analyzer itself.

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

Change-Id: I98292f2c9402a806d36039c7cc54846cea8bb3c8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111264
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 16:11:12 +00:00
Konstantin Shcheglov 92633cf4af Tests for using getter in MethodInvocation.
R=brianwilkerson@google.com

Change-Id: If63713da5e01608b63d697360faac1fce376550b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111261
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 16:03:10 +00:00
Konstantin Shcheglov c12deff784 Test that AMBIGUOUS_EXPORT is reported for extensions.
R=brianwilkerson@google.com

Change-Id: I5b89b6216d7f6d679f176cc9492407ab4bbdf77b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111262
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 16:03:09 +00:00
lambdabaa 33ce54e45d Don't try to start ML completion isolate on 32 bit systems
Instead of asking tflite_native to try to load a 32 bit dynamic library only to have it notice that none exist, we can simply disable ML ranking in analysis server.

Change-Id: Ibd001430d3e37f2fe854c72ef9f3f5466c96f46e
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111300
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Ari Aye <ariaye@google.com>
2019-07-30 15:03:47 +00:00
Teagan Strickland 795290a614 [vm/compiler] Free, not delete, buffer taken from TextBuffer.
Change-Id: Ic4309c097689d4dade2efc5d027cc04493b8f7d4
Cq-Include-Trybots: luci.dart.try:vm-kernel-asan-linux-release-x64-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111281
Commit-Queue: Teagan Strickland <sstrickl@google.com>
Reviewed-by: Clement Skau <cskau@google.com>
2019-07-30 14:48:57 +00:00
Paul Berry 55fb47e1de Migration: Add an analysis_options.yaml file.
And disable implicit downcasts in the nnbd_migration package.

Change-Id: I00c95d709215a12c04e71ee5da1fb9c4af7b5d3c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111266
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2019-07-30 13:40:07 +00:00
Teagan Strickland 4932ee6d80 Reland "[vm/compiler] Use pragmas instead of separately-maintained lists for inlining."
This closes https://github.com/dart-lang/sdk/issues/36571.

Change-Id: I14c623aba6b7183191ae93e294e26af9f4dcf34f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110441
Commit-Queue: Teagan Strickland <sstrickl@google.com>
Reviewed-by: Martin Kustermann <kustermann@google.com>
2019-07-30 13:14:57 +00:00
Paul Berry e147c3e0c8 Migration: ensure that NodeBuilder visits executable metadata and constructor initializers.
Change-Id: I9969e9f35f89a91890f9ef15d6505df5c36c1b0a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111181
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2019-07-30 12:43:37 +00:00
Paul Berry db16503080 Migration: test generic field initializers
These already work, but we want to make sure they don't regress.

Change-Id: I7b67b2e105e0de6be9db5cac66c0278d03ccb3bd
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111143
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
2019-07-30 12:43:37 +00:00
Teagan Strickland f822dfa3ea Reland "[vm/compiler] Implement new inlining pragmas."
This change introduces two new pragmas:

* `@pragma('vm:never-inline')`
* `@pragma('vm:prefer-inline')`

These replaces the old way of specifying AlwaysInline or NeverInline
annotations when the (now removed) --enable-inlining-annotations flag
was used.

Bug: https://github.com/dart-lang/sdk/issues/36571
Change-Id: Iee152c1d67abde8d58c58fa967449d36e77c8c93
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110440
Commit-Queue: Teagan Strickland <sstrickl@google.com>
Reviewed-by: Samir Jindel <sjindel@google.com>
2019-07-30 12:01:06 +00:00
Martin Kustermann 5f627b7a54 [vm/compiler] Fix inliner bug which did not emit an explicit unboxing instruction for the value
The StoreIndexedInstr have been changed to require their value to be
unboxed. Even though SelectRepresentations will insert an unboxing
instruction, we want to manually create the unbox instruction to ensure
it is marked as truncating.

Closes https://github.com/dart-lang/sdk/issues/37551

Change-Id: I31e2204c6ecacd0d27ac9b6fd71974c84b882ef2
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110912
Reviewed-by: Vyacheslav Egorov <vegorov@google.com>
Commit-Queue: Martin Kustermann <kustermann@google.com>
2019-07-30 11:51:27 +00:00
William Hesse 5e6b200d2a [infra] Add previous commit info to results records for new tests
Change-Id: I4f4b5a133668a3a5926f6d59abc3054b8c47e00b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110223
Reviewed-by: Jonas Jensen <jonasfj@google.com>
2019-07-30 11:42:26 +00:00
Teagan Strickland b87386da26 [vm/compiler] Add S-expression deserializer.
Bug: https://github.com/dart-lang/sdk/issues/36882
Change-Id: I991d160ecbde45a09e7533867f51f1ab72c3b215
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110220
Commit-Queue: Teagan Strickland <sstrickl@google.com>
Reviewed-by: Martin Kustermann <kustermann@google.com>
2019-07-30 11:08:06 +00:00
Clement Skau 50c68d1f30 [Test] Fixes flaky racing in lib_2/isolate/spawn_uri_test.
This is similar to https://dart-review.googlesource.com/c/sdk/+/108410.

Prior to this fix, the test was running at ~3% flaky locally:
  ~> tools/test.py -n dartk-optcounter-linux-release-x64 lib_2/isolate/spawn_uri_test --repeat=100

Change-Id: I7d6422eaf5e56ec8fbe0c287da12b19bf13716db
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110913
Commit-Queue: Clement Skau <cskau@google.com>
Reviewed-by: Martin Kustermann <kustermann@google.com>
2019-07-30 10:26:46 +00:00
Teagan Strickland d6e9e15cf1 [vm] Clear recognized method info on dynamic invocation forwarders.
Similar change to b68e5c4338, where this information gets copied
over into the new dynamic invocation forwarder from the original
function.

Change-Id: I728fa459837a8bb2e831c5314d12f6a00f7f32b6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110722
Reviewed-by: Martin Kustermann <kustermann@google.com>
Commit-Queue: Teagan Strickland <sstrickl@google.com>
2019-07-30 09:13:37 +00:00
Devon Carew 6ebaa7c587 [vm service lib] expose the current service protocol version as a field
Change-Id: I56167fa7c91519a250b31c584a5d3981d0c96859
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110860
Reviewed-by: Ben Konyi <bkonyi@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
2019-07-30 05:15:02 +00:00
Mayank Patke 440d1f3578 [dart2js] Register TypeUse for HAsCheck SSA nodes.
Change-Id: Ic8a104eb3108d7b1cede86a2e4955c3a5e37586d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111260
Commit-Queue: Stephen Adams <sra@google.com>
Reviewed-by: Stephen Adams <sra@google.com>
2019-07-30 04:24:22 +00:00
Stephen Adams 4b35a044c9 [dart2js] new-rti: Make all closures have class type 'Closure'
Change-Id: I6713449b61a717907a5f8eac8fa3e2bea7e11ddd
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111202
Commit-Queue: Mayank Patke <fishythefish@google.com>
Reviewed-by: Mayank Patke <fishythefish@google.com>
2019-07-30 01:29:47 +00:00
Robert Nystrom f9f005cdb9 Tool to automatically update expectations in static error tests.
Change-Id: Ie1faacd66448efe209b35dfd66fded622e59812a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110766
Auto-Submit: Bob Nystrom <rnystrom@google.com>
Commit-Queue: Bob Nystrom <rnystrom@google.com>
Reviewed-by: Nicholas Shahan <nshahan@google.com>
2019-07-30 01:23:22 +00:00
Mayank Patke 90c1e5a35b [dart2js] Remove _ticks from new RTI.
Change-Id: I11ec2088e05b461307de5f14c468e99e216f1ea1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111201
Auto-Submit: Mayank Patke <fishythefish@google.com>
Reviewed-by: Stephen Adams <sra@google.com>
Commit-Queue: Mayank Patke <fishythefish@google.com>
2019-07-30 01:21:42 +00:00
Konstantin Shcheglov b21b6f7086 Set resolution types for extension override method invocation.
Both for unresolved and resolved cases.

R=brianwilkerson@google.com

Change-Id: I2cc83dabefcec5815db4d05400404c190e2309e8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111180
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2019-07-30 01:03:24 +00:00
lambdabaa 8f0b6c89e2 Fix uri of dart_completion_ranking_internal_test.dart import
Change-Id: Iafaff006e6b2d52b88498d3674e854fe284ae927
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111241
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2019-07-30 00:44:04 +00:00
Alexander Markov b3293a4263 [vm/bytecode] Use faster method/field lookups
Issue: https://github.com/dart-lang/sdk/issues/36429
Change-Id: I9d6b209529394aebe8ba64bf1c06cee72e7082b9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111200
Reviewed-by: Ryan Macnak <rmacnak@google.com>
Commit-Queue: Alexander Markov <alexmarkov@google.com>
2019-07-30 00:04:59 +00:00
Mayank Patke 928dff2b3e [dart2js] Omit Object from new RTI type rules.
Change-Id: I63a4dbdbbf301128523f93958567735f9e2a19d2
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111142
Commit-Queue: Mayank Patke <fishythefish@google.com>
Commit-Queue: Stephen Adams <sra@google.com>
Auto-Submit: Mayank Patke <fishythefish@google.com>
Reviewed-by: Stephen Adams <sra@google.com>
2019-07-29 21:10:03 +00:00
Stephen Adams 3d578c7037 [dart2js] new-rti: recognize 'raw' types as precise abstract values
Change-Id: Ie68a36695c8ad93ec4ea280d989cd2001ed5cc78
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110755
Reviewed-by: Mayank Patke <fishythefish@google.com>
Commit-Queue: Stephen Adams <sra@google.com>
2019-07-29 20:59:03 +00:00
Ari Aye 752f06fff5 Make sure to import and run completion_ranking_test_internal.dart in test_all.dart
Change-Id: I410521737c7c7c2bd45377408769b3b6cbcb8425
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111141
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Ari Aye <ariaye@google.com>
2019-07-29 20:41:43 +00:00
Ari Aye db796d5b96 Fix for bad handling of predict response from model isolate
I also went ahead and updated the CIPD dart/language_model version to the latest model version.

Change-Id: I4522209150a90acfba210885316e78381c2aee45
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111160
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Ari Aye <ariaye@google.com>
2019-07-29 20:41:35 +00:00
Mike Fairhurst d54ab60990 Fix TODO now that == considers NullabilitySuffix
Change-Id: Ieae40dd889cf869a1f56f5572e3187c7ab9d04db
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111101
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Mike Fairhurst <mfairhurst@google.com>
2019-07-29 20:35:53 +00:00
Mike Fairhurst 2f96be9ee3 [analyzer] Allow references of 'this' in late field initializers
Change-Id: Ic464a1d8402404ca79f2281780bf3d408bbb948a
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110550
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Mike Fairhurst <mfairhurst@google.com>
2019-07-29 20:23:03 +00:00
Stephen Adams 73cbfd3826 [dart2js] new-rti: Emit new rti for constants
Change-Id: Ia4b5f55a4836d02d79b8c01cb68b6d23fb3cdc3f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110940
Reviewed-by: Mayank Patke <fishythefish@google.com>
Commit-Queue: Stephen Adams <sra@google.com>
2019-07-29 20:17:03 +00:00
Paul Berry 60029e19fa Migration: add support for constructor field initializers.
Change-Id: I69642d52a0ff996c73806731d4ed4d3df260efbc
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111001
Reviewed-by: Mike Fairhurst <mfairhurst@google.com>
Commit-Queue: Paul Berry <paulberry@google.com>
2019-07-29 20:10:03 +00:00
Martin Kustermann 8db1408ffd [vm] Use RareType for fast is-checks optimization instead of TypeArguments::null() comparison
This fixes the ASSERT introduced in 92faca77d9

Change-Id: I6564a1b0436b15e2adf762ead111860f8dd5650c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110909
Commit-Queue: Martin Kustermann <kustermann@google.com>
Reviewed-by: Alexander Markov <alexmarkov@google.com>
Reviewed-by: Martin Kustermann <kustermann@google.com>
2019-07-29 19:58:34 +00:00
Konstantin Shcheglov 60725e7030 Report an error when 'covariant' is used in extension.
R=brianwilkerson@google.com

Change-Id: I69e2d35bdc485448266706239e199911411f720f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/111120
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
2019-07-29 19:11:23 +00:00
Konstantin Shcheglov 1a0fe21fb2 Add ExecutableElement(s) based InheritanceManager3, and switch analyzer to it.
This reverts commit b8614ca613.

Change-Id: Ic31ee323b21311d8a89f50bcb533316eb55f75e9
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/110941
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Paul Berry <paulberry@google.com>
2019-07-29 19:02:24 +00:00