[WIP] Encode more info about order-dependence of linker arguments #71081
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
rust-highfive added the S-waiting-on-review 
| 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 latest upstream changes (presumably #71023) made this pull request unmergeable. Please resolve the merge conflicts. |
mati865 reviewed Apr 13, 2020
src/librustc_target/spec/mod.rs Outdated
There was a problem hiding this comment.
Suggested change
| /// These option can be overridden by options placed to the left from them on the command line. | |
| /// These options can be overridden by options placed to the left from them on the command line. |
src/librustc_target/spec/mod.rs Outdated
There was a problem hiding this comment.
Suggested change
| /// These option can be overridden by options placed to the right from them on the command line. | |
| /// These options can be overridden by options placed to the right from them on the command line. |
petrochenkov added the T-compiler 
bjorn3 reviewed Apr 15, 2020
There was a problem hiding this comment.
It doesn't om RISC-V as there is no windows abi.
There was a problem hiding this comment.
Funnily enough, Windows NT was originally developed for RISC processors.
There was a problem hiding this comment.
We don't have a RISC-V UEFI target though, this is a base for x86_64/i686.
The RISC-V target will need a separate "base" then.
petrochenkov force-pushed the linkorder3 branch from ec7e976 to 4f9aa69 Compare | 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 |
petrochenkov added S-waiting-on-author 
| @petrochenkov any updates? |
| Closing for now, there are more urgent things to do in this area. |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Based on #71030, only the last commit is new.
This is a continuation of #70868.
So, the goals are (copypaste):
-C link-argsand-Z pre-link-args.-Z pre-link-arg(s)as-C pre-link-arg(s)#70505).This PR starts with target specifications for MSVC-based targets specifically.
r? @nagisa