Page MenuHomeFreeBSD
Differential D8194

Define max_align_t for C11.
ClosedPublic
Actions

Authored by jhb on Oct 8 2016, 6:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 23, 1:05 AM
Unknown Object (File)
Wed, Oct 22, 5:27 PM
Unknown Object (File)
Wed, Oct 22, 5:20 AM
Unknown Object (File)
Wed, Oct 22, 5:20 AM
Unknown Object (File)
Wed, Oct 22, 5:20 AM
Unknown Object (File)
Tue, Oct 21, 5:20 PM
Unknown Object (File)
Tue, Oct 21, 5:50 AM
Unknown Object (File)
Sat, Oct 18, 9:13 PM
View All Files
Subscribers
emaste

Details

Reviewers
dim
Commits
rS307756: Define max_align_t for C11.
Summary

Define max_align_t for C11.

libc++'s stddef.h includes an existing definition of max_align_t for
C++11, but it is only defined for C++, not for C. In addition, GCC and
clang both define an alternate version of max_align_t that uses a
union of multiple types rather than a plain long double as in libc++.
This adds a __max_align_t to <sys/_types.h> that matches the GCC and
clang definition that is mapped to max_align_t in <stddef.h>.

PR: 210890

Test Plan
  • universe and an exp-run. One port failed during the exp-run which I still haven't fixed.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb updated this revision to Diff 21180.Oct 8 2016, 6:26 AM
jhb retitled this revision from to Define max_align_t for C11..
jhb updated this object.
jhb edited the test plan for this revision. (Show Details)
jhb added a reviewer: dim.
jhb updated this revision to Diff 21556.Oct 20 2016, 6:30 PM
  • Fix name of GCC macro used to note that max_align_t is defined.
dim accepted this revision.Oct 20 2016, 6:50 PM
dim edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Oct 20 2016, 6:50 PM
emaste added a subscriber: emaste.Oct 20 2016, 7:11 PM
Closed by commit rS307756: Define max_align_t for C11. (authored by jhb). · Explain WhyOct 21 2016, 11:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
include/
8 lines
sys/
sys/
5 lines

Diff 21607

View Options

head/include/stddef.h

Loading...
View Options

head/sys/sys/_types.h

Loading...