Skip to content

Complete the various renamings of terms belonging to feature checked_type_aliases - #164528

Merged
rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
fmease:cta-renamings
Oct 23, 2026
Merged

rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
fmease:cta-renamings

Conversation

@fmease

@fmease fmease commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

Various terms used in the context of feature checked_type_aliases have undergone renamings. See PRs #140249 and #158758. However, these PRs didn't update every single occurrence. This PR rectifies that.

For context, the renamings are

  1. weak (alias (type)) → free (alias (type))
  2. lazy (type alias) → checked (type alias), LTA → CTA
  3. eager (type alias) → unchecked (type alias)

Moreover, I've renamed tests/ui/checked-type-alias/ to tests/ui/checked-type-aliases/ in accordance with the name of the feature, checked_type_aliases.

(No LLM was or will be used by me during the entire creation process of this PR)

@fmease fmease added C-cleanup Category: PRs that clean code up or issues documenting cleanup. F-checked_type_aliases `#![feature(checked_type_aliases)]` (formerly: `lazy_type_alias`) labels Oct 18, 2026
@rustbot rustbot added 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. labels Oct 18, 2026
@rustbot

rustbot commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

r? @chenyukang

rustbot has assigned @chenyukang.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 77 candidates
  • Random selection from 20 candidates


