Skip to content

Conversation

@the8472
Copy link
Member

@the8472 the8472 commented Jan 18, 2026

Redoing #130998
It was reverted in #151150 due to flakiness. I have traced this due to layout randomization perturbing the test (the failure reproduces locally with layout randomization), which is now excluded.

@rustbot
Copy link
Collaborator

rustbot commented Jan 18, 2026

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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

@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. 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. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jan 18, 2026
@the8472 the8472 changed the title optimize vec.append(slice.to_vec()), take 2 optimize vec.extend(slice.to_vec()), take 2 Jan 18, 2026
@tbu-
Copy link
Contributor

tbu- commented Jan 20, 2026

Does it make sense to add an optimization that breaks with layout randomization?

@the8472
Copy link
Member Author

the8472 commented Jan 20, 2026

Layout randomization is primarily for debugging (it pessimizes struct layouts after all), and the optimization is nice-to-have so I think it's ok.
At worst it indicates that the optimization is brittle, but even with randomization it still worked for the Vec->Vec case, just the String->String case broke for some reason, not sure why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. 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. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants