Page MenuHomeFreeBSD

__builtin_align_down: Cast value to __uintptr_t in the fallback
ClosedPublic

Authored by jhb on Feb 7 2026, 3:46 PM.
Tags
None
Referenced Files
F163497121: D55161.id.diff
Thu, Jul 23, 7:56 PM
Unknown Object (File)
Thu, Jul 23, 2:41 AM
Unknown Object (File)
Wed, Jul 22, 3:50 AM
Unknown Object (File)
Tue, Jul 21, 6:10 AM
Unknown Object (File)
Mon, Jul 20, 12:35 PM
Unknown Object (File)
Wed, Jul 15, 5:45 AM
Unknown Object (File)
Wed, Jul 15, 5:45 AM
Unknown Object (File)
Sun, Jul 12, 11:56 AM
Subscribers

Diff Detail

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

Event Timeline

I worry that this was intentionally left out to avoid breaking there use of rounddown() on integer types larger than uintptr_t? (E.g. vm_paddr_t on i386)

If that is really a worry, maybe we use _Generic (including the fallback for pre-C11) that maps pointer types to versions that use uintptr_t and otherwise doesn't use the cast?

This revision is now accepted and ready to land.Feb 17 2026, 9:10 PM

Almost certainly an accidental omission