math.h: Assume gcc, gnu c extensions, and modern compiler
AbandonedPublic
Actions

Authored by imp on Mar 15 2023, 5:53 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Feb 24 2026, 9:15 AM

	Unknown Object (File)
	Feb 24 2026, 2:50 AM

	Unknown Object (File)
	Feb 18 2026, 7:44 AM

	Unknown Object (File)
	Feb 3 2026, 1:09 AM

	Unknown Object (File)
	Nov 30 2025, 9:36 PM

	Unknown Object (File)
	Nov 23 2025, 11:24 AM

	Unknown Object (File)
	Nov 23 2025, 10:55 AM

	Unknown Object (File)
	Nov 9 2025, 2:40 AM

View All 66 Files

Subscribers

None

Details

Reviewers

kib
emaste
vangyzen
jrtc27

Summary

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 50387
Build 47278: arc lint + arc unit

Event Timeline

imp requested review of this revision.Mar 15 2023, 5:53 PM

imp created this revision.

Harbormaster completed remote builds in B50387: Diff 118869.Mar 15 2023, 5:53 PM

imp added a parent revision: D39097: cdefs.h: Start to modernize.Mar 15 2023, 5:53 PM

imp mentioned this in D26819: cdefs: require gcc 4.2 or later.Mar 15 2023, 5:53 PM

imp edited the test plan for this revision. (Show Details)Mar 15 2023, 5:59 PM

imp added reviewers: kib, emaste, vangyzen, jrtc27.

exp run https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=270249

kib added inline comments.Mar 16 2023, 12:46 AM

lib/msun/src/math.h
122	For me, the !GNU block clearly shows the intent of the macros, while leaving only the built-in version requires use of gcc docs to figure out the contract. The non-GCC version should be left there, even if as a comment, IMO.

On second thought, go ahead and drop doing this for now.

Revision Contents
Changeset List

Path

Size

lib/

msun/

src/

math.h

33 lines

Diff 118869

View Options

math.h: Assume gcc, gnu c extensions, and modern compilerAbandonedPublicActions