if !usage_spans.is_empty() {
// First, check if the ADT/LTA is (probably) cyclical. We say probably here, since we're
// First, check if the ADT is (probably) cyclical. We say probably here, since we're

@fmease fmease Oct 18, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since #141030 we no longer consider CTAs here.

View changes since the review

@chenyukang

Copy link
Copy Markdown
Member

with file name changed, the comment also need to updated?

 tests/ui/checked-type-aliases/constrained-late-bound-regions.rs:2   // Weak alias types constrain late-bound regions ...
 tests/ui/checked-type-aliases/unconstrained-late-bound-regions.rs:1 // Weak alias types only constrain ...
 tests/ui/coherence/orphan-check-free-aliases-covering.rs:1          // Weak aliases cover type parameters ...
 tests/ui/coherence/orphan-check-free-aliases-not-covering.rs:1      // Weak aliases might not cover ...

Comment thread tests/ui/type-alias/lack-of-wfcheck.rs Outdated
//
// Listed below are ill-formed type system entities which we don't reject since they appear inside
// the definition of (eager) type aliases. These type aliases are intentionally not referenced from
// anywhere to prevent the eagerly expanded / instantiated aliased types from getting wfchecked
// the definition of (unchcked) type aliases. These type aliases are intentionally not referenced

@chenyukang chenyukang Oct 23, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// the definition of (unchcked) type aliases. These type aliases are intentionally not referenced
// the definition of (unchecked) type aliases. These type aliases are intentionally not referenced

View changes since the review

@chenyukang

Copy link
Copy Markdown
Member

r=me after fixed issues in comment.

@fmease

fmease commented Oct 23, 2026 •

Copy link
Copy Markdown
Member Author

with file name changed, the comment also need to updated?

Oops, I only did a case sensitive ripgrep for weak, eager & lazy ^^'. Gonna use rg -i now.

@fmease

fmease commented Oct 23, 2026

Copy link
Copy Markdown
Member Author

@bors r=chenyukang rollup

@rust-bors

rust-bors Bot commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

📌 Commit fe64545 has been tentatively approved by chenyukang

It will be put into the queue for this repository once PR CI succeeds.

@rust-bors rust-bors Bot 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 Oct 23, 2026
in accordance with the corresponding feature name, `checked_type_aliases`.
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Oct 23, 2026
Complete the various renamings of terms belonging to feature `checked_type_aliases`

Various terms used in the context of feature `checked_type_aliases` have undergone renamings. See PRs rust-lang#140249 and rust-lang#158758. However, these PRs didn't update every single occurrence. This PR rectifies that.

For context, the renamings are

1. weak (alias (type)) → free (alias (type))
2. lazy (type alias) → checked (type alias), LTA → CTA
3. eager (type alias) → unchecked (type alias)

Moreover, I've renamed `tests/ui/checked-type-alias/` to `tests/ui/checked-type-aliases/` in accordance with the name of the feature, `checked_type_aliases`.

<sub>(No LLM was or will be used by me during the entire creation process of this PR)</sub>
rust-bors Bot pushed a commit that referenced this pull request Oct 23, 2026
Rollup of 9 pull requests

Successful merges:

 - #164792 (rustc_codegen_cranelift subtree update)
 - #163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run)
 - #164781 (Bump dist-x86_64-illumos to Ubuntu 22.04)
 - #160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments)
 - #163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700)
 - #164167 (Account for `use foo as bar;` on resolution error suggestions)
 - #164438 (Make async drop shim proxy pinned)
 - #164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`)
 - #164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
@rust-bors
rust-bors Bot merged commit 3be2d85 into rust-lang:main Oct 23, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Checked Type Aliases (CTA) Oct 23, 2026
@rustbot rustbot added this to the 1.101.0 milestone Oct 23, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 23, 2026
Rollup merge of #164528 - fmease:cta-renamings, r=chenyukang

Complete the various renamings of terms belonging to feature `checked_type_aliases`

Various terms used in the context of feature `checked_type_aliases` have undergone renamings. See PRs #140249 and #158758. However, these PRs didn't update every single occurrence. This PR rectifies that.

For context, the renamings are

1. weak (alias (type)) → free (alias (type))
2. lazy (type alias) → checked (type alias), LTA → CTA
3. eager (type alias) → unchecked (type alias)

Moreover, I've renamed `tests/ui/checked-type-alias/` to `tests/ui/checked-type-aliases/` in accordance with the name of the feature, `checked_type_aliases`.

<sub>(No LLM was or will be used by me during the entire creation process of this PR)</sub>
@fmease
fmease deleted the cta-renamings branch October 23, 2026 18:55
programskillforverification pushed a commit to programskillforverification/miri that referenced this pull request Oct 24, 2026
Rollup of 9 pull requests

Successful merges:

 - rust-lang/rust#164792 (rustc_codegen_cranelift subtree update)
 - rust-lang/rust#163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run)
 - rust-lang/rust#164781 (Bump dist-x86_64-illumos to Ubuntu 22.04)
 - rust-lang/rust#160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments)
 - rust-lang/rust#163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700)
 - rust-lang/rust#164167 (Account for `use foo as bar;` on resolution error suggestions)
 - rust-lang/rust#164438 (Make async drop shim proxy pinned)
 - rust-lang/rust#164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`)
 - rust-lang/rust#164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
bjorn3 pushed a commit to rust-lang/rustc_codegen_cranelift that referenced this pull request Oct 26, 2026
Rollup of 9 pull requests

Successful merges:

 - rust-lang/rust#164792 (rustc_codegen_cranelift subtree update)
 - rust-lang/rust#163799 (std: keep value of `local_pointer!` until `thread_cleanup` is run)
 - rust-lang/rust#164781 (Bump dist-x86_64-illumos to Ubuntu 22.04)
 - rust-lang/rust#160902 (borrowck dataflow: respect borrow shallowness when killing borrows that conflict with assignments)
 - rust-lang/rust#163684 (suggest named lifetimes instead of `'_` in `use<...>` for E0700)
 - rust-lang/rust#164167 (Account for `use foo as bar;` on resolution error suggestions)
 - rust-lang/rust#164438 (Make async drop shim proxy pinned)
 - rust-lang/rust#164528 (Complete the various renamings of terms belonging to feature `checked_type_aliases`)
 - rust-lang/rust#164796 (miri: fix ICE when caller signature is c-variadic but callee is not)
@rust-timer

Copy link
Copy Markdown
Collaborator

Note

This PR was benchmarked as part of triage of its containing rollup: triage URL.

Finished benchmarking commit (575f569): comparison URL.

Overall result: ✅ improvements - no action needed

@rustbot label: -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

mean range count
Regressions ❌
(primary)
- - 0
Regressions ❌
(secondary)
- - 0
Improvements ✅
(primary)
- - 0
Improvements ✅
(secondary)
-0.2% [-0.4%, -0.0%] 12
All ❌✅ (primary) - - 0

Max RSS (memory usage)

This perf run didn't have relevant results for this metric.

Cycles

This perf run didn't have relevant results for this metric.

Binary size

This perf run didn't have relevant results for this metric.

Artifact size: 406.24 MiB -> 406.29 MiB (0.01%)

@fmease fmease mentioned this pull request Oct 29, 2026
2 of 6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-cleanup Category: PRs that clean code up or issues documenting cleanup. F-checked_type_aliases `#![feature(checked_type_aliases)]` (formerly: `lazy_type_alias`) 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

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants