[Bug 1560] sparc64 [1/5]: _Objects_Extend_information improper alignment for 64-bit targets

bugzilla-daemon at rtems.org bugzilla-daemon at rtems.org
Tue Feb 12 14:00:41 UTC 2013


https://www.rtems.org/bugzilla/show_bug.cgi?id=1560

--- Comment #21 from Gedare <gedare at rtems.org> 2013-02-12 08:00:41 CST ---
(In reply to comment #20)
> I would prefer to add and use a
> 
> RTEMS_INLINE_ROUTINE const void *_Addresses_Align_up_immutable(
>   const void *address
> );
> 
> RTEMS_INLINE_ROUTINE void *_Addresses_Align_up(
>   void *address
> );
> 
> RTEMS_INLINE_ROUTINE const void *_Addresses_Align_down_immutable(
>   const void *address
> );
> 
> RTEMS_INLINE_ROUTINE void *_Addresses_Align_down(
>   void *address
> );
> 
> We must have the invariants:
> 
> _Addresses_Align_up(x) == _Addresses_Is_aligned(x) ? x : _Addresses_Align_up(x)
> 
> _Addresses_Align_down(x) == _Addresses_Is_aligned(x) ? x :
> _Addresses_Align_down(x)

I'm fine with these changes. There are other places in RTEMS with alignment
macros and hard-coded alignment that would benefit. The issue is separate from
this particular bug, but I'll work on a patch to add these routines.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the bugs mailing list