Skip to content

Conversation

compiler-errors
Copy link
Member

Regions don't really mean anything anyways during hir typeck.

If this erase_regions makes anyone nervous, it's probably equally valid to just equate the types using a type relation, but regardless we should not be using strict type equality while region variables are present.

Fixes #103384

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

r? @eholk

(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 Oct 22, 2022
@eholk
Copy link
Contributor

eholk commented Nov 18, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 18, 2022

📌 Commit 8333253 has been approved by eholk

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 18, 2022
@bors
Copy link
Collaborator

bors commented Nov 20, 2022

⌛ Testing commit 8333253 with merge 9cdfe03...

@bors
Copy link
Collaborator

bors commented Nov 20, 2022

☀️ Test successful - checks-actions
Approved by: eholk
Pushing 9cdfe03 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 20, 2022
@bors bors merged commit 9cdfe03 into rust-lang:master Nov 20, 2022
@rustbot rustbot added this to the 1.67.0 milestone Nov 20, 2022
@rust-timer
Copy link
Collaborator

Finished benchmarking commit (9cdfe03): comparison URL.

Overall result: ❌ regressions - ACTION NEEDED

Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please open an issue or create a new PR that fixes the regressions, add a comment linking to the newly created issue or PR, and then add the perf-regression-triaged label to this PR.

@rustbot label: +perf-regression
cc @rust-lang/wg-compiler-performance

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

mean range count
Regressions ❌
(primary)
0.7% [0.6%, 1.0%] 7
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
- - 0
All ❌✅ (primary) 0.7% [0.6%, 1.0%] 7

Max RSS (memory usage)

This benchmark run did not return any relevant results for this metric.

Cycles

This benchmark run did not return any relevant results for this metric.

@rustbot rustbot added the perf-regression Performance regression. label Nov 20, 2022
@lqd
Copy link
Member

lqd commented Nov 20, 2022

I'd think this is the bitmaps noise I expected to see when looking at today's results.

image

@compiler-errors
Copy link
Member Author

Yeah, I think it's noise too. This erase_regions call I added is neither expensive nor hot.

@lqd
Copy link
Member

lqd commented Nov 21, 2022

bitmaps is currently being noisy

@rustbot label: +perf-regression-triaged

@rustbot rustbot added the perf-regression-triaged The performance regression has been triaged. label Nov 21, 2022
Aaron1011 pushed a commit to Aaron1011/rust that referenced this pull request Jan 6, 2023
…, r=eholk

Check fat pointer metadata compatibility modulo regions

Regions don't really mean anything anyways during hir typeck.

If this `erase_regions` makes anyone nervous, it's probably equally valid to just equate the types using a type relation, but regardless we should _not_ be using strict type equality while region variables are present.

Fixes rust-lang#103384
@compiler-errors compiler-errors deleted the metadata-mod-regions branch August 11, 2023 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. perf-regression Performance regression. perf-regression-triaged The performance regression has been triaged. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

Removing an identity pointer cast can cause vtable kind unification for projected types to fail
7 participants