Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

rustc: remove ty_unboxed_vec. #13344

Merged
merged 1 commit into from
Apr 6, 2014
Merged

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Apr 5, 2014

Removes the special ty_unboxed_vec type from the type system.
It was previously used only during translating ~[T]/~str allocation and drop glue.

@alexcrichton
Copy link
Member

Travis looks like it thinks that there's some llvm codegen bugs (or possibly somehow related).

Could you also add description/rationale for what this change is for? I personally don't know what ty_unboxed_vec is, and it'd be nice to have more than a few words in the commit message.

bors added a commit that referenced this pull request Apr 6, 2014
Removes the special `ty_unboxed_vec` type from the type system.
It was previously used only during translating `~[T]`/`~str` allocation and drop glue.
@bors bors closed this Apr 6, 2014
@bors bors merged commit 2d22243 into rust-lang:master Apr 6, 2014
@eddyb eddyb deleted the kill-unboxed-vec branch April 6, 2014 14:11
matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this pull request Oct 11, 2022
…, r=Veykril

fix: use `BoundVar`s from current generic scope

Fixup for rust-lang#13335, addresses rust-lang/rust-analyzer#13339 (comment)

Before the change in generic parameter order, `BoundVar`s for trait reference didn't change whether you are in an impl's scope or in an associated item's scope. Now that item's generic params come before its parent's, we need to shift their indices when we are in an associated item's scope.
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 24, 2024
Expand missing_transmute_annotations docs

Fixes rust-lang#13339

changelog: [`missing_transmute_annotations `]: Expand docs, raison d'être
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.

4 participants