]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Rollup merge of #88147 - FabianWolff:issue-88097, r=jackh726
authorJubilee <46493976+workingjubilee@users.noreply.github.com>
Sat, 11 Sep 2021 15:23:39 +0000 (08:23 -0700)
committerGitHub <noreply@github.com>
Sat, 11 Sep 2021 15:23:39 +0000 (08:23 -0700)
commit94cbefb52a601a9eb73396edcb286bb9e9eb8f0e
tree7be06618c25d688e8da96613a1240d7e954361c7
parent95b50eb6629a7877c06fb0ab545a6016ea579897
parentbbe3be9bf8d8e8503b8effbe392d0e449224b406
Rollup merge of #88147 - FabianWolff:issue-88097, r=jackh726

Fix non-capturing closure return type coercion

Fixes #88097. For the example given there:
```rust
fn peculiar() -> impl Fn(u8) -> u8 {
    return |x| x + 1
}
```
which incorrectly reports an error, I noticed something weird in the debug log:
```
DEBUG rustc_typeck::check::coercion coercion::try_find_coercion_lub([closure@test.rs:2:12: 2:21], [closure@test.rs:2:12: 2:21], exprs=1 exprs)
```
Apparently, `try_find_coercion_lub()` thinks that the LUB for two closure types always has to be a function pointer (which explains the `expected closure, found fn pointer` error in #88097). There is one corner case where that isn't true, though — namely, when the two closure types are equal, in which case the trivial LUB is the type itself. This PR fixes this by inserting an explicit check for type equality in `try_find_coercion_lub()`.
compiler/rustc_typeck/src/check/coercion.rs