Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(521)

Issue 2996533003: Migrate language block 46 - canonical_const ... char_escape. (Closed)

Created:
3 years, 4 months ago by Bob Nystrom
Modified:
3 years, 4 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Migrate language block 46 - canonical_const ... char_escape. R=jmesserly@google.com Committed: https://github.com/dart-lang/sdk/commit/243d1c74eb0c7924540185e309862c88fca6fdae

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -2701 lines) Patch
D tests/language/canonical_const2_test.dart View 1 chunk +0 lines, -11 lines 0 comments Download
D tests/language/canonical_const3_test.dart View 1 chunk +0 lines, -31 lines 0 comments Download
D tests/language/canonical_const_test.dart View 1 chunk +0 lines, -46 lines 0 comments Download
D tests/language/cascade2_test.dart View 1 chunk +0 lines, -28 lines 0 comments Download
D tests/language/cascade_2_test.dart View 1 chunk +0 lines, -55 lines 0 comments Download
D tests/language/cascade_in_expression_function_test.dart View 1 chunk +0 lines, -26 lines 0 comments Download
D tests/language/cascade_in_initializer_list_test.dart View 1 chunk +0 lines, -29 lines 0 comments Download
D tests/language/cascade_precedence_test.dart View 1 chunk +0 lines, -183 lines 0 comments Download
D tests/language/cascade_test.dart View 1 chunk +0 lines, -88 lines 0 comments Download
D tests/language/case_expression_with_assignment_test.dart View 1 chunk +0 lines, -28 lines 0 comments Download
D tests/language/cast2_test.dart View 1 chunk +0 lines, -34 lines 0 comments Download
D tests/language/cast_test.dart View 1 chunk +0 lines, -74 lines 0 comments Download
D tests/language/catch_liveness_test.dart View 1 chunk +0 lines, -29 lines 0 comments Download
D tests/language/cha_deopt1_deferred_lib.dart View 1 chunk +0 lines, -11 lines 0 comments Download
D tests/language/cha_deopt1_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language/cha_deopt2_deferred_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language/cha_deopt2_lib.dart View 1 chunk +0 lines, -17 lines 0 comments Download
D tests/language/cha_deopt3_deferred_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language/cha_deopt3_lib.dart View 1 chunk +0 lines, -17 lines 0 comments Download
D tests/language/char_escape_test.dart View 1 chunk +0 lines, -533 lines 0 comments Download
M tests/language/language.status View 1 chunk +0 lines, -3 lines 0 comments Download
M tests/language/language_analyzer2.status View 1 chunk +0 lines, -1 line 0 comments Download
M tests/language/language_dart2js.status View 3 chunks +0 lines, -33 lines 0 comments Download
M tests/language/language_kernel.status View 1 chunk +0 lines, -3 lines 0 comments Download
A + tests/language_2/canonical_const2_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/canonical_const3_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
A + tests/language_2/canonical_const_test.dart View 1 chunk +0 lines, -2 lines 0 comments Download
A + tests/language_2/cascade2_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
A + tests/language_2/cascade_2_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cascade_in_expression_function_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cascade_in_initializer_list_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cascade_nested_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cascade_precedence_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cascade_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/case_expression_with_assignment_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cast2_test.dart View 1 chunk +1 line, -1 line 0 comments Download
A + tests/language_2/cast_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
A + tests/language_2/catch_liveness_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt1_deferred_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt1_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt1_test.dart View 1 chunk +1 line, -1 line 0 comments Download
A + tests/language_2/cha_deopt2_deferred_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt2_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt2_test.dart View 1 chunk +2 lines, -1 line 2 comments Download
A + tests/language_2/cha_deopt3_deferred_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt3_lib.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/language_2/cha_deopt3_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
A + tests/language_2/char_escape_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
M tests/language_2/language_2.status View 1 chunk +5 lines, -0 lines 0 comments Download
M tests/language_2/language_2_analyzer.status View 1 chunk +3 lines, -0 lines 0 comments Download
M tests/language_2/language_2_dart2js.status View 4 chunks +35 lines, -0 lines 0 comments Download
M tests/language_2/language_2_dartdevc.status View 1 chunk +2 lines, -0 lines 0 comments Download
M tests/language_2/language_2_kernel.status View 2 chunks +6 lines, -0 lines 0 comments Download
M tests/language_2/language_2_precompiled.status View 1 chunk +2 lines, -0 lines 0 comments Download
M tests/language_2/language_2_vm.status View 1 chunk +2 lines, -0 lines 0 comments Download
D tests/language_strong/canonical_const2_test.dart View 1 chunk +0 lines, -11 lines 0 comments Download
D tests/language_strong/canonical_const3_test.dart View 1 chunk +0 lines, -31 lines 0 comments Download
D tests/language_strong/canonical_const_test.dart View 1 chunk +0 lines, -46 lines 0 comments Download
D tests/language_strong/cascade2_test.dart View 1 chunk +0 lines, -29 lines 0 comments Download
D tests/language_strong/cascade_2_test.dart View 1 chunk +0 lines, -55 lines 0 comments Download
D tests/language_strong/cascade_in_expression_function_test.dart View 1 chunk +0 lines, -26 lines 0 comments Download
D tests/language_strong/cascade_in_initializer_list_test.dart View 1 chunk +0 lines, -29 lines 0 comments Download
D tests/language_strong/cascade_nested_test.dart View 1 chunk +0 lines, -24 lines 0 comments Download
D tests/language_strong/cascade_precedence_test.dart View 1 chunk +0 lines, -183 lines 0 comments Download
D tests/language_strong/cascade_test.dart View 1 chunk +0 lines, -88 lines 0 comments Download
D tests/language_strong/cast2_test.dart View 1 chunk +0 lines, -34 lines 0 comments Download
D tests/language_strong/cast_test.dart View 1 chunk +0 lines, -74 lines 0 comments Download
D tests/language_strong/catch_liveness_test.dart View 1 chunk +0 lines, -29 lines 0 comments Download
D tests/language_strong/cha_deopt1_deferred_lib.dart View 1 chunk +0 lines, -11 lines 0 comments Download
D tests/language_strong/cha_deopt1_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language_strong/cha_deopt1_test.dart View 1 chunk +0 lines, -48 lines 0 comments Download
D tests/language_strong/cha_deopt2_deferred_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language_strong/cha_deopt2_lib.dart View 1 chunk +0 lines, -17 lines 0 comments Download
D tests/language_strong/cha_deopt2_test.dart View 1 chunk +0 lines, -39 lines 0 comments Download
D tests/language_strong/cha_deopt3_deferred_lib.dart View 1 chunk +0 lines, -9 lines 0 comments Download
D tests/language_strong/cha_deopt3_lib.dart View 1 chunk +0 lines, -17 lines 0 comments Download
D tests/language_strong/cha_deopt3_test.dart View 1 chunk +0 lines, -39 lines 0 comments Download
D tests/language_strong/char_escape_test.dart View 1 chunk +0 lines, -533 lines 0 comments Download
M tests/language_strong/language_strong.status View 2 chunks +0 lines, -5 lines 0 comments Download
M tests/language_strong/language_strong_kernel.status View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
Bob Nystrom
3 years, 4 months ago (2017-08-04 22:45:52 UTC) #2
Jennifer Messerly
LGTM, did you test this in DDC? ./tools/test.py -m release -r chrome -c dartdevc --strong ...
3 years, 4 months ago (2017-08-04 23:02:06 UTC) #3
Bob Nystrom
On 2017/08/04 23:02:06, Jennifer Messerly wrote: > LGTM, did you test this in DDC? > ...
3 years, 4 months ago (2017-08-04 23:51:49 UTC) #4
Bob Nystrom
Committed patchset #1 (id:1) manually as 243d1c74eb0c7924540185e309862c88fca6fdae (presubmit successful).
3 years, 4 months ago (2017-08-04 23:52:18 UTC) #6
Bob Nystrom
3 years, 4 months ago (2017-08-05 00:16:25 UTC) #7
Message was sent while issue was closed.
https://codereview.chromium.org/2996533003/diff/1/tests/language_2/cha_deopt2...
File tests/language_2/cha_deopt2_test.dart (right):

https://codereview.chromium.org/2996533003/diff/1/tests/language_2/cha_deopt2...
tests/language_2/cha_deopt2_test.dart:5: //
VMOptions=--optimization-counter-threshold=100 --no-background-compilation
--no-use-field-guards
On 2017/08/04 23:02:06, Jennifer Messerly wrote:
> just curious, what's this about? what does test.dart do when it sees two
> VMOptions lines?

I think it runs the test twice, once with each info. See:

https://github.com/dart-lang/sdk/blob/master/tools/testing/dart/test_suite.da...

Powered by Google App Engine
This is Rietveld 408576698