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 16 pull requests #52735

Merged
merged 59 commits into from
Jul 26, 2018
Merged

Rollup of 16 pull requests #52735

merged 59 commits into from
Jul 26, 2018

Commits on Jul 23, 2018

  1. Point at internal span in format string

    Esteban Küber committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    38abca8 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    87f0c1f View commit details
    Browse the repository at this point in the history
  3. Point at incorrect named arg in format string

    Esteban Küber committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    4230659 View commit details
    Browse the repository at this point in the history
  4. Don't match on region kinds when reporting NLL errors

    With NLL region kinds are always ReVar
    matthewjasper committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    338d545 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    6bcf877 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    c55a698 View commit details
    Browse the repository at this point in the history
  7. Add test for raw string

    estebank committed Jul 23, 2018
    Configuration menu
    Copy the full SHA
    bde2be0 View commit details
    Browse the repository at this point in the history

Commits on Jul 24, 2018

  1. Pass multiple linker arguments rather than concatenate with commas; -…

    …l library -> -llibrary to work with apple's ld
    alecmocatta committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    22d0ab0 View commit details
    Browse the repository at this point in the history
  2. Add tests for rust-lang#34784

    wesleywiser committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    5363911 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    296a179 View commit details
    Browse the repository at this point in the history
  4. Add test for rust-lang#33264

    wesleywiser committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    63ed6a1 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    bbbbf2d View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    8c5ef0a View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    3290774 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    715005c View commit details
    Browse the repository at this point in the history
  9. Mark the suggestion applicable

    csmoe committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    27c0d56 View commit details
    Browse the repository at this point in the history
  10. Fix ui test

    csmoe committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    a71deb2 View commit details
    Browse the repository at this point in the history
  11. update the stdsimd submodule

    to bring in the doc generation fix
    japaric committed Jul 24, 2018
    Configuration menu
    Copy the full SHA
    9808f7c View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    f487e39 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    f9e3762 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    75d2226 View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    4d8aa59 View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2018

  1. Suggest in separate line

    csmoe committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    9492176 View commit details
    Browse the repository at this point in the history
  2. Add regression test for issue rust-lang#18804

    Signed-off-by: Gabriel Smith <ga29smith@gmail.com>
    yodaldevoid committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    e8bc064 View commit details
    Browse the repository at this point in the history
  3. Properly set the linkage type on non-local statics

    Fixes issue rust-lang#18804
    
    Signed-off-by: Gabriel Smith <ga29smith@gmail.com>
    yodaldevoid committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    a20262c View commit details
    Browse the repository at this point in the history
  4. Deduplicate linkage checking code for statics

    Signed-off-by: Gabriel Smith <ga29smith@gmail.com>
    yodaldevoid committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    0bcbe91 View commit details
    Browse the repository at this point in the history
  5. Disable regression test for issue rust-lang#18804 on Emscripten and A…

    …smjs
    
    The Emscripten compiler does not support weak symbols at the moment.
    
    Signed-off-by: Gabriel Smith <ga29smith@gmail.com>
    yodaldevoid committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    e58e7b0 View commit details
    Browse the repository at this point in the history
  6. Fix test

    csmoe committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    40c0339 View commit details
    Browse the repository at this point in the history
  7. Update comment and do suggest

    csmoe committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    d5256b7 View commit details
    Browse the repository at this point in the history
  8. Update ui test

    csmoe committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    1d79588 View commit details
    Browse the repository at this point in the history
  9. Fix unittest

    estebank committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    3298b9f View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    7bd94e0 View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    9a893cc View commit details
    Browse the repository at this point in the history
  12. ARM: expose rclass and dsp target features

    - `dsp`: the subtarget supports the DSP (saturating arith. and such)
             instructions
    - `rclass`: target is a Cortex-R
    
    Both features are useful to support ARM MCUs on `coresimd`.
    
    Note: Cortex-R52 is the first Armv8-R with `neon` support
    paoloteti committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    580f437 View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    f653bf4 View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    ef5fba0 View commit details
    Browse the repository at this point in the history
  15. Clarify what a task is

    MajorBreakfast committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    eacfd72 View commit details
    Browse the repository at this point in the history
  16. Place the ignore comments in the correct file for test issue-18804

    Signed-off-by: Gabriel Smith <ga29smith@gmail.com>
    yodaldevoid committed Jul 25, 2018
    Configuration menu
    Copy the full SHA
    be5b668 View commit details
    Browse the repository at this point in the history
  17. Configuration menu
    Copy the full SHA
    abb704e View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2018

  1. Configuration menu
    Copy the full SHA
    bce8a91 View commit details
    Browse the repository at this point in the history
  2. rustc: Register crates under their real names

    Whenever we register a crate into the crate store, make sure to use the real
    name mentioned in the metadata instead of the name mentioned in the `extern
    crate` statement, as the statement can be wrong!
    
    Closes rust-lang#51796
    alexcrichton committed Jul 26, 2018
    Configuration menu
    Copy the full SHA
    f069a57 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    051eddd View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    cc2bd71 View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#52558 - wesleywiser:ice_melting, r=estebank

    Add tests for ICEs which no longer repro
    
    Adds tests for some ICEs which no longer repro and closes the associated issues.
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    e29f15b View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#52610 - MajorBreakfast:task-terminology, r=…

    …cramertj
    
    Clarify what a task is
    
    Currently we call two distinct concepts "task":
    1. The top-level future that is polled until completion
    2. The lightweight "thread" that is responsible for polling the top-level future. What additional data beside the future is stored in this type varies between different `Executor` implementations.
    
    I'd prefer to return to the old formulation by @alexcrichton:
    ```rust
    /// A handle to a "task", which represents a single lightweight "thread" of
    /// execution driving a future to completion.
    pub struct Task {
    ```
    Source: [`task_impl/mod.rs` in futures-rs 0.1](https://github.com/rust-lang-nursery/futures-rs/blob/1328fc9e8af5737183df477c7501e6ea24ff2053/src/task_impl/mod.rs#L49-L50)
    
    I think that this change will make it much easier to explain everything.
    
    r? @aturon
    @cramertj
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    0127704 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#52617 - matthewjasper:remove-unused-code, r…

    …=nikomatsakis
    
    Don't match on region kinds when reporting NLL errors
    
    First half (by number of tests affected) of the changes to "does not live long enough".
    
    Now that lexical MIR borrowck is gone, region kinds are always ReVar, so matching on them to change errors does nothing.
    
    Changes "borrowed value only lives until here" to "`x` is dropped here while still borrowed".
    
    r? @pnkfelix  cc @nikomatsakis
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    2aeb765 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#52635 - yodaldevoid:issue_18804, r=oli-obk

    Fix #[linkage] propagation though generic functions
    
    Fixes rust-lang#18804
    
    In the non-local branch of `get_static` (where the fix was implemented) `span_fatal` had to be replaced with `bug!` as we have no span in that case.
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    c7555ce View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#52647 - csmoe:closure_arg_ignore, r=estebank

    Suggest to take and ignore args while closure args count mismatching
    
    Closes rust-lang#52473
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    662fb06 View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#52649 - estebank:fmt-span, r=oli-obk

    Point spans to inner elements of format strings
    
    - Point at missing positional specifiers in string literal
    ```
    error: invalid reference to positional arguments 3, 4 and 5 (there are 3 arguments)
      --> $DIR/ifmt-bad-arg.rs:34:38
       |
    LL |     format!("{name} {value} {} {} {} {} {} {}", 0, name=1, value=2);
       |                                      ^^ ^^ ^^
       |
       = note: positional arguments are zero-based
    ```
    
    - Point at named formatting specifier in string literal
    ```
    error: there is no argument named `foo`
      --> $DIR/ifmt-bad-arg.rs:37:17
       |
    LL |     format!("{} {foo} {} {bar} {}", 1, 2, 3);
       |                 ^^^^^
    ```
    
    - Update label for formatting string in "multiple unused formatting arguments" to be more correct
    ```
    error: multiple unused formatting arguments
      --> $DIR/ifmt-bad-arg.rs:42:17
       |
    LL |     format!("", 1, 2);               //~ ERROR: multiple unused formatting arguments
       |             --  ^  ^
       |             |
       |             multiple missing formatting specifiers
    ```
    
    - When using `printf` string formatting, provide a structured suggestion instead of a note
    ```
    error: multiple unused formatting arguments
      --> $DIR/format-foreign.rs:12:30
       |
    LL |     println!("%.*3$s %s!/n", "Hello,", "World", 4); //~ ERROR multiple unused formatting arguments
       |              --------------  ^^^^^^^^  ^^^^^^^  ^
       |              |
       |              multiple missing formatting specifiers
       |
       = note: printf formatting not supported; see the documentation for `std::fmt`
    help: format specifiers in Rust are written using `{}`
       |
    LL |     println!("{:.2$} {}!/n", "Hello,", "World", 4); //~ ERROR multiple unused formatting arguments
       |               ^^^^^^ ^^
    ```
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    2aec4e8 View commit details
    Browse the repository at this point in the history
  11. Rollup merge of rust-lang#52654 - alecmocatta:master, r=alexcrichton

    Format linker args in a way that works for gcc and ld
    
    Pass multiple linker arguments rather than concatenate with commas (fixes rust-lang#52634).
    
    `-l library` -> `-llibrary` to work with apple's ld.
    
    To build with apple's ld I'm currently also passing `-C link-args="-arch x86_64 -macosx_version_min 10.13.0"`. I'll try and understand the latter flag better before PRing that.
    
    This PR currently works for me. Hopefully CI will pick up any grievous ramifications in other toolchains?
    
    Thanks to @alexcrichton for the pointer to the relevant code!
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    9f91195 View commit details
    Browse the repository at this point in the history
  12. Rollup merge of rust-lang#52667 - japaric:stdsimd-up, r=alexcrichton

    update the stdsimd submodule
    
    to bring in the doc generation fix
    
    r? @alexcrichton
    cc @Mark-Simulacrum
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    007177c View commit details
    Browse the repository at this point in the history
  13. Rollup merge of rust-lang#52674 - tinaun:patch-2, r=cramertj

    Impl Executor for Box<E: Executor>
    
    removes the need for the compatibility lib between futures 0.1 and 0.3 to use a wrapper type to implement Executor for Box<Executor>
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    7c09bab View commit details
    Browse the repository at this point in the history
  14. Rollup merge of rust-lang#52690 - paoloteti:rclass-dsp, r=alexcrichton

    ARM: expose `rclass` and `dsp` target features
    
    - `dsp`: the subtarget supports the DSP (saturating arith. and such)
             instructions
    - `rclass`: target is a Cortex-R
    
    Both features are useful to support ARM MCUs on `coresimd`.
    
    Note: Cortex-R52 is the first Armv8-R with `neon` support.
    
    r? @alexcrichton
    cc @japaric
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    37835be View commit details
    Browse the repository at this point in the history
  15. Rollup merge of rust-lang#52692 - ljedrz:sort_improvements, r=petroch…

    …enkov
    
    Improve readability in a few sorts
    
    Use `sort_by_key` where possible.
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    55fa4c7 View commit details
    Browse the repository at this point in the history
  16. Rollup merge of rust-lang#52695 - oli-obk:const_err_panic, r=petroche…

    …nkov
    
    Hide some lints which are not quite right the way they are reported to the user
    
    fixes rust-lang#52603
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    cc7a826 View commit details
    Browse the repository at this point in the history
  17. Rollup merge of rust-lang#52718 - fintelia:patch-2, r=cramertj

    State default capacity for BufReader/BufWriter
    
    Addresses rust-lang#52513
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    091a10e View commit details
    Browse the repository at this point in the history
  18. Rollup merge of rust-lang#52721 - cramertj:try-poll, r=aturon

    std::ops::Try impl for std::task::Poll
    
    I originally left out the `Try` impl for `Poll` because I was curious if we needed it, and @MajorBreakfast and I had discussed the potential for it to introduce confusion about exactly what control-flow was happening at different points. However, after porting a pretty significant chunk of Fuchsia over to futures 0.3, I discovered that I was *constantly* having to do repetitive matching on `Poll<Result<...>>` or `Poll<Option<Result<...>>>` in order to propagate errors correctly. `try_poll` (propagate `Poll::Ready(Err(..))`s) helped in some places, but it was far more common to need some form of conversion between `Result`, `Poll<Result<...>>`, and `Poll<Option<Result<...>>>`. The `Try` trait conveniently provides all of these conversions in addition to a more concise syntax (`?`), so I'd like to experiment with using these instead.
    
    cc @seanmonstar
    
    r? @aturon
    
    Note: this change means that far more futures 0.1 code can work without significant changes since it papers over the fact that `Result` is no longer at the top-level when using `Stream` and `Future` (since it's now `Poll<Result<...>>` or `Poll<Option<Result<...>>>` instead of `Result<Poll<..>>` and `Result<Poll<Option<...>>>`).
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    858adfe View commit details
    Browse the repository at this point in the history
  19. Rollup merge of rust-lang#52723 - alexcrichton:fix-extern-rename-ice,…

    … r=estebank
    
    rustc: Register crates under their real names
    
    Whenever we register a crate into the crate store, make sure to use the real
    name mentioned in the metadata instead of the name mentioned in the `extern
    crate` statement, as the statement can be wrong!
    
    Closes rust-lang#51796
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    d1e549c View commit details
    Browse the repository at this point in the history
  20. Rollup merge of rust-lang#52734 - nagisa:sparcfix, r=oli-obk

    sparc ABI issue - structure returning from function is returned in 64bit registers (with tests)
    
    Fixes rust-lang#52638
    
    Supersedes rust-lang#52730
    
    cc @psumbera
    Mark-Simulacrum authored Jul 26, 2018
    Configuration menu
    Copy the full SHA
    995d719 View commit details
    Browse the repository at this point in the history