Skip to content

rustc::back::link: redo symbol mangling - #6956

Closed
Blei wants to merge 1 commit into
rust-lang:incomingfrom
Blei:fix-symbol-mangling
Closed

rustc::back::link: redo symbol mangling#6956
Blei wants to merge 1 commit into
rust-lang:incomingfrom
Blei:fix-symbol-mangling

Conversation

@Blei

@BleiBlei commented Jun 5, 2013

Copy link
Copy Markdown
Contributor

Handle more characters that appear in types, most notably <>): were
missing. Also the new scheme takes care that no two different input
strings result in the same mangled string, which was not the case before.

Fixes#6921

Handle more characters that appear in types, most notably <>): were
missing. Also the new scheme takes care that no two different input
strings result in the same mangled string, which was not the case before.
Fixes#6921
@Blei

Blei commented Jun 5, 2013

Copy link
Copy Markdown
ContributorAuthor

Hmm, that might be a good idea. Also, I'm not really sure if we handle the last case correctly (pushing all characters that are in XID Continue). Maybe this should be replaced by a warning instead.

bors added a commit that referenced this pull request Jun 5, 2013
Handle more characters that appear in types, most notably <>): were
missing. Also the new scheme takes care that no two different input
strings result in the same mangled string, which was not the case before.
Fixes#6921
@borsbors closed this Jun 5, 2013
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
7386: Handle box with allocator r=flodiebold a=tweksteen
In 1.49.0, the definition of Box was modified to support an optional
Allocator[1]. Adapt the parsing of the `box` keyword to supply the
expected number of parameters to the constructor.
[1] rust-lang@f288cd2Fixes: rust-lang#6956 Co-authored-by: Thiébaud Weksteen <tweek@google.com>
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
7374: Add step to run analysis-stats on std r=lnicola a=tweksteen
Run `analysis-stats` on `std` as part of the release workflow. Uses the `--with-deps` to also parse other crates defined in `stdlibs`. Remove the condition for exection, both analysis will be run for nightly and release builds.
Do not submit until rust-lang#6956 has been fixed.
Bug: rust-lang#6956
Co-authored-by: Thiébaud Weksteen <tweek@google.com>
Sign up for freeto 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

@Blei@bors