mbox series

[REPOST,0/3] Add support for -mcpu=power11

Message ID Zo7FkkNpgCSpuiZ6@cowardly-lion.the-meissners.org
Headers show
Series Add support for -mcpu=power11 | expand

Message

Michael Meissner July 10, 2024, 5:32 p.m. UTC 
Note, this is a repost of the 3 patches I posted on June 4th.  The first two
patches are the same.  The third patch modifies the power11 tests to do a
compile instead of assemble, and I removed the power11 specific target support
that was posted as suggested by Kewen.Lin.

The following 3 patches add support for -mcpu=power11 to GCC 15.  Assuming
these patches are approved and go into GCC 15, I will need to back port them to
GCC 14.

The first patch adds the basic support for -mcpu=power11, except for the
scheduling infomration.

The second patch goes through power10.md and adds scheduling support for
power11, treating -mtune=power11 to be the same as -mtune=power10 at the
current time.

The third patch adds some new tests for -mcpu=power11 support.

In order to use -mcpu=power11, you will need to use a new enough binutils that
supports the .machine power11 option.

I have bootstrapped the compiler on both little endian and big endian systems.
There were no regressions in either case.  Can I check these patches into the
GCC trunk?  After a waiting period assuming there are no issues, can I check
these patches into the GCC 14 branch?

Comments

Kewen.Lin July 12, 2024, 7:58 a.m. UTC | #1 
Hi Mike,

on 2024/7/11 01:32, Michael Meissner wrote:
> Note, this is a repost of the 3 patches I posted on June 4th.  The first two
> patches are the same.  The third patch modifies the power11 tests to do a
> compile instead of assemble, and I removed the power11 specific target support
> that was posted as suggested by Kewen.Lin.
> 
> The following 3 patches add support for -mcpu=power11 to GCC 15.  Assuming
> these patches are approved and go into GCC 15, I will need to back port them to
> GCC 14.
> 
> The first patch adds the basic support for -mcpu=power11, except for the
> scheduling infomration.
> 
> The second patch goes through power10.md and adds scheduling support for
> power11, treating -mtune=power11 to be the same as -mtune=power10 at the
> current time.
> 
> The third patch adds some new tests for -mcpu=power11 support.
> 
> In order to use -mcpu=power11, you will need to use a new enough binutils that
> supports the .machine power11 option.
> 
> I have bootstrapped the compiler on both little endian and big endian systems.
> There were no regressions in either case.  Can I check these patches into the
> GCC trunk?  After a waiting period assuming there are no issues, can I check
> these patches into the GCC 14 branch?
> 

Thanks for doing this!  This patch series looks good to me, as it's still on
Segher's review list, I'd leave this to him for final approval, thanks!

BR,
Kewen
Segher Boessenkool July 18, 2024, 12:08 p.m. UTC | #2 
Hi!

On Wed, Jul 10, 2024 at 01:32:02PM -0400, Michael Meissner wrote:
> Note, this is a repost of the 3 patches I posted on June 4th.  The first two
> patches are the same.  The third patch modifies the power11 tests to do a
> compile instead of assemble, and I removed the power11 specific target support
> that was posted as suggested by Kewen.Lin.
> 
> The following 3 patches add support for -mcpu=power11 to GCC 15.  Assuming
> these patches are approved and go into GCC 15, I will need to back port them to
> GCC 14.
> 
> The first patch adds the basic support for -mcpu=power11, except for the
> scheduling infomration.
> 
> The second patch goes through power10.md and adds scheduling support for
> power11, treating -mtune=power11 to be the same as -mtune=power10 at the
> current time.
> 
> The third patch adds some new tests for -mcpu=power11 support.
> 
> In order to use -mcpu=power11, you will need to use a new enough binutils that
> supports the .machine power11 option.

This is a general thing: we always assume you have a binutils at least
as new as your GCC.


Segher