Page MenuHomeFreeBSD
Differential D53821

sys/_types.h: add void * to __max_align_t
ClosedPublic
Actions

Authored by brooks on Nov 19 2025, 11:18 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 21, 9:46 AM
Unknown Object (File)
Sat, Jan 10, 3:22 PM
Unknown Object (File)
Sat, Jan 10, 5:27 AM
Unknown Object (File)
Dec 20 2025, 3:19 AM
Unknown Object (File)
Dec 12 2025, 4:59 PM
Unknown Object (File)
Dec 3 2025, 2:36 AM
Unknown Object (File)
Dec 1 2025, 11:38 AM
Unknown Object (File)
Nov 30 2025, 2:35 AM
View All 21 Files
Subscribers
None

Details

Reviewers
imp
kib
Group Reviewers
cheri
Commits
rG528e70a12ca6: sys/_types.h: add void * to __max_align_t
Summary

Add a pointer member to __max_align_t as pointers may have different
alignment requirements than long long or long double.

Effort: CHERI upstreaming
Sponsored by: Innovate UK

Diff Detail

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

Event Timeline

brooks requested review of this revision.Nov 19 2025, 11:18 AM
brooks created this revision.
brooks added a parent revision: D53820: sys/_types.h: define fallback __(u)intcap_t.
Harbormaster completed remote builds in B68729: Diff 166754.Nov 19 2025, 11:18 AM
brooks added a child revision: D53822: sys/_types.h: add __intptr_t to __mbstate_t.Nov 19 2025, 11:18 AM
kib accepted this revision.Nov 19 2025, 1:51 PM
kib added inline comments.
sys/sys/_types.h
207

Should this be long double?

This revision is now accepted and ready to land.Nov 19 2025, 1:51 PM
brooks added inline comments.Nov 19 2025, 2:09 PM
sys/sys/_types.h
207

Indeed, that's quite odd. Clang's version uses long double. I'm also not sure why the alignment attributes are used at all since structures members are going to be naturally aligned regardless.

kib added inline comments.Nov 19 2025, 2:50 PM
sys/sys/_types.h
207

Might be it is a counter-measure against #pragma packed() around it.

I will post a review to change to long double.

Closed by commit rG528e70a12ca6: sys/_types.h: add void * to __max_align_t (authored by brooks). · Explain WhyNov 27 2025, 2:57 PM
This revision was automatically updated to reflect the committed changes.
brooks added a commit: rG528e70a12ca6: sys/_types.h: add void * to __max_align_t.

Revision Contents
Changeset List

PathSize
sys/
sys/
1 line

Diff 167178

View Options

sys/sys/_types.h

Loading...