M-Labs/nac3
M-Labs
/
nac3
8
0
Fork 10
Code Issues 99 Pull Requests 21 Releases Wiki Activity

meta: Use mold as linker #451

Open
derppening wants to merge 1 commits from misc/mold into master
pull from: misc/mold
merge into: M-Labs:master
Conversation 2 Commits 1 Files Changed 2 +12 -5
derppening commented 2024-07-08 12:39:41 +08:00
Collaborator
Copy Link

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
sb10q commented 2024-07-09 04:49:35 +08:00
Owner
Copy Link

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...
derppening commented 2024-07-09 14:00:49 +08:00
Author
Collaborator
Copy Link

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.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b misc/mold master
git pull origin misc/mold

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff misc/mold
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
sb10q
Labels
Clear labels   
future-extension

   
high-priority

   
low-priority

   
UROP0622
No Label
future-extension
high-priority
low-priority
UROP0622
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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.
Delete Branch "misc/mold"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?