Commit graph

33542 commits

Author SHA1 Message Date
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
William Hesse
e40f303178 Remove checkout of NSS and sqlite from DEPS, remove support files.
BUG=
R=sgjesse@google.com

Review URL: https://codereview.chromium.org//1309903010 .
2015-09-10 12:56:22 +02:00
Asger Feldthaus
7d7d70a0e4 dart2js cps: Check strictness flag in redundant join eliminator.
BUG=
R=kmillikin@google.com

Review URL: https://codereview.chromium.org//1316163004 .
2015-09-10 09:54:43 +02:00
Johnni Winther
0dc96c4c2c Wrap Import uses in deferred_load.dart
This prepares for the addition of ImportElement for imports.

BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1313753004.
2015-09-10 09:53:49 +02:00
Terry L. Lucas
9b0d2582fc Revert "Dartium w/ JsInterop enabled"
This reverts commit 346af54711.

TBR=alanknight@google.com

Review URL: https://codereview.chromium.org//1330163010 .
2015-09-09 17:33:24 -07:00
Terry L. Lucas
346af54711 Dartium w/ JsInterop enabled
TBR=alanknight@google.com

Review URL: https://codereview.chromium.org//1321613005 .
2015-09-09 17:01:40 -07:00
Daniel Andersson
a1bc527306 Migrate logging infrastructure Isolate->Thread
This enables thread-safe logging (e.g., ISL_Print, which will soon be renamed to THR_Print), which is needed for concurrent
marking (DetachCode) and compilation.

Make finalization of GC marking tasks concurrent, now that it's thread-safe.

BUG=
R=iposva@google.com

Review URL: https://codereview.chromium.org//1314673008 .
2015-09-09 15:30:38 -07:00
Konstantin Shcheglov
12614041e5 Use contributed ResultDescriptor(s) while computing DART_ERRORS.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1317213004 .
2015-09-09 15:12:41 -07:00
pq
a1d528fae8 Linter rev bump.
R=keertip@google.com

Review URL: https://codereview.chromium.org//1305153010 .
2015-09-09 14:34:21 -07:00
Konstantin Shcheglov
d13a9ec666 Rename NavigationHolder to NavigationCollector.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1307663007 .
2015-09-09 13:27:24 -07:00
Konstantin Shcheglov
cfc52cfbbf Make NavigationOperation a MergeableOperation.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1327173002 .
2015-09-09 11:01:01 -07:00
Brian Wilkerson
4797c5fecc Fix incremental resolution tests
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1318943006 .
2015-09-09 10:03:51 -07:00
Konstantin Shcheglov
0e6eec7968 Make navigation notification operation Source based, without Dart knowledge.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1307143004 .
2015-09-09 10:02:12 -07:00
Todd Turnidge
5dcad9482d Fix analyzer problems in object.dart.
BUG=

Review URL: https://codereview.chromium.org//1320793005 .
2015-09-09 09:56:36 -07:00
Todd Turnidge
dee6ddf966 Support column-based breakpoints in the VM and Observatory.
BUG=
R=hausner@google.com, rmacnak@google.com

Review URL: https://codereview.chromium.org//1312763010 .
2015-09-09 09:01:13 -07:00
keertip
f95e883a0d rev up dartdoc in DEPS
BUG=
R=pquitslund@google.com

Review URL: https://codereview.chromium.org//1333563002 .
2015-09-09 08:33:02 -07:00
Brian Wilkerson
24b24a3569 Reformat code to reduce churn
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1322513004 .
2015-09-09 08:19:41 -07:00
Brian Wilkerson
42d1d4532d Reformat code to minimize churn
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1310263003 .
2015-09-09 08:19:18 -07:00
Johnni Winther
234f09fa7e Shared commandline flags.
BUG=
R=sigurdm@google.com

Review URL: https://codereview.chromium.org//1314833003.
2015-09-09 15:53:32 +02:00
Michael Thomsen
bc4fdc3e3a Fix typo in changelog
Fixed typo
2015-09-09 15:18:24 +02:00
Sigurd
18bb772cdd Fix crash when deferred loading would encounter an unresolved super setter.
BUG=
R=johnniwinther@google.com

