Handle continue and break in for-in and while loops

Handle continue and break in for-in and while loops

R=sigmund@google.com

Committed: https://github.com/dart-lang/sdk/commit/7d026b538ea04685cb5bae36194df804f5f6b5fb

[Johnni Winther, 2017-06-26 13:06:11]

johnniwinther@google.com changed reviewers:
+ sigmund@google.com

[Siggi Cherem (dart-lang), 2017-06-26 17:25:28]

lgtm, w/comments

https://codereview.chromium.org/2956883002/diff/1/pkg/compiler/lib/src/kernel...
File pkg/compiler/lib/src/kernel/kernel_backend_strategy.dart (right):

https://codereview.chromium.org/2956883002/diff/1/pkg/compiler/lib/src/kernel...
pkg/compiler/lib/src/kernel/kernel_backend_strategy.dart:420: node is
ir.DoStatement;
remove this `.parent` in getJumpTargetForDo as well? Maybe add a test for it
too?

[Johnni Winther, 2017-06-27 09:02:12]

https://codereview.chromium.org/2956883002/diff/1/pkg/compiler/lib/src/kernel...
File pkg/compiler/lib/src/kernel/kernel_backend_strategy.dart (right):

https://codereview.chromium.org/2956883002/diff/1/pkg/compiler/lib/src/kernel...
pkg/compiler/lib/src/kernel/kernel_backend_strategy.dart:420: node is
ir.DoStatement;
On 2017/06/26 17:25:28, Siggi Cherem (dart-lang) wrote:
> remove this `.parent` in getJumpTargetForDo as well? Maybe add a test for it
> too?

Removed. Do statements are not handled yet.

[Johnni Winther, 2017-06-27 09:07:23]

Description was changed from

==========
Handle continue and break in for-in and while loops
==========

to

==========
Handle continue and break in for-in and while loops

R=sigmund@google.com

Committed:
https://github.com/dart-lang/sdk/commit/7d026b538ea04685cb5bae36194df804f5f6b5fb
==========

[Johnni Winther, 2017-06-27 09:07:25]

Committed patchset #2 (id:20001) manually as
7d026b538ea04685cb5bae36194df804f5f6b5fb (presubmit successful).