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

[CRATER] separate type equality from mutual subtyping #97427

Closed
wants to merge 2 commits into from

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented May 26, 2022

cc #97156

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label May 26, 2022
@rust-highfive
Copy link
Collaborator

r? @davidtwco

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 26, 2022
@lcnr
Copy link
Contributor Author

lcnr commented May 26, 2022

r? @nikomatsakis

@lcnr
Copy link
Contributor Author

lcnr commented May 26, 2022

@bors try

@bors
Copy link
Contributor

bors commented May 26, 2022

⌛ Trying commit 1554eb1bbbb7d38be371ab397e11bf8c2fef49b2 with merge 54f7166aef449d83951621ef9cd9750293f42bbc...

@rust-log-analyzer

This comment has been minimized.

@lcnr
Copy link
Contributor Author

lcnr commented May 26, 2022

@bors retry

@bors
Copy link
Contributor

bors commented May 26, 2022

⌛ Trying commit 1554eb1bbbb7d38be371ab397e11bf8c2fef49b2 with merge c1245a7f927a572455320a69e31499110394fd30...

@rust-log-analyzer

This comment has been minimized.

@lcnr
Copy link
Contributor Author

lcnr commented May 27, 2022

@bors try

@bors
Copy link
Contributor

bors commented May 27, 2022

⌛ Trying commit 124673b with merge ba0adcb6607ec53c8b2f7626bcdbe21161b99c69...

@lcnr
Copy link
Contributor Author

lcnr commented May 27, 2022

@bors try

@bors
Copy link
Contributor

bors commented May 27, 2022

⌛ Trying commit de40841 with merge 4e3d7541a523df82cea75064e44e60ae9a008bcb...

@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-llvm-12 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
   Compiling tempfile v3.2.0
   Compiling serde_json v1.0.59
   Compiling lint-docs v0.1.0 (/checkout/src/tools/lint-docs)
    Finished release [optimized] target(s) in 6.06s
warning: the code example in lint `coherence_leak_check` in /checkout/compiler/rustc_lint_defs/src/builtin.rs failed to generate the expected output: did not find lint `coherence_leak_check` in output of example, got:

Rustbook (x86_64-unknown-linux-gnu) - rustc
Rustbook (x86_64-unknown-linux-gnu) - cargo
Rustbook (x86_64-unknown-linux-gnu) - embedded-book
---
doc tests for: /checkout/src/doc/rustc/src/tests/index.md
doc tests for: /checkout/src/doc/rustc/src/what-is-rustc.md
 finished in 1.246 seconds
Generating lint docs (x86_64-unknown-linux-gnu)
error: failed to test example in lint docs for `coherence_leak_check` in /checkout/compiler/rustc_lint_defs/src/builtin.rs:1433: did not find lint `coherence_leak_check` in output of example, got:


This error was generated by the lint-docs tool.
This error was generated by the lint-docs tool.
This tool extracts documentation for lints from the source code and places
them in the rustc book. See the declare_lint! documentation
https://doc.rust-lang.org/nightly/nightly-rustc/rustc_lint_defs/macro.declare_lint.html


To re-run these tests, run: ./x.py test --keep-stage=0 src/tools/lint-docs
The --keep-stage flag should be used if you have already built the compiler
Build completed unsuccessfully in 0:31:56

@bors
Copy link
Contributor

bors commented May 27, 2022

☀️ Try build successful - checks-actions
Build commit: 4e3d7541a523df82cea75064e44e60ae9a008bcb (4e3d7541a523df82cea75064e44e60ae9a008bcb)

@lcnr
Copy link
Contributor Author

lcnr commented May 27, 2022

@craterbot check 4e3d7541a523df82cea75064e44e60ae9a008bcb

@craterbot
Copy link
Collaborator

🚨 Error: failed to parse the command

🆘 If you have any trouble with Crater please ping @rust-lang/infra!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@lcnr
Copy link
Contributor Author

lcnr commented May 27, 2022

@craterbot check

@craterbot
Copy link
Collaborator

👌 Experiment pr-97427 created and queued.
🤖 Automatically detected try build 4e3d7541a523df82cea75064e44e60ae9a008bcb
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 27, 2022
@RalfJung
Copy link
Member

bidirectional subtyping

A note on terminology: "bidirectional typing" is a thing and it has nothing to do with what you do here, so to people like me your terminology is very confusing. :) I would recommend something like "mutual subtyping", or "symmetric closure of subtyping".

@lcnr lcnr changed the title [CRATER] separate type equality from bidirectional subtyping [CRATER] separate type equality from mutual subtyping May 30, 2022
@craterbot
Copy link
Collaborator

🚧 Experiment pr-97427 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot
Copy link
Collaborator

🎉 Experiment pr-97427 is completed!
📊 25 regressed and 9 fixed (236156 total)
📰 Open the full report.

⚠️ If you notice any spurious failure please add them to the blacklist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Jun 18, 2022
@Mark-Simulacrum
Copy link
Member

Closing as crate results are available whenever needed, but this PR otherwise doesn't serve any purpose.

bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 24, 2023
…r=<try>

Fix for TypeId exposes equality-by-subtyping vs normal-form-syntactic-equality unsoundness

Fixes rust-lang#97156

This PR revives rust-lang#97427 idea, it sits on top of rust-lang#118118 because the idea uncovered some problems with IATs.

r? `@lcnr`

This is ICEing yet for `tests/ui/traits/new-solver/escaping-bound-vars-in-writeback-normalization.rs` using the new trait solver.
After rust-lang#118118 and this ICE is fixed, we would need a rebase and a crater run.

Opening as a WIP for now.
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 28, 2023
…r=<try>

Fix for TypeId exposes equality-by-subtyping vs normal-form-syntactic-equality unsoundness

Fixes rust-lang#97156

This PR revives rust-lang#97427 idea, it sits on top of rust-lang#118118 because the idea uncovered some problems with IATs.

r? `@lcnr`

This is ICEing yet for `tests/ui/traits/new-solver/escaping-bound-vars-in-writeback-normalization.rs` using the new trait solver.
After rust-lang#118118 and this ICE is fixed, we would need a rebase and a crater run.

Opening as a WIP for now.
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 6, 2023
…r=<try>

Fix for TypeId exposes equality-by-subtyping vs normal-form-syntactic-equality unsoundness

Fixes rust-lang#97156

This PR revives rust-lang#97427 idea.

r? `@lcnr`
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 7, 2023
…r=<try>

Fix for TypeId exposes equality-by-subtyping vs normal-form-syntactic-equality unsoundness

Fixes rust-lang#97156

This PR revives rust-lang#97427 idea.

r? `@lcnr`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants