Page MenuHomeFreeBSD
Differential D34803

Remove checks for __CC_SUPPORTS_WARNING assuming it is always true.
ClosedPublic
Actions

Authored by jhb on Apr 6 2022, 2:26 PM.
Tags
None
Referenced Files
F142911422: D34803.diff
Sat, Jan 24, 4:21 PM
Unknown Object (File)
Mon, Jan 12, 10:16 PM
Unknown Object (File)
Fri, Jan 9, 10:28 PM
Unknown Object (File)
Dec 14 2025, 4:57 PM
Unknown Object (File)
Dec 14 2025, 2:58 AM
Unknown Object (File)
Dec 3 2025, 9:06 AM
Unknown Object (File)
Dec 2 2025, 10:22 AM
Unknown Object (File)
Nov 29 2025, 12:18 AM
View All Files
Subscribers
andrew
imp
jhibbits

Details

Reviewers
brooks
emaste
imp
Commits
rG5f9c9ae2f22b: Remove checks for __CC_SUPPORTS_WARNING assuming it is always true.
Summary

All supported compilers (modern versions of GCC and clang) support
this.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45028
Build 41916: arc lint + arc unit

Event Timeline

jhb created this revision.Apr 6 2022, 2:26 PM
Herald added subscribers: jhibbits, andrew, imp. · View Herald TranscriptApr 6 2022, 2:26 PM
jhb requested review of this revision.Apr 6 2022, 2:26 PM
Harbormaster completed remote builds in B45028: Diff 104669.Apr 6 2022, 2:26 PM
jhb added a parent revision: D34802: Remove checks for __CC_SUPPORTS__INLINE assuming it is always true..Apr 6 2022, 2:26 PM
jhb added a child revision: D34804: Remove checks for <sys/cdefs.h> being included..
imp accepted this revision.Apr 6 2022, 2:38 PM
This revision is now accepted and ready to land.Apr 6 2022, 2:38 PM
brooks accepted this revision.Apr 8 2022, 9:25 PM

This one is perhaps the most debatable one since #warning isn't standard, but we've got plenty of unguarded uses.

It's probably time to finish off machine/speaker.h and machine/limits.h (or at least do an exp-run to see how bad it is)

Closed by commit rG5f9c9ae2f22b: Remove checks for __CC_SUPPORTS_WARNING assuming it is always true. (authored by jhb). · Explain WhyApr 12 2022, 5:13 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rG5f9c9ae2f22b: Remove checks for __CC_SUPPORTS_WARNING assuming it is always true..

Revision Contents
Changeset List

PathSize
sys/
amd64/
include/
4 lines
arm/
include/
4 lines
i386/
include/
4 lines
4 lines
powerpc/
include/
4 lines
sys/
5 lines

Diff 104669

View Options

sys/amd64/include/limits.h

Loading...
View Options

sys/arm/include/limits.h

Loading...
View Options

sys/i386/include/limits.h

Loading...
View Options

sys/i386/include/speaker.h

Loading...
View Options

sys/powerpc/include/limits.h

Loading...
View Options

sys/sys/syslimits.h

Loading...