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

Rollup of 5 pull requests #64597

Closed
wants to merge 28 commits into from
Closed

Commits on Sep 14, 2019

  1. Update installed compiler dependencies

    there is a soft-error for older toolchains
    from https://reviews.llvm.org/D57264 in preparation for the
    hard error in https://reviews.llvm.org/D66188
    andjo403 committed Sep 14, 2019
    Configuration menu
    Copy the full SHA
    166719a View commit details
    Browse the repository at this point in the history

Commits on Sep 17, 2019

  1. Configuration menu
    Copy the full SHA
    9f4351d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    222e920 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    3ef980a View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    34d3167 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    f9c7329 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    ceb72e4 View commit details
    Browse the repository at this point in the history

Commits on Sep 18, 2019

  1. Configuration menu
    Copy the full SHA
    5496a1f View commit details
    Browse the repository at this point in the history
  2. Temporarily add #[allow(unused)] for CI

    This can be removed once dataflow-based const validation is merged.
    ecstatic-morse committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    2886037 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    d583fef View commit details
    Browse the repository at this point in the history
  4. Fix Analysis example

    ecstatic-morse committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    cbaaf05 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    a526e4f View commit details
    Browse the repository at this point in the history
  6. Fix typo

    ecstatic-morse committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    19bf0f4 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    08aff1a View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    a7f5252 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    b4e94d9 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    3fc34a1 View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    71f7b1c View commit details
    Browse the repository at this point in the history
  12. review comments

    estebank committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    404a488 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    f43376a View commit details
    Browse the repository at this point in the history
  14. Fix a minor grammar nit, update UI tests

    James Munns committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    f233a4c View commit details
    Browse the repository at this point in the history
  15. Fix rebase

    estebank committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    829c3c3 View commit details
    Browse the repository at this point in the history
  16. Restore whitespace

    James Munns committed Sep 18, 2019
    Configuration menu
    Copy the full SHA
    333230f View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2019

  1. Rollup merge of rust-lang#63630 - andjo403:bump_compiler, r=nikomatsakis

    Update installed compiler dependencies
    
    As llvm have updated the minimum toolchain
    https://reviews.llvm.org/D66188
    Centril authored Sep 19, 2019
    Configuration menu
    Copy the full SHA
    f4f16cd View commit details
    Browse the repository at this point in the history
  2. Rollup merge of rust-lang#64151 - estebank:binding-error, r=varkor

    On obligation errors point at the unfulfilled binding when possible
    
    CC rust-lang#42855, rust-lang#64130, rust-lang#64135.
    Centril authored Sep 19, 2019
    Configuration menu
    Copy the full SHA
    e314d39 View commit details
    Browse the repository at this point in the history
  3. Rollup merge of rust-lang#64554 - lqd:polonius_tests4, r=nikomatsakis

    Polonius: more `ui` test suite fixes
    
    Since rust-lang#62736, new tests have been added, and the `run-pass` suite was merged into the `ui` suite.
    
    This PR adds the missing tests expectations for Polonius, and updates the existing ones where the NLL output has changed in some manner (e.g. ordering of notes)
    
    Those are the trivial cases, but a more-detailed explanation is available [in this write-up](https://hackmd.io/CjYB0fs4Q9CweyeTdKWyEg?both#26-async-awaitasync-borrowck-escaping-closure-errorrs-outputs-from-NLL-Polonius-diff) starting at test case 26: they are only differing in diagnostics and instances of other existing test cases differences.
    
    Only 3 of the 9020 tests are still "failing" at the moment (1 failure, 2 OOMs).
    
    r? @nikomatsakis
    Centril authored Sep 19, 2019
    Configuration menu
    Copy the full SHA
    9f8b6bd View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#64566 - ecstatic-morse:generic-dataflow, r=…

    …oli-obk
    
    A more generic interface for dataflow analysis
    
    rust-lang#64470 requires a transfer function that is slightly more complex than the typical `gen`/`kill` one. Namely, it must copy state bits between locals when assignments occur (see rust-lang#62547 for an attempt to make this fit into the existing framework). This PR contains a dataflow interface that allows for arbitrary transfer functions. The trade-off is efficiency: we can no longer coalesce transfer functions for blocks and must visit each statement individually while iterating to fixpoint.
    
    Another issue is that poorly behaved transfer functions can result in an analysis that fails to converge. `gen`/`kill` sets do not have this problem. I believe that, in order to guarantee convergence, flipping a bit from `false` to `true` in the entry set cannot cause an output bit to go from `true` to `false` (negate all preceding booleans when `true` is the bottom value). Perhaps someone with a more formal background can confirm and we can add a section to the docs?
    
    This approach is not maximally generic: it still requires that the lattice used for analysis is the powerset of values of `Analysis::Idx` for the `mir::Body` of interest. This can be done at a later date. Also, this is the bare minimum to get rust-lang#64470 working. I've not adapted the existing debug framework to work with the new analysis, so there are no `rustc_peek` tests either. I'm planning to do this after rust-lang#64470 is merged.
    
    Finally, my ultimate plan is to make the existing, `gen`/`kill`-based `BitDenotation` a special case of `generic::Analysis`. Currently they share a ton of code. I should be able to do this without changing any implementers of `BitDenotation`. Something like:
    
    ```rust
    struct GenKillAnalysis<A: BitDenotation> {
        trans_for_block: IndexVec<BasicBlock, GenKillSet<A::Idx>>,
        analysis: A,
    }
    
    impl<A> generic::Analysis for GenKillAnalysis<A> {
        // specializations of `apply_{partial,whole}_block_effect`...
    }
    ```
    
    r? @pnkfelix
    Centril authored Sep 19, 2019
    Configuration menu
    Copy the full SHA
    ce48352 View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#64591 - jamesmunns:grammar-fix, r=jonas-sch…

    …ievink
    
    Fix a minor grammar nit, update UI tests
    
    Minor fix, but I noticed it while debugging some code
    Centril authored Sep 19, 2019
    Configuration menu
    Copy the full SHA
    77bae1c View commit details
    Browse the repository at this point in the history