Fix lld detection if stage0 rustc built with custom libdir#72692
Fix lld detection if stage0 rustc built with custom libdir#72692o01eg wants to merge 1 commit intorust-lang:masterfrom o01eg:fix-rust-lld-detection
Conversation
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
Can we use some real targets instead of A, B, C? Maybe get list of targets with rust/src/bootstrap/builder/tests.rs Line 20 in 4512721 |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
I've reordered test data to pass CI but I wonder if it correct. |
|
Reordering test data should be fine, it's testing that we are running those steps -- order doesn't really matter. However, I'm not too happy about using the first target in the |
|
@Mark-Simulacrum Thanks, I'll try to see if it helps. |
|
This looks good to me so once CI passes I'll approve, thanks! |
|
@Mark-Simulacrum Yes, CI passed. |
|
@bors r+ |
|
📌 Commit ce72299 has been approved by |
|
I'll close it in favor accepted #72728 |
Make bootstrap aware of relative libdir in stage0 compiler Follows up rust-lang#72692 Fixes rust-lang#72661
Make bootstrap aware of relative libdir in stage0 compiler Follows up rust-lang#72692 Fixes rust-lang#72661
While working on issues with stage0 built with custom libdir #72661 I found out
rust-lldcann't be correctly detected in this case.This PR uses recently introduced #69608 to get correct place of
rust-lldso stage0 should support it.