Page MenuHomeFreeBSD
Differential D49206

amd64 csu: microoptimize startup
ClosedPublic
Actions

Authored by kib on Mar 3 2025, 12:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 2, 8:37 AM
Unknown Object (File)
Tue, Nov 25, 6:38 PM
Unknown Object (File)
Tue, Nov 25, 1:21 PM
Unknown Object (File)
Wed, Nov 19, 11:35 PM
Unknown Object (File)
Mon, Nov 10, 9:43 PM
Unknown Object (File)
Oct 25 2025, 3:14 PM
Unknown Object (File)
Oct 23 2025, 6:49 PM
Unknown Object (File)
Oct 19 2025, 3:15 AM
View All 77 Files
Subscribers
imp

Details

Reviewers
jhb
markj
emaste
Commits
rGcb991a228179: amd64 csu: microoptimize startup
Summary

Reorder operations to reduce moving the values among registers.
Use the LEA instruction to combine arithmetic with the moves.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Mar 3 2025, 12:02 AM
Herald added a subscriber: imp. · View Herald TranscriptMar 3 2025, 12:02 AM
kib requested review of this revision.Mar 3 2025, 12:02 AM
kib updated this revision to Diff 151763.Mar 3 2025, 4:38 AM

Correct the offset of env.

markj added inline comments.Mar 3 2025, 8:01 AM
lib/csu/amd64/crt1_s.S
53

What is *apl?

55

Shouldn't the comment say + 2?

kib updated this revision to Diff 151771.Mar 3 2025, 8:26 AM
kib marked 2 inline comments as done.

Fix typos in comments.

markj accepted this revision.Mar 3 2025, 11:22 AM
This revision is now accepted and ready to land.Mar 3 2025, 11:22 AM
Closed by commit rGcb991a228179: amd64 csu: microoptimize startup (authored by kib). · Explain WhyMar 4 2025, 4:25 AM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rGcb991a228179: amd64 csu: microoptimize startup.

Revision Contents
Changeset List

PathSize
lib/
csu/
amd64/
15 lines

Diff 151829

View Options

lib/csu/amd64/crt1_s.S

Loading...