Florian Schneider
c3fb2b5d89
Fix two io tests for precompilation, update status
...
BUG=#26376
R=rmacnak@google.com
Review URL: https://codereview.chromium.org/2441743002 .
2016-10-20 13:14:12 -07:00
Paul Berry
74d8298777
Fix handling of default formal parameters in DeclarationResolver.
...
When visiting a DefaultFormalParameter, we can't call
super.visitDefaultFormalParameter, because that would cause the
default value to be visited twice.
R=brianwilkerson@google.com
Review URL: https://codereview.chromium.org/2438113002 .
2016-10-20 12:34:16 -07:00
Paul Berry
78ed4066c8
Add a "correction" field to front end compilation errors.
...
R=brianwilkerson@google.com
Review URL: https://codereview.chromium.org/2429213008 .
2016-10-20 12:33:12 -07:00
Vyacheslav Egorov
1f6e03883e
Teach analyzer parser to handle native const factories/const constructors.
...
This is needed to parse VM patch files, which contain things like:
@patch const factory String.fromEnvironment(String name,
{String defaultValue})
native "String_fromEnvironment";
Currently analyzer would throw "Only redirecting factory constructors can be
declared to be 'const'." error when it sees this code.
BUG=
R=paulberry@google.com
Review URL: https://codereview.chromium.org/2435013003 .
2016-10-20 20:09:07 +02:00
Brian Wilkerson
96750cd5cb
Split out options from ContextBuilder
...
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2425423009 .
2016-10-20 11:03:11 -07:00
Bob Nystrom
59ec5d4f60
Update dart_style to 0.2.11+1.
...
R=paulberry@google.com
Review URL: https://codereview.chromium.org/2442513002 .
2016-10-20 10:30:35 -07:00
Jennifer Messerly
dfac88ec18
fix typo in future union inference code
...
R=leafp@google.com
Review URL: https://codereview.chromium.org/2431313005 .
2016-10-20 10:22:23 -07:00
Paul Berry
105f568452
Mark some other instances of visibleLength/visibleOffset as informative.
...
These were inadvertenly left out of 84ec7f01c0
.
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2428413003 .
2016-10-20 09:09:07 -07:00
Brian Wilkerson
9f0e4e1162
Split out error codes into separate files
...
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2442463002 .
2016-10-20 08:40:36 -07:00
Brian Wilkerson
d42a22bb1d
Deprecate ToSourceVisitor
...
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2439773002 .
2016-10-20 08:38:22 -07:00
Zachary Anderson
1baa650388
Fuchsia: Fix eventhandler for changed syscall return
...
Review URL: https://codereview.chromium.org/2433833005 .
2016-10-20 07:47:26 -07:00
Florian Loitsch
6beb1fdd11
Change Set.difference API to accept Set<Object>.
...
Fixes #27573
BUG= http://dartbug.com/27573
R=lrn@google.com
Review URL: https://codereview.chromium.org/2413233002 .
2016-10-20 14:37:15 +02:00
Siva Annamalai
941b0c8c39
Fix build.
...
BUG=
Review URL: https://codereview.chromium.org/2426383003 .
2016-10-19 18:01:41 -07:00
Siva Annamalai
36a237c88d
Fix build.
...
Review URL: https://codereview.chromium.org/2431253004 .
2016-10-19 17:48:01 -07:00
Siva Annamalai
af4d246ef1
Cleanup options for use of application snapshots
...
- unified kAppAfterRun and kAppJITAfterRun to kAppJIT
- adjusted the test configuration to make dart2app and dart2appjit to mean
the same
- delete GenerateFullSnapshot function as it is not used anymore
R=rmacnak@google.com
Review URL: https://codereview.chromium.org/2429023002 .
2016-10-19 17:42:01 -07:00
Florian Schneider
b85f900cb7
test.dart: Push all required files to Android device when running AOT tests
...
Some tests have resources in sub-directories (e.g. certificates/)
Remove status file lines for tests that are now passing.
Fixes #26376,#26377.
R=rmacnak@google.com
Review URL: https://codereview.chromium.org/2434123002 .
2016-10-19 16:50:23 -07:00
Alan Knight
47150f50d5
Use dart:html prefixes (e.g. webkitAudioNode) on other browsers
...
BUG=
R=vsm@google.com
Review URL: https://codereview.chromium.org/2431443002 .
2016-10-19 16:43:41 -07:00
Ryan Macnak
61bf8abd75
Make fatal out of memory messages uniform.
...
Add checks in a few more places with large allocations.
R=fschneider@google.com
Review URL: https://codereview.chromium.org/2418323002 .
2016-10-19 16:37:59 -07:00
Ryan Macnak
c9c33d9db3
Implement File::Map on Windows.
...
R=zra@google.com
Review URL: https://codereview.chromium.org/2430473002 .
2016-10-19 16:36:01 -07:00
Stephen Adams
50e557d176
Prevent inlining of js_library core.RegExp constructor
...
Inlining was achieving no beneficial effect.
- Enabled GVN of match start.
- Made string quote style uniform.
R=sigmund@google.com
Review URL: https://codereview.chromium.org/2437863002 .
2016-10-19 16:25:11 -07:00
Ryan Macnak
21a6f50892
Tearing off a top-level variable introduces a synthetic field which is added to the top-level class not but the library.
...
R=hausner@google.com
Review URL: https://codereview.chromium.org/2428353004 .
2016-10-19 15:34:25 -07:00
Matthias Hausner
f73ea62c99
Add check for constness in constructor redirection
...
Const constructor can only redirect to another const constructor.
BUG=#27617
R=regis@google.com
Review URL: https://codereview.chromium.org/2439593003 .
2016-10-19 15:15:02 -07:00
Zachary Anderson
16db33b7a2
Propagate errors correctly in Socket.connect
...
socket.port can throw an exception. The error from the
exception, if there is one, has to go to the completer,
not be thrown up the stack, otherwise it may go to the
enclosing Zone rather than e.g. a try ... catch around
await Socket.connetc().
This is a possible fix for the issue below, but it is
tough to say because I don't have a reliable repro.
related #27440
R=asiva@google.com
Review URL: https://codereview.chromium.org/2426413006 .
2016-10-19 15:12:59 -07:00
Zachary Anderson
9d29251368
Remove dependency on Fuchsia's runtime library.
...
This library is no longer publicly exposed. The only functionality used
by it is the utility function for printing mx errors, which has now
moved into the vDSO and is available in <magenta/status.h>.
R=zra@google.com , zra@chromium.org
Review URL: https://codereview.chromium.org/2428373005 .
2016-10-19 15:09:32 -07:00
Stephen Adams
3b5c154138
Don't gvn HForeignCode without source text
...
There are a few synthetic templates that are generated from trees instead of source text. I don't think any are marked for GVN, but this is a hazard so best fix it.
TBR=sigmund@google.com
BUG=
Review URL: https://codereview.chromium.org/2431103003 .
2016-10-19 14:46:54 -07:00
Brian Wilkerson
b3f66b03a1
Provide an extensible form of ToSourceVisitor (issue 27169)
...
R=matan@lurey.org
Review URL: https://codereview.chromium.org/2429243003 .
2016-10-19 14:31:21 -07:00
Paul Berry
84ec7f01c0
Make visibleLength/visibleOffset informative.
...
These summary fields are related to source offsets, so they are
affected by minute changes to the source code that don't affect its
API; therefore they should be marked as "@informative" so that they
don't appear in API summaries.
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2432183003 .
2016-10-19 14:24:42 -07:00
Paul Berry
4729ba0055
In DeclarationResolver, handle generic function-typed parameters.
...
R=brianwilkerson@google.com
Review URL: https://codereview.chromium.org/2441483002 .
2016-10-19 14:21:52 -07:00
Konstantin Shcheglov
bf43e4b43e
Validate local resolution scope, and compute initial name scope.
...
R=brianwilkerson@google.com , paulberry@google.com
BUG=
Review URL: https://codereview.chromium.org/2439583003 .
2016-10-19 14:08:46 -07:00
Stephen Adams
1e793c96b2
Properties of flattenString
...
TBR=sigmund@google.com
Review URL: https://codereview.chromium.org/2432993004 .
2016-10-19 13:53:46 -07:00
Paul Berry
148af481ff
Fix DeclarationResolver handling of annotations containing closures.
...
R=brianwilkerson@google.com
Review URL: https://codereview.chromium.org/2436733004 .
2016-10-19 13:23:11 -07:00
Paul Berry
d631a142a4
Create a README and pubspec for the front_end package.
...
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2436733003 .
2016-10-19 12:14:38 -07:00
Paul Berry
4054d93fc9
Handle default formal parameters in LocalElementBuilder
...
Fixes #27615 .
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2437793002 .
2016-10-19 11:17:43 -07:00
Konstantin Shcheglov
38e2b55fff
Implement 'local' mode in TypeResolverVisitor.
...
If we start resolution not from CompilationUnit, we need to put type
parameters into scopes. This is not implemented yet.
R=brianwilkerson@google.com , paulberry@google.com
BUG=
Review URL: https://codereview.chromium.org/2432273003 .
2016-10-19 09:47:40 -07:00
Ryan Macnak
5ee94eb837
Generate an app snapshot even if the training run does a hard exit.
...
(As the analyzer and pub do.)
R=asiva@google.com
Review URL: https://codereview.chromium.org/2426843002 .
2016-10-19 09:37:47 -07:00
Zachary Anderson
8a909671e6
Fix Double.pow() for arm and mips
...
R=fschneider@google.com
Review URL: https://codereview.chromium.org/2433933003 .
2016-10-19 09:01:55 -07:00
William Hesse
cdfe272b37
Update status for chrome on macos co19 tests.
...
BUG=
Review URL: https://codereview.chromium.org/2434773002 .
2016-10-19 14:27:27 +02:00
Zachary Anderson
088347fd9d
Reland "Add DoubleTestOp instruction"
...
Fixes DoubteTestOpInstr::AttributesEqual()
Review URL: https://codereview.chromium.org/2433813002 .
2016-10-18 22:46:36 -07:00
Zachary Anderson
f6296cc91b
Revert "Add DoubleTestOp instruction"
...
Review URL: https://codereview.chromium.org/2432903002 .
2016-10-18 21:57:05 -07:00
Zachary Anderson
fa855cf1c2
Add DoubleTestOp instruction
...
This gives a ~5% bump to the Flutter layout benchmark
in checked mode.
R=fschneider@google.com
Review URL: https://codereview.chromium.org/2423843002 .
2016-10-18 21:24:12 -07:00
Stephen Adams
5a646b7827
Property extraction getters (tear-offs) are effect-free
...
This mostly allows better scheduling of the surrounding instructions.
R=sigmund@google.com
Review URL: https://codereview.chromium.org/2430683003 .
2016-10-18 20:13:26 -07:00
Konstantin Shcheglov
b64e116422
Add TypeResolverMode and implement 'api' in TypeResolverVisitor.
...
R=brianwilkerson@google.com , paulberry@google.com
BUG=
Review URL: https://codereview.chromium.org/2428233002 .
2016-10-18 19:31:14 -07:00
Brian Wilkerson
d378dfcff8
Fix error codes (issue 27608)
...
R=scheglov@google.com
Review URL: https://codereview.chromium.org/2430873002 .
2016-10-18 17:30:04 -07:00
Florian Schneider
197d56d853
VM: Fix another dead-lock in background compiler.
...
Grabbing the lock must always take safepoints into account.
Observed as a flaky timeout on standalone/http_launch_test.
BUG=
R=asiva@google.com
Review URL: https://codereview.chromium.org/2426213002 .
2016-10-18 16:53:10 -07:00
Jacob Richman
4f6ce11bc8
Emulate compiling a source file in the context of an existing library. Add --debugger-compile flag that performs permissive dynamic calls on private members. Polish web_command.dart adding the ability to evaluate code in the context of an existing library.
...
BUG=
R=brianwilkerson@google.com , jmesserly@google.com
Review URL: https://codereview.chromium.org/2423313002 .
2016-10-18 16:34:48 -07:00
Nicholas Shahan
3f67c24cb8
Revert "Fix library name insert and access inconsistency on windows"
...
This reverts commit d8505ee264
.
It was found to cause issues in the web file system when one file
was found to be within a root path.
R=jacobr@google.com
Review URL: https://codereview.chromium.org/2432603002 .
2016-10-18 16:25:03 -07:00
Jennifer Messerly
7b5e6be4a8
fix #27284 , allow FutureUnionType to be preserved during generic inference
...
R=leafp@google.com
Review URL: https://codereview.chromium.org/2425143002 .
2016-10-18 15:55:27 -07:00
Konstantin Shcheglov
412cceb7d0
Add tests showing using LocalElementBuilder separately, for variable or body.
...
R=brianwilkerson@google.com , paulberry@google.com
BUG=
Review URL: https://codereview.chromium.org/2428773004 .
2016-10-18 14:38:06 -07:00
Paul Berry
8fca9ba968
First cut at a file system abstraction for the front end.
...
R=brianwilkerson@google.com , scheglov@google.com
Review URL: https://codereview.chromium.org/2426773004 .
2016-10-18 14:10:14 -07:00
Konstantin Shcheglov
19702989a8
Get rid of _inFunction in ApiElementBuilder.
...
This visitor does not enter into function bodies.
R=brianwilkerson@google.com
BUG=
Review URL: https://codereview.chromium.org/2430513004 .
2016-10-18 14:03:42 -07:00