Uh oh!
There was an error while loading. Please reload this page.
[experiment] Support linking from a .rlink file - #68487
Conversation
rust-highfive
commented
Jan 23, 2020
r? @eddyb (rust_highfive has picked a reviewer for you, use r? to override) |
0dvictor
commented
Jan 23, 2020
r? @tmandry |
rust-highfive
commented
Jan 23, 2020
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 |
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
This is already performed when creating the .rlink file. I think it can be moved before linking for join_codegen_and_link too.
There was a problem hiding this comment.
This is technically a breaking change, as a source file may have .rlink as extension. Checking if the file starts with { or magic bytes (eg b"rlink", should be prepended before writing it) will fix this.
There was a problem hiding this comment.
I'm not sure if this is still necessary as the feature is now gated by -Z link-only.
There was a problem hiding this comment.
I think automatically detecting the .rlink file will be desired when we stabilize this feature; however, we will likely change the .rlink file to a binary at that time. So I prefer not to include automatic file format detection at this stage.
There was a problem hiding this comment.
This isn't feature gated. With a hand made json file it is possible to use this.
Uh oh!
There was an error while loading. Please reload this page.
bors
commented
Feb 4, 2020
☔ The latest upstream changes (presumably #68601) made this pull request unmergeable. Please resolve the merge conflicts. |
0dvictor
commented
Feb 4, 2020
Rebased so that #68601 is included. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
tmandry
commented
Feb 10, 2020
@bors delegate=0dvictor |
tmandry
commented
Feb 10, 2020
Flag `-Z no-link` was previously introduced, which allows creating an `.rlink` file to perform compilation without linking. This change enables linking from an `.rlink` file.
0dvictor
commented
Feb 11, 2020
I've squashed git history and all four checks passed. This PR is ready to go. @bors r+ |
bors
commented
Feb 11, 2020
@0dvictor: 🔑 Insufficient privileges: Not in reviewers |
bjorn3
commented
Feb 11, 2020
@tmandry I think the correct syntax is |
mati865
commented
Feb 11, 2020
@bjorn3 both syntax are correct but bors had downtime yesterday. |
tmandry
commented
Feb 11, 2020
@bors r+ |
bors
commented
Feb 11, 2020
📌 Commit a47fdb9 has been approved by |
[experiment] Support linking from a .rlink file Flag `-Z no-link` was previously introduced, which allows creating an `.rlink` file to perform compilation without linking. This change enables linking from an `.rlink` file. Part of Issue rust-lang#64191
Rollup of 11 pull requests Successful merges: - #67695 (Added dyn and true keyword docs) - #68487 ([experiment] Support linking from a .rlink file) - #68554 (Split lang_items to crates `rustc_hir` and `rustc_passes`.) - #68937 (Test failure of unchecked arithmetic intrinsics in const eval) - #68947 (Python script PEP8 style guide space formatting and minor Python source cleanup) - #68999 (remove dependency on itertools) - #69026 (Remove common usage pattern from `AllocRef`) - #69027 (Add missing `_zeroed` varants to `AllocRef`) - #69058 (Preparation for allocator aware `Box`) - #69070 (Add self to .mailmap) - #69077 (Fix outdated doc comment.) Failed merges: r? @ghost
bjorn3
commented
Feb 12, 2020
Hooray! Next step: cargo support. |
Flag
-Z no-linkwas previously introduced, which allows creating an.rlinkfile to perform compilation without linking. This change enables linking from an.rlinkfile.Part of Issue #64191