Skip to content

Implement demangling for #[splat]#90

Merged
wesleywiser merged 2 commits into
rust-lang:mainfrom
teor2345:splat-mangling
Jul 8, 2026
Merged

Implement demangling for #[splat]#90
wesleywiser merged 2 commits into
rust-lang:mainfrom
teor2345:splat-mangling

Conversation

@teor2345

@teor2345 teor2345 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

This PR implements demangling for splat, and adds tests.

The symbols for the tests are from PR rust-lang/rust#158890, UI tests:

This is part of the splat lang experiment: rust-lang/rust#153629

It is needed to fix symbol clashes between function pointer types, see rust-lang/rust#158644

@wesleywiser wesleywiser merged commit 53ea128 into rust-lang:main Jul 8, 2026
6 checks passed
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 13, 2026
…rk-Simulacrum

Bump rustc-demangle to 0.1.28

This PR is part of the splat lang experiment: rust-lang#153629

It brings in the changes from rust-lang/rustc-demangle#90, which encode splatted types with a `w` prefix in type mangling. This is required to fix the symbol clashes in rust-lang#158644.

We need to upgrade the compiler and standard library to avoid "can't demangle" panics or errors. Some tools might work without the upgrade, but we might as well do it there as well.

@rustbot label +C-enhancement +F-splat +T-compiler +A-name-mangling
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 13, 2026
…rk-Simulacrum

Bump rustc-demangle to 0.1.28

This PR is part of the splat lang experiment: rust-lang#153629

It brings in the changes from rust-lang/rustc-demangle#90, which encode splatted types with a `w` prefix in type mangling. This is required to fix the symbol clashes in rust-lang#158644.

We need to upgrade the compiler and standard library to avoid "can't demangle" panics or errors. Some tools might work without the upgrade, but we might as well do it there as well.

@rustbot label +C-enhancement +F-splat +T-compiler +A-name-mangling
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 13, 2026
…rk-Simulacrum

Bump rustc-demangle to 0.1.28

This PR is part of the splat lang experiment: rust-lang#153629

It brings in the changes from rust-lang/rustc-demangle#90, which encode splatted types with a `w` prefix in type mangling. This is required to fix the symbol clashes in rust-lang#158644.

We need to upgrade the compiler and standard library to avoid "can't demangle" panics or errors. Some tools might work without the upgrade, but we might as well do it there as well.

@rustbot label +C-enhancement +F-splat +T-compiler +A-name-mangling
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 13, 2026
…rk-Simulacrum

Bump rustc-demangle to 0.1.28

This PR is part of the splat lang experiment: rust-lang#153629

It brings in the changes from rust-lang/rustc-demangle#90, which encode splatted types with a `w` prefix in type mangling. This is required to fix the symbol clashes in rust-lang#158644.

We need to upgrade the compiler and standard library to avoid "can't demangle" panics or errors. Some tools might work without the upgrade, but we might as well do it there as well.

@rustbot label +C-enhancement +F-splat +T-compiler +A-name-mangling
rust-timer added a commit to rust-lang/rust that referenced this pull request Jul 13, 2026
Rollup merge of #159202 - teor2345:bump-rustc-demangle, r=Mark-Simulacrum

Bump rustc-demangle to 0.1.28

This PR is part of the splat lang experiment: #153629

It brings in the changes from rust-lang/rustc-demangle#90, which encode splatted types with a `w` prefix in type mangling. This is required to fix the symbol clashes in #158644.

We need to upgrade the compiler and standard library to avoid "can't demangle" panics or errors. Some tools might work without the upgrade, but we might as well do it there as well.

@rustbot label +C-enhancement +F-splat +T-compiler +A-name-mangling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants