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

regression: linking with cc failed #130581

Closed
BoxyUwU opened this issue Sep 19, 2024 · 8 comments
Closed

regression: linking with cc failed #130581

BoxyUwU opened this issue Sep 19, 2024 · 8 comments
Labels
A-linkage Area: linking into static, shared libraries and binaries regression-from-stable-to-stable Performance or correctness regression from one stable version to another. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Milestone

Comments

@BoxyUwU BoxyUwU added I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 19, 2024
@rustbot rustbot added I-prioritize Issue: Indicates that prioritization has been requested for this issue. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Sep 19, 2024
@BoxyUwU BoxyUwU removed the I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ label Sep 19, 2024
@ChrisDenton
Copy link
Member

cc @NobodyXu in case you have any ideas

@saethlin
Copy link
Member

saethlin commented Sep 19, 2024

Something very fishy is going on here. libwebp-sys is small, well-maintained and has quite a few downloads, so I'm looking into that.

cargo +beta build --tests fails as crater indicates. I've tried every nightly in 2024-08-XX and 2024-09-XX and all of them can build the crate's tests.

The cc version doesn't seem to matter, I've tried versions back to 1.0.50 which is years old.

@saethlin
Copy link
Member

I also cannot reproduce these linker errors using a toolchain that I've built from the beta branch.

@saethlin saethlin added the A-linkage Area: linking into static, shared libraries and binaries label Sep 19, 2024
@saethlin
Copy link
Member

Ah! My confusion is because nightly now uses lld by default, which is a good little linker and takes care of you even if you give it a confusing argument order.

@saethlin saethlin removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Sep 19, 2024
@saethlin
Copy link
Member

Bisects to #128400

A crater run was done in that PR but the results were ignored because of the same confusion I ran into above #128400 (comment)

@apiraino apiraino removed the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Oct 7, 2024
@apiraino
Copy link
Contributor

apiraino commented Oct 7, 2024

As discussed on Zulip is there any actionable here, or can this issue be closed? #128400 was noted in the release notes (unsure if this is enough tho)

@lqd
Copy link
Member

lqd commented Oct 7, 2024

cc @petrochenkov

@Mark-Simulacrum Mark-Simulacrum added this to the 1.82.0 milestone Oct 11, 2024
@Mark-Simulacrum Mark-Simulacrum added regression-from-stable-to-stable Performance or correctness regression from one stable version to another. and removed regression-from-stable-to-beta Performance or correctness regression from stable to beta. labels Nov 22, 2024
@rustbot rustbot added the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Nov 22, 2024
@apiraino apiraino removed the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Nov 25, 2024
@apiraino
Copy link
Contributor

Closing since #128400 will be mentioned in the release notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linkage Area: linking into static, shared libraries and binaries regression-from-stable-to-stable Performance or correctness regression from one stable version to another. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

7 participants