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

Issue 2742333005: Move some of analyzer's code generation utilities into front_end. (Closed)

Created:
3 years, 9 months ago by Paul Berry
Modified:
3 years, 9 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Move some of analyzer's code generation utilities into front_end. These utilities are needed for generating the code that supports summaries; in order to move summary logic into front_end, we will need these code generation tools in front_end as well. Also make a minor change to summary code generation logic so that it talks to the file system directly rather than depending on analyzer's file system abstraction. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/c78b4d66bb3e0440fdd3020b45f3d00d78082525

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+273 lines, -256 lines) Patch
M pkg/analysis_server/tool/spec/check_all_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/tool/spec/codegen_analysis_server.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analysis_server/tool/spec/codegen_dart_protocol.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analysis_server/tool/spec/codegen_inttest_methods.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analysis_server/tool/spec/codegen_java.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analysis_server/tool/spec/codegen_java_types.dart View 2 chunks +4 lines, -1 line 0 comments Download
M pkg/analysis_server/tool/spec/codegen_matchers.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analysis_server/tool/spec/generate_all.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analysis_server/tool/spec/to_html.dart View 6 chunks +25 lines, -23 lines 0 comments Download
M pkg/analyzer/lib/src/codegen/tools.dart View 3 chunks +0 lines, -217 lines 0 comments Download
M pkg/analyzer/tool/summary/check_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/tool/summary/generate.dart View 2 chunks +5 lines, -8 lines 0 comments Download
M pkg/analyzer/tool/task_dependency_graph/check_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/tool/task_dependency_graph/generate.dart View 2 chunks +1 line, -1 line 0 comments Download
M pkg/analyzer_plugin/tool/spec/check_all_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer_plugin/tool/spec/codegen_dart_protocol.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analyzer_plugin/tool/spec/codegen_inttest_methods.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analyzer_plugin/tool/spec/codegen_matchers.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analyzer_plugin/tool/spec/codegen_protocol_constants.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analyzer_plugin/tool/spec/generate_all.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer_plugin/tool/spec/to_html.dart View 1 chunk +1 line, -0 lines 0 comments Download
A pkg/front_end/lib/src/codegen/tools.dart View 1 chunk +221 lines, -0 lines 0 comments Download
M pkg/front_end/test/subpackage_relationships_test.dart View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
Paul Berry
3 years, 9 months ago (2017-03-14 17:46:12 UTC) #3
Brian Wilkerson
It might be cleaner for the tools.dart in analyzer to export the tools.dart in front_end ...
3 years, 9 months ago (2017-03-14 18:00:51 UTC) #4
Paul Berry
On 2017/03/14 18:00:51, Brian Wilkerson wrote: > It might be cleaner for the tools.dart in ...
3 years, 9 months ago (2017-03-14 19:26:09 UTC) #5
Paul Berry
3 years, 9 months ago (2017-03-14 19:30:25 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
c78b4d66bb3e0440fdd3020b45f3d00d78082525 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698