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

Issue 2980813002: Pass backend Target into Kernel driver. (Closed)

Created:
3 years, 5 months ago by scheglov
Modified:
3 years, 5 months ago
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Pass backend Target into Kernel driver. Mock SDK in analyzer does not include all the internal libraries that VM requires, and probably does not have to for analysis. So, we might use NoneTarget to avoid requiring extra libraries and transformations. R=ahe@google.com, paulberry@google.com, sigmund@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/bead5672c6816ed8a76550054335610c2f68454a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -14 lines) Patch
M pkg/front_end/lib/src/incremental/kernel_driver.dart View 5 chunks +7 lines, -10 lines 0 comments Download
M pkg/front_end/lib/src/incremental_kernel_generator_impl.dart View 2 chunks +8 lines, -2 lines 0 comments Download
M pkg/front_end/test/src/incremental/kernel_driver_test.dart View 2 chunks +7 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
scheglov
3 years, 5 months ago (2017-07-13 04:48:22 UTC) #1
Paul Berry
lgtm
3 years, 5 months ago (2017-07-13 15:42:04 UTC) #2
scheglov
3 years, 5 months ago (2017-07-13 16:13:44 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
bead5672c6816ed8a76550054335610c2f68454a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698