Reviewed-by: Balasubramanian Manoharan <[email protected]>

On 10 May 2016 at 07:25, Bill Fischofer <[email protected]> wrote:

> The tick_buf_t struct may be larger than 16 bytes when a lock char is
> needed so correct the ODP_STATIC_ASSERT to reflect this. This addresses
> bug https://bugs.linaro.org/show_bug.cgi?id=2211 when compiling with
> clang.
>
> Signed-off-by: Bill Fischofer <[email protected]>
> ---
>  platform/linux-generic/odp_timer.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/platform/linux-generic/odp_timer.c
> b/platform/linux-generic/odp_timer.c
> index f4fb1f6..89ec5f5 100644
> --- a/platform/linux-generic/odp_timer.c
> +++ b/platform/linux-generic/odp_timer.c
> @@ -107,7 +107,7 @@ ODP_ALIGNED(16) /* 16-byte atomic operations need
> properly aligned addresses */
>  #endif
>  ;
>
> -ODP_STATIC_ASSERT(sizeof(tick_buf_t) == 16, "sizeof(tick_buf_t) == 16");
> +ODP_STATIC_ASSERT(sizeof(tick_buf_t) >= 16, "sizeof(tick_buf_t) >= 16");
>
>  typedef struct odp_timer_s {
>         void *user_ptr;
> --
> 2.7.4
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to