[PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

[Vijay Kumar Banerjee]

On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill <joel at rtems.org> wrote:
>
> Vijay.. go ahead and push this if you don't mind.
>
Sure, I will push this.

> I was just wondering if we could make a blanket statement that all motorola_powerpc variants need this flag.

I do not have a test case other than the EPICS error. At least 2 BSPs
(2100, 2700/2307) in motorola_powerpc need this, and except mcp750 all
other BSPs in motorola_powerpc seem to have it in the spec. It is
likely that all motorola_powerpc variants need this.

>
> On Wed, Aug 9, 2023 at 12:01 PM Vijay Kumar Banerjee <vijay at rtems.org> wrote:
>>
>> On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill <joel at rtems.org> wrote:
>> >
>> > Reading the EPICS discussion, I wonder if this should be added for all motorola_powerpc BSP variants.
>> >
>> MVME2700 and the qemu variants already add it separately
>> https://git.rtems.org/rtems/tree/spec/build/bsps/powerpc/motorola_powerpc/abi.yml#n24
>>
>> > Gedare/Chris/anyone with another board in the family? What do you think?
>>
>> 2700 works for me, and it already has the flag added.
>>
>> >
>> > --joel
>> >
>> > On Wed, Aug 9, 2023 at 11:14 AM Uchenna Ezeobi <uchenna.ezeobi3 at gmail.com> wrote:
>> >>
>> >> On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill <joel at rtems.org> wrote:
>> >> >
>> >> > I'm ok with this if there is a known case of an alignment exception that causes this.
>> >> >
>> >>  We were getting the error from the
>> >> https://github.com/epics-base/epics-base/issues/29 while trying to
>> >> build EPICS on RTEMS-MVME2100, doing this fixed the error.
>> >>
>> >> > We didn't want to force it into every PowerPC BSP.
>> >> >
>> >> > Do you have a case that failed and was resolved by adding this?
>> >> >
>> >> > On Wed, Aug 9, 2023 at 7:36 AM Uchenna Ezeobi <uchenna.ezeobi3 at gmail.com> wrote:
>> >> >>
>> >> >> Update #3767
>> >> >> ---
>> >> >>  spec/build/bsps/powerpc/motorola_powerpc/abi.yml | 4 ++++
>> >> >>  1 file changed, 4 insertions(+)
>> >> >>
>> >> >> diff --git a/spec/build/bsps/powerpc/motorola_powerpc/abi.yml b/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
>> >> >> index 6734e9babf..2438c30f1d 100644
>> >> >> --- a/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
>> >> >> +++ b/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
>> >> >> @@ -17,6 +17,10 @@ default:
>> >> >>    - -mcpu=powerpc
>> >> >>    - -mmultiple
>> >> >>    - -mstrict-align
>> >> >> +- enabled-by: [powerpc/mvme2100]
>> >> >> +  value:
>> >> >> +  - -mcpu=603e
>> >> >> +  - -mstrict-align
>> >> >>  - enabled-by: [powerpc/mvme2307, powerpc/mvme2700]
>> >> >>    value:
>> >> >>    - -mcpu=604
>> >> >> --
>> >> >> 2.39.3
>> >> >>
>> >> >> _______________________________________________
>> >> >> devel mailing list
>> >> >> devel at rtems.org
>> >> >> http://lists.rtems.org/mailman/listinfo/devel
>> >
>> > _______________________________________________
>> > devel mailing list
>> > devel at rtems.org
>> > http://lists.rtems.org/mailman/listinfo/devel