Commit graph

33727 commits

Author SHA1 Message Date
Konstantin Shcheglov cf88943a10 Issue 24338. Don't propose creating a local class for prefixed identifiers.
Importing with prefix would take more time than I'd like to spend now.

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1336323003 .
2015-09-14 08:17:18 -07:00
Konstantin Shcheglov e4b7c93961 Merge occurrences.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1338153002 .
2015-09-14 08:10:02 -07:00
John McCutchan 28a2239289 Move CheckedInSdkPath helpers to utils
BUG=
R=ricow@google.com

Review URL: https://codereview.chromium.org//1342493003 .
2015-09-14 07:05:51 -07:00
Sigurd dc6b09a6d7 Makes the $ variable local to the deferred enclosing function.
This fixes https://github.com/dart-lang/sdk/issues/24328
BUG=
R=floitsch@google.com

Review URL: https://codereview.chromium.org//1337923006 .
2015-09-14 15:35:52 +02:00
Johnni Winther afc7f9acae Avoid creating ConstructedConstantExpression for deferred constant.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1336303002.
2015-09-14 15:28:26 +02:00
Johnni Winther bae758fa61 Show import chains for unsupported dart:* library imports.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1338783002.
2015-09-14 15:19:06 +02:00
Rico Wind 3507b7d3f0 suppress service/process_service_test on arm
Filled issue 24344 to track this

R=sgjesse@google.com
BUG=

Review URL: https://codereview.chromium.org//1336303003 .
2015-09-14 14:29:49 +02:00
Rico Wind 38b579fa78 Add tracking of open processes to the new io resource tracking.
I plan on refactoring the io_resource_info.dart file a bit in a follow up, but I keept this simple

I also plan on adding some sort of sanity checking when the vm exits, to make sure
all 3 maps are empty (in debug mode)

This fixes issue #24314, before we would hold on to process that failed to start
running in the _processes map.

Closes #24314
BUG=
R=johnmccutchan@google.com

Review URL: https://codereview.chromium.org//1331033003 .
2015-09-14 12:54:36 +02:00
regis de456dac24 Update language status file for dart2js.
Review URL: https://codereview.chromium.org//1337923005 .
2015-09-14 18:48:04 +08:00
Rico Wind 8917d7b6c7 Fix suppression of no_assert_test for dartium.
R=iposva@google.com
BUG=

Review URL: https://codereview.chromium.org//1338243002 .
2015-09-14 12:34:24 +02:00
regis e8139d4d92 Fix right shifting of a Mint value by an amount larger than 63.
Add regression test.
Enhance existing bit shift test.

Fixes #24283

R=ager@google.com

Review URL: https://codereview.chromium.org//1338233002 .
2015-09-14 18:20:48 +08:00
Johnni Winther 83b2c7f68a Type check await for.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1338803002.
2015-09-14 11:37:33 +02:00
Rico Wind 99dc218f4a Mark no_assert_test also passing on drt
R=iposva@google.com
BUG=

Review URL: https://codereview.chromium.org//1340903002 .
2015-09-14 09:51:59 +02:00
Rico Wind 2e22f41612 This is a fixed up version of:
https://codereview.chromium.org/1305183010/

- Reorder the priority of flags in the test harness when passed to the VM:
  default flags, file flags, command line flags
- Ensure to not enable assertions when not being requested.

Peter, any reason to not move the flags in to the
configuration file?  Why don't we actually have all of them in there? (not
that I want to move them, I am just currious why we don't add them in
there, it seems more logical to me)

BUG=
R=iposva@google.com, whesse@google.com

Committed: dd7767d95e

Review URL: https://codereview.chromium.org//1330643003 .
2015-09-14 07:58:18 +02:00
regis 9ca5ca6316 Fix a typo in Bigint dart implementation of modInverse.
Fixes #24324

