Skip to content

Implement tree-shaking based on MirrorsUsed #11524

Description

@peter-ahe-google

dart2js needs to respect MirrorsUsed metadata in dart:mirrors.

Activity

  1. peter-ahe-google commented on Jun 28, 2013

    @peter-ahe-google
    ContributorAuthor

    Added Library-Mirrors label.

  2. justinfagnani commented on Jul 26, 2013

    @justinfagnani
    Contributor

    Removed Priority-Medium label.
    Added Priority-High, C1 labels.

  3. peter-ahe-google commented on Jul 30, 2013

    @peter-ahe-google
    ContributorAuthor

    Issue #12125 has been merged into this issue.

  4. peter-ahe-google commented on Jul 30, 2013

    @peter-ahe-google
    ContributorAuthor
  5. peter-ahe-google commented on Jul 30, 2013

    @peter-ahe-google
    ContributorAuthor

    Marked this as blocking #6490.

  6. peter-ahe-google commented on Aug 7, 2013

    @peter-ahe-google
    ContributorAuthor

    CL 21242002 landed in r25873.

  7. peter-ahe-google commented on Aug 7, 2013

    @peter-ahe-google
    ContributorAuthor

    https://codereview.chromium.org/22301009/ reduces the number of symbols stored in compiled output.

  8. peter-ahe-google commented on Aug 7, 2013

    @peter-ahe-google
    ContributorAuthor

    CL 22301009 landed in r25878.

  9. larsbak commented on Aug 27, 2013

    @larsbak

    Added this to the M7 milestone.

  10. peter-ahe-google commented on Aug 27, 2013

    @peter-ahe-google
    ContributorAuthor

    I'm lowering the priority: MirrorsUsed is not fully implemented, but can be used to control tree-shaking.

    The missing part is supporting metaTargets, which we don't deem a high priority at the moment.


    Removed Priority-High label.
    Added Priority-Medium label.

  11. kasperl commented on Oct 2, 2013

    @kasperl

    Removed this from the M7 milestone.
    Added this to the M8 milestone.

  12. kasperl commented on Jun 4, 2014

    @kasperl

    Removed this from the M8 milestone.
    Added this to the 1.6 milestone.

  13. kasperl commented on Jul 10, 2014

    @kasperl

    Removed this from the 1.6 milestone.
    Added Oldschool-Milestone-1.6 label.

  14. kasperl commented on Aug 4, 2014

    @kasperl
  15. added
    P2A bug or feature request we're likely to work on
    type-bugIncorrect behavior (everything from a crash to more subtle misbehavior)
    and removed on Mar 1, 2016
  16. matanlurey commented on Mar 3, 2018

    @matanlurey
    Contributor

    Mirrors is no longer supported in Dart2JS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

P2A bug or feature request we're likely to work onclosed-obsoleteClosed as the reported issue is no longer relevantlibrary-mirrorstype-bugIncorrect behavior (everything from a crash to more subtle misbehavior)web-dart2js

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions