Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

On demandify region mapping #41662

Merged
merged 11 commits into from
May 2, 2017

Commits on Apr 30, 2017

  1. On-demandify region mapping

    cramertj authored and nikomatsakis committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    eff39b7 View commit details
    Browse the repository at this point in the history
  2. Remove RefCells from RegionMaps

    cramertj authored and nikomatsakis committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    119c38e View commit details
    Browse the repository at this point in the history
  3. remove ROOT_CODE_EXTENT and DUMMY_CODE_EXTENT

    Instead, thread around `Option<CodeExtent>` where applicable.
    nikomatsakis committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    55d6066 View commit details
    Browse the repository at this point in the history
  4. intern CodeExtents

    Make a `CodeExtent<'tcx>` be something allocated in an arena
    instead of an index into the `RegionMaps`.
    nikomatsakis committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    c7dc39d View commit details
    Browse the repository at this point in the history
  5. introduce per-fn RegionMaps

    Instead of requesting the region maps for the entire crate, request for
    a given item etc. Several bits of code were modified to take
    `&RegionMaps` as input (e.g., the `resolve_regions_and_report_errors()`
    function). I am not totally happy with this setup -- I *think* I'd
    rather have the region maps be part of typeck tables -- but at least the
    `RegionMaps` works in a "parallel" way to `FreeRegionMap`, so it's not
    too bad. Given that I expect a lot of this code to go away with NLL, I
    didn't want to invest *too* much energy tweaking it.
    cramertj authored and nikomatsakis committed Apr 30, 2017
    Configuration menu
    Copy the full SHA
    73cd9bd View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    6c2f64b View commit details
    Browse the repository at this point in the history

Commits on May 1, 2017

  1. remove unused is_fn

    nikomatsakis committed May 1, 2017
    Configuration menu
    Copy the full SHA
    d4d74da View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c0434e2 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    3438cda View commit details
    Browse the repository at this point in the history
  4. kill regr test using ad-hoc lint

    This was a pretty narrow test to start with, and it's kind of a pain to
    update it. Not worth the trouble IMO.
    nikomatsakis committed May 1, 2017
    Configuration menu
    Copy the full SHA
    b393d64 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2017

  1. Configuration menu
    Copy the full SHA
    c008f05 View commit details
    Browse the repository at this point in the history