Review URL: https://codereview.chromium.org//1344483003 .
2015-09-14 12:03:50 +08:00
Konstantin Shcheglov ef2fc0d778 Add OCCURRENCES_CONTRIBUTOR_EXTENSION_POINT_ID.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1337143002 .
2015-09-11 13:59:28 -07:00
Todd Turnidge b0ca0789be Fix a variety of service protocol bugs.
Add documentation for breakpoint.

Fix Instance.pattern documentation.

Fix Instance.bytes documentation.

Add class field for null instances.

Fix documentation of Context and closureContext.

Make getObject return Collected for deleted breakpoints.

Closes #24247
Closes #24249
Closes #24309
Closes #24325
Closes #24326
Closes #24327

BUG=
R=johnmccutchan@google.com

Review URL: https://codereview.chromium.org//1341473002 .
2015-09-11 12:29:21 -07:00
Ryan Macnak 690cccf03c Remove remaining uses of null's absolute address from non-IA32.
Note not every file is touched for every architecture because some already load null via Thread in most places.

Don't use absolute stack overflow address when precompiling.

R=fschneider@google.com

Review URL: https://codereview.chromium.org//1332923005 .
2015-09-11 11:34:21 -07:00
Stephen Adams ce350d7ea3 Test for consistent JSArray indexer errors
Optimized and unoptimized paths should generate the same errors.

R=lrn@google.com

Review URL: https://codereview.chromium.org//1327423002 .
2015-09-11 10:33:00 -07:00
John McCutchan fdf79d85a2 Add LookupHeapObjectMessage and hook into LookupHeapObject
- This makes isolate messages behave like other instances over the service protocol.

R=turnidge@google.com

Review URL: https://codereview.chromium.org//1339713002 .
2015-09-11 10:22:40 -07:00
pq 5441e21750 Fixed server plugin status reporting.
R=brianwilkerson@google.com

Review URL: https://codereview.chromium.org//1336733003 .
2015-09-11 08:55:15 -07:00
Brian Wilkerson e84d069467 Fixed the last unit test that was failing with the new task model
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1327363002 .
2015-09-11 08:47:05 -07:00
Konstantin Shcheglov 57958d7643 Keep known valid Dart results and invalidate all the other results during incremental resolution.
By doing this we invalidate Angular results, so cause their recomputing and
updating Angular specific errors and resolved ranges.

R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1331223003 .
2015-09-11 07:59:23 -07:00
William Hesse 078d5b79a5 Revert "Update status of try tests on dartium"
The problem was fixed in f850f312e3.
This reverts commit d044fc84ee.

Fix conversion from Uri to File in test, to use File.fromUri().

BUG=
TBR=johnniwinther@google.com

Review URL: https://codereview.chromium.org//1332403002 .
2015-09-11 16:30:35 +02:00
William Hesse e14568cf83 Fix conversion from Uri to File in test, to use File.fromUri().
BUG=
R=wibling@google.com

Review URL: https://codereview.chromium.org//1333313002 .
2015-09-11 16:21:44 +02:00
William Hesse d044fc84ee Update status of try tests on dartium
BUG=https://github.com/dart-lang/sdk/issues/24334
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1337033002 .
2015-09-11 16:13:59 +02:00
Johnni Winther f850f312e3 Avoid dart:io dependency in try
BUG=

Review URL: https://codereview.chromium.org//1336843003.
2015-09-11 16:03:18 +02:00
Johnni Winther bda5146f40 Compute Compiler.proxyConstant on demand.
BUG=
R=karlklose@google.com

Review URL: https://codereview.chromium.org//1334203002.
2015-09-11 14:47:49 +02:00
Johnni Winther 7407a72115 Add related types check to analyze_dart2js_test
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1338683002.
2015-09-11 14:38:04 +02:00
Johnni Winther b76143b55e Mark value_range_test as slow.
BUG=

