<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 21, 2021, 4:31 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This seems alright to me. At least, it should get some complaints<br>
quickly if it doesn't work :)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The old version should have gotten complaints but didn't. Probably indicates people are not being careful about specifying the language version they want to use and just taking the GCC default which periodically changes. </div><div dir="auto"><br></div><div dir="auto">--joel</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Tue, Jul 20, 2021 at 3:04 PM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> wrote:<br>
><br>
> The default implementation was completely broken. Use the GCC specific<br>
> __alignof__ if compiling for C99 or C++03. If not C++11, C11, or<br>
> GCC, then it is an error.<br>
> ---<br>
>  freebsd/sys/sys/linker_set.h | 4 +++-<br>
>  1 file changed, 3 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/freebsd/sys/sys/linker_set.h b/freebsd/sys/sys/linker_set.h<br>
> index baa5ae4..9af5307 100755<br>
> --- a/freebsd/sys/sys/linker_set.h<br>
> +++ b/freebsd/sys/sys/linker_set.h<br>
> @@ -73,8 +73,10 @@<br>
>    #define RTEMS_BSD_ALIGNOF( _type_name ) alignof( _type_name )<br>
>  #elif __STDC_VERSION__ >= 201112L<br>
>    #define RTEMS_BSD_ALIGNOF( _type_name ) _Alignof( _type_name )<br>
> +#elif defined(__GNUC__)<br>
> +  #define RTEMS_BSD_ALIGNOF( _type_name ) __alignof__( _type_name )<br>
>  #else<br>
> -  #define RTEMS_BSD_ALIGNOF( _type_name ) sizeof( _type_name )<br>
> +  #error "FIX ME! Implement RTEMS_BSD_ALIGNOF() for this environment"<br>
>  #endif<br>
><br>
>  #define RTEMS_BSD_SET_ALIGN( type )    \<br>
> --<br>
> 1.8.3.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>