[vlc-devel] [vlc-commits] mmxext: use standard syntax for mmx_t alignment

Rémi Denis-Courmont remi at remlab.net
Sun Feb 26 21:09:36 CET 2017


Le sunnuntaina 26. helmikuuta 2017, 21.00.25 EET Rémi Denis-Courmont a écrit :
> vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sun Feb 26
> 21:56:06 2017 +0200| [5f47b75d32ac803c5cbeb919e3d3b97772ab2cb0] |
> committer: Rémi Denis-Courmont
> 
> mmxext: use standard syntax for mmx_t alignment
> 

> diff --git a/modules/video_filter/deinterlace/mmx.h
> b/modules/video_filter/deinterlace/mmx.h index 6f64a75..072efe5 100644
> --- a/modules/video_filter/deinterlace/mmx.h
> +++ b/modules/video_filter/deinterlace/mmx.h
> @@ -37,7 +37,7 @@ typedef    union {
>      int8_t           b[8]; /* 8 Byte (8-bit) values */
>      uint8_t         ub[8]; /* 8 Unsigned Byte */
>      float            s[2]; /* Single-precision (32-bit) value */
> -} ATTR_ALIGN(8) mmx_t;     /* On an 8-byte (64-bit) boundary */
> +} mmx_t; /* NOTE: must be on an 8-byte (64-bit) boundary */

In hindsight, that might not be the most straightforward way to achieve the 
intended result.

If somebody prefers to factor the alignment directive to the first union 
member, s/he is welcome to help his-/herself to it.

-- 
雷米‧德尼-库尔蒙
https://www.remlab.net/



More information about the vlc-devel mailing list