Review URL: https://codereview.chromium.org//1338713002.
2015-09-11 13:37:33 +02:00
Lasse R.H. Nielsen ff65079c47 Add packages parameter to Isolate.spawnUri.
No implementation supplied, marked as "not implemented everywhere".

BUG= http://dartbug.com/23951
R=iposva@google.com, sigmund@google.com

Review URL: https://codereview.chromium.org//1323373002 .
2015-09-11 13:07:54 +02:00
Lasse R.H. Nielsen 7464947ca0 Make Map constructor external to allow dart2js to specialize it.
Review URL: https://codereview.chromium.org//1330303002 .
2015-09-11 13:06:09 +02:00
Lasse R.H. Nielsen d755dd65b9 Update range errors to agree on the numbers.
Also ensure that typed-data errors are consistent with other lists.

Fixes issue #24295
BUG= http://dartbug.com/24295
R=floitsch@google.com, iposva@google.com, sra@google.com

Review URL: https://codereview.chromium.org//1318943005 .
2015-09-11 13:05:36 +02:00
Johnni Winther 8bcbccc067 Use memory compiler in compiler_helper.dart.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1322973007.
2015-09-11 12:49:34 +02:00
Johnni Winther d655374736 Add warnings and hints to for-in.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1334673002.
2015-09-11 12:38:34 +02:00
Michael Thomsen f24acf542d Merge pull request #24315 from dart-lang/mit-mit-typo
Fix typo in changelog
2015-09-11 12:28:16 +02:00
Lasse R.H. Nielsen 931e41777e Merge pull request #24236 from a14n/use-const
use const constructors of Utf8* at several places
2015-09-11 12:26:27 +02:00
Kevin Millikin 95860326cd Fix a typo in the documentation for HashMap.
BUG=
R=lrn@google.com

Review URL: https://codereview.chromium.org//1337873003 .
2015-09-11 11:28:10 +02:00
Florian Schneider 1907b5a3e1 VM: Fix bug in ARM v5/v6 loading of immediates.
The condition was not correctly passed on as argument in LoadDecodableImmediate.

BUG=
R=regis@google.com

Review URL: https://codereview.chromium.org//1318003003 .
2015-09-11 10:00:22 +02:00
Zachary Anderson 4bc59321ce Fix mac build
BUG=

Review URL: https://codereview.chromium.org//1337783004 .
2015-09-11 00:35:12 -07:00
Zachary Anderson 16393ee9c6 Uses SNPRINT macro where possible. Otherwise uses #define for format.
I was able to convert 49 SNPrint(NULL, 0, ...) patterns to use the
macro. I had to use a #define for the format string 5 times due to
uses that didn't fit the macro.

2 occurences of SNPrint(NULL, 0, ...) had >=2 possibilities for the
format string based on runtime values, and I didn't try to convert
them.

Fixed ~10 format strings.

BUG=
R=iposva@google.com

Review URL: https://codereview.chromium.org//1331623002 .
2015-09-11 00:18:14 -07:00
Ryan Macnak 718ae3a704 Don't optimize away static field initialization when precompiling.
R=hausner@google.com

Review URL: https://codereview.chromium.org//1327383002 .
2015-09-10 17:27:38 -07:00
Ryan Macnak ebad0abfa9 ISL_Print -> THR_Print
TBR=koda@google.com

Review URL: https://codereview.chromium.org//1335773002 .
2015-09-10 15:58:54 -07:00
Ryan Macnak 8ebb9f9b39 Track which entries in the ObjectPool are native entries, and reset them when loading a precompiled snapshot.
R=fschneider@google.com

Review URL: https://codereview.chromium.org//1316673005 .
2015-09-10 15:41:37 -07:00
Stephen Adams 1aa97c36b5 dart2js ssa: Open coding of removeLast should use -1 as index
This makes the error consistent between unoptimized and optimizer code.

R=sigmund@google.com

