Skip to content

[CRATER] separate type equality from mutual subtyping #97427

New issue

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

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

Already on GitHub? # to your account

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
Contributor

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
Collaborator

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
Collaborator

bors commented May 26, 2022

⌛ Trying commit 1554eb1bbbb7d38be371ab397e11bf8c2fef49b2 with merge c1245a7f927a572455320a69e31499110394fd30...

@rust-log-analyzer

This comment has been minimized.

@lcnr lcnr force-pushed the equate-uwu-uwu branch from d2b47f0 to 124673b Compare May 27, 2022 06:19
@lcnr
Copy link
Contributor Author

lcnr commented May 27, 2022

@bors try

@bors
Copy link
Collaborator

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
Collaborator

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
Collaborator

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`
# for free to join this conversation on GitHub. Already have an account? # 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