meta: Use mold as linker #451

Open
derppening wants to merge 1 commits from misc/mold into master
Collaborator

Tested by cargo build --all --tests on a clean directory.

Ryzen 7 6850U:

ld: 349.96user 31.11system 0:52.96elapsed 719%CPU (0avgtext+0avgdata 983032maxresident)k
mold: 339.10user 26.82system 0:49.64elapsed 737%CPU (0avgtext+0avgdata 1002016maxresident)k

Ryzen 9 5900X:

ld: 186.05user 17.59system 0:32.98elapsed 617%CPU (0avgtext+0avgdata 1021228maxresident)k
mold: 183.31user 14.95system 0:31.04elapsed 638%CPU (0avgtext+0avgdata 1044292maxresident)k

Sidenote: I am not sure whether I am putting the packages/stdenv overrides at the right place in flake.nix. Would appreciate if someone can review it.

Tested by `cargo build --all --tests` on a clean directory. Ryzen 7 6850U: ``` ld: 349.96user 31.11system 0:52.96elapsed 719%CPU (0avgtext+0avgdata 983032maxresident)k mold: 339.10user 26.82system 0:49.64elapsed 737%CPU (0avgtext+0avgdata 1002016maxresident)k ``` Ryzen 9 5900X: ``` ld: 186.05user 17.59system 0:32.98elapsed 617%CPU (0avgtext+0avgdata 1021228maxresident)k mold: 183.31user 14.95system 0:31.04elapsed 638%CPU (0avgtext+0avgdata 1044292maxresident)k ``` Sidenote: I am not sure whether I am putting the packages/stdenv overrides at the right place in `flake.nix`. Would appreciate if someone can review it.
derppening requested review from sb10q 2024-07-08 12:40:49 +08:00
Owner

Not sure if this is worth the extra code, complexity, and potential bugs...

Not sure if this is worth the extra code, complexity, and potential bugs...
Author
Collaborator

Updated stats using cargo build --all-targets on a clean directory.

Ryzen 7 6850U:

ld: 388.20user 33.36system 0:54.27elapsed 776%CPU (0avgtext+0avgdata 958444maxresident)k
mold: 365.84user 26.51system 0:50.18elapsed 781%CPU (0avgtext+0avgdata 972560maxresident)k

Ryzen 9 5900X:

ld: 202.71user 20.43system 0:33.52elapsed 665%CPU (0avgtext+0avgdata 1014964maxresident)k
mold: 195.02user 16.24system 0:31.20elapsed 677%CPU (0avgtext+0avgdata 1045960maxresident)k

Do you think it might be better if we only merge the changes to flake.nix so that people can opt into mold rather than enforcing it for everyone?

Updated stats using `cargo build --all-targets` on a clean directory. Ryzen 7 6850U: ``` ld: 388.20user 33.36system 0:54.27elapsed 776%CPU (0avgtext+0avgdata 958444maxresident)k mold: 365.84user 26.51system 0:50.18elapsed 781%CPU (0avgtext+0avgdata 972560maxresident)k ``` Ryzen 9 5900X: ``` ld: 202.71user 20.43system 0:33.52elapsed 665%CPU (0avgtext+0avgdata 1014964maxresident)k mold: 195.02user 16.24system 0:31.20elapsed 677%CPU (0avgtext+0avgdata 1045960maxresident)k ``` Do you think it might be better if we only merge the changes to `flake.nix` so that people can opt into mold rather than enforcing it for everyone?
derppening force-pushed misc/mold from ffe42a67ea to c03743e9b0 2024-07-09 14:06:28 +08:00 Compare
derppening force-pushed misc/mold from c03743e9b0 to 7d40808c12 2024-08-28 16:34:44 +08:00 Compare
derppening force-pushed misc/mold from 7d40808c12 to 8f21c33f5b 2024-09-11 16:39:20 +08:00 Compare
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin misc/mold:misc/mold
git checkout misc/mold
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/nac3#451
No description provided.