Review URL: https://codereview.chromium.org//1315523012 .
2015-09-10 15:38:49 -07:00
Natalie Weizenbaum b49d1b912c Bring in the latest yaml version.
R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1334893002 .
2015-09-10 14:52:06 -07:00
John McCutchan e07d9e8918 Use the checked in SDK to build Observatory
BUG=
R=turnidge@google.com

Review URL: https://codereview.chromium.org//1337503003 .
2015-09-10 11:15:41 -07:00
John McCutchan 38ab14f734 Use a source_set instead of static_library for libdart_embedder_noio
Some linkers (like libtool) complain when trying to create a
static_library with no sources. A source_set communicates the same thing
to GN without upsetting the linker.

patch from issue 1313973005 at patchset 1 (http://crrev.com/1313973005#ps1)

BUG=

Review URL: https://codereview.chromium.org//1331193002 .
2015-09-10 09:33:01 -07:00
John McCutchan fe548c4bcd Add --timing
- Dump per thread timing information when an isolate shuts down.

DeltaBlue(RunTime): 3654.6660583941607 us.
Timing for isolate DeltaBlue.dart$main-109078706 (from 2 threads)

Thread 0 (f68f8b40):
HandleMessage : 2163.455 ms total on stack; 940.038 ms total executing; 2114.231 ms max on stack; 930.085 ms max executing.
CompileFunction : 81.020 ms total on stack; 81.020 ms total executing; 7.745 ms max on stack; 7.745 ms max executing.
CompileOptimizedFunction : 109.817 ms total on stack; 109.817 ms total executing; 10.443 ms max on stack; 10.443 ms max executing.
CollectNewGeneration : 1032.580 ms total on stack; 1032.580 ms total executing; 7.677 ms max on stack; 7.677 ms max executing.

Thread 1 (f7419700):
InitializeIsolate : 12.121 ms total on stack; 0.418 ms total executing; 12.121 ms max on stack; 0.418 ms max executing.
ObjectStore::Init : 0.002 ms total on stack; 0.002 ms total executing; 0.002 ms max on stack; 0.002 ms max executing.
Object::Init : 0.268 ms total on stack; 0.268 ms total executing; 0.268 ms max on stack; 0.268 ms max executing.
IsolateSnapshotReader : 11.433 ms total on stack; 11.433 ms total executing; 11.433 ms max on stack; 11.433 ms max executing.
CompileFunction : 111.849 ms total on stack; 111.849 ms total executing; 10.798 ms max on stack; 10.704 ms max executing.

Totals:
HandleMessage : 2163.455 ms total on stack; 940.038 ms total executing; 2114.231 ms max on stack; 930.085 ms max executing.
CompileFunction : 192.869 ms total on stack; 192.869 ms total executing; 10.798 ms max on stack; 10.704 ms max executing.
CompileOptimizedFunction : 109.817 ms total on stack; 109.817 ms total executing; 10.443 ms max on stack; 10.443 ms max executing.
CollectNewGeneration : 1032.580 ms total on stack; 1032.580 ms total executing; 7.677 ms max on stack; 7.677 ms max executing.
InitializeIsolate : 12.121 ms total on stack; 0.418 ms total executing; 12.121 ms max on stack; 0.418 ms max executing.
ObjectStore::Init : 0.002 ms total on stack; 0.002 ms total executing; 0.002 ms max on stack; 0.002 ms max executing.
Object::Init : 0.268 ms total on stack; 0.268 ms total executing; 0.268 ms max on stack; 0.268 ms max executing.
IsolateSnapshotReader : 11.433 ms total on stack; 11.433 ms total executing; 11.433 ms max on stack; 11.433 ms max executing.

R=rmacnak@google.com

Review URL: https://codereview.chromium.org//1296353002 .
2015-09-10 09:31:16 -07:00
Brian Wilkerson 5b925adf82 Addition bug fixes for new task model
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1331843004 .
2015-09-10 08:10:29 -07:00