Re: [coreboot] [Coreboot] Misalignment with GCC 4.9.1

[Peter Stuge]

Lin, Ryan wrote:
> If we add aligned(4) to the define as the following :
>
> #define BOOT_STATE_INIT_ATTR  __attribute__ ((used, aligned(4), section (".bs_init)))
>
> We can get 4-byte alignment for .bs_init.
>
> What is your thoughts on this?

Is that a GCC extension?

The coreboot codebase also supports other compilers.

//Peter

[Coreboot] Misalignment with GCC 4.9.1

[ron minnich]

Ah, peter, thanks for asking this question. Yes, it's very good to
avoid gcc-isms. We have had many, and probably have some left, but
it's nice to avoid them.

ron

On Mon, Oct 13, 2014 at 6:18 PM, Peter Stuge  wrote:
> Lin, Ryan wrote:
> > If we add aligned(4) to the define as the following :
> >
> > #define BOOT_STATE_INIT_ATTR  __attribute__ ((used, aligned(4), section (".bs_init)))
> >
> > We can get 4-byte alignment for .bs_init.
> >
> > What is your thoughts on this?
>
> Is that a GCC extension?
>
> The coreboot codebase also supports other compilers.
>
> //Peter