Review URL: https://codereview.chromium.org//1327123003 .
2015-09-09 14:56:11 +02:00
Johnni Winther
8855bc8161 Support metadata on patches.
Closes #24291

R=karlklose@google.com

Review URL: https://codereview.chromium.org//1302333006.
2015-09-09 13:04:51 +02:00
Rico Wind
c6918fb126 Refactor the io resource classes
Move stopwatch so that we will only ever need one instance

Add utility functions for doing reads and writes

The only functional change here is that I say that an except is a read
of one byte, which also updates the readcount. I think this is just as
correct as what we did before.

BUG=
R=sgjesse@google.com

Review URL: https://codereview.chromium.org//1320023008 .
2015-09-09 09:18:07 +02:00
Konstantin Shcheglov
85dadad5f7 Shift AnalysisError instances only once during incremental resolution.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1302333005 .
2015-09-08 17:47:00 -07:00
Natalie Weizenbaum
35f86f7ab8 Bring in the latest version of Pub and various packages.
R=rnystrom@google.com

Review URL: https://codereview.chromium.org//1321903004 .
2015-09-08 17:39:26 -07:00
Brian Wilkerson
de51d0ab45 Improve failure message
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1315233007 .
2015-09-08 15:07:15 -07:00
Sigmund Cherem
d78be6beb0 Fix typo in lookup map, use pub_semver to check version constraints in dart2js
R=sra@google.com

Review URL: https://codereview.chromium.org//1301373009 .
2015-09-08 14:51:23 -07:00
Terry L. Lucas
012c998f64 Revert "Patched in Dartium JsInterop"
This reverts commit 4bb2e617db.

TBR=alanknight@google.com

Review URL: https://codereview.chromium.org//1327083002 .
2015-09-08 13:35:03 -07:00
Brian Wilkerson
8b9c5f231e Remove unneeded test
R=scheglov@google.com

Review URL: https://codereview.chromium.org//1310593004 .
2015-09-08 13:24:09 -07:00
Konstantin Shcheglov
c67b274852 Explicitly document that NavigationRegion.targets cannot be empty.
R=brianwilkerson@google.com
BUG=

Review URL: https://codereview.chromium.org//1328273002 .
2015-09-08 13:16:43 -07:00
Brian Wilkerson
5efe30d458 Stop propagating return types for top-level function declarations
R=paulberry@google.com, scheglov@google.com

Review URL: https://codereview.chromium.org//1309243007 .
2015-09-08 13:14:36 -07:00
Terry L. Lucas
4bb2e617db Patched in Dartium JsInterop
TBR=alanknight@google.com

Review URL: https://codereview.chromium.org//1310363006 .
2015-09-08 12:46:41 -07:00
Brian Wilkerson
38bf731d54 There are several improvements:
- analyzer no longer reports errors in the initializers for static variables until the final resolution
- the collection of static variables is now computed more efficiently
- an unused analysis result is no longer being computed

In addition, this fixes several (but not all) of the tests that fail when the task model is enabled.

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

Review URL: https://codereview.chromium.org//1305863011 .
2015-09-08 10:12:20 -07:00
Asger Feldthaus
a7cba674f5 dart2js cps: Insert bounds check for removeLast() -> pop() optimization.
BUG=
R=kmillikin@google.com

Review URL: https://codereview.chromium.org//1328093004 .
2015-09-08 16:21:18 +02:00
Asger Feldthaus
249a2affaa dart2js cps: Use TypeMaskSystem when inserting refinement nodes.
This also fixes a precision bug where checks like 'x is String'
would refine to [subtype=String] instead of [exact=JSString].

R=kmillikin@google.com

Review URL: https://codereview.chromium.org//1316673006 .
2015-09-08 16:17:02 +02:00
Johnni Winther
7d4751dae8 Add ClassSet to support ClassWorld.strictSubtypesOf
BUG=
R=karlklose@google.com

Review URL: https://codereview.chromium.org//1304083008.
2015-09-08 15:02:42 +02:00