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

Issue 1457733002: dart2js: Recanonicalize selectors in static fields between runs. (Closed)

Created:
5 years, 1 month ago by asgerf
Modified:
5 years, 1 month ago
Reviewers:
Johnni Winther
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

dart2js: Recanonicalize selectors in static fields between runs. This fixes a bug in batch-mode. Selectors are canonicalized upon construction, so two equal selectors are always the same object. To avoid memory leaks in batch mode, the compiler clears the canonical cache before each run. However, some selectors were stored in static fields and thus shared between runs. These selectors lost their canonical status when the cache was cleared. This patch "fixes" that by reinserting them in the cache after clearing it. BUG= R=johnniwinther@google.com Committed: https://github.com/dart-lang/sdk/commit/392bed583cce5470913d378d1f7577cfd7d83974

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -1 line) Patch
M pkg/compiler/lib/src/common/names.dart View 1 chunk +8 lines, -0 lines 0 comments Download
M pkg/compiler/lib/src/compiler.dart View 3 chunks +10 lines, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
asgerf
5 years, 1 month ago (2015-11-18 12:16:39 UTC) #3
Johnni Winther
lgtm
5 years, 1 month ago (2015-11-18 12:45:54 UTC) #4
asgerf
5 years, 1 month ago (2015-11-18 12:55:20 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
392bed583cce5470913d378d1f7577cfd7d83974 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698