mbox series

[PATCHv2,0/3] ada: Add GNU/Hurd x86_64 support

Message ID 20241103163716.2228976-1-samuel.thibault@ens-lyon.org
Headers show
Series ada: Add GNU/Hurd x86_64 support | expand

Message

Samuel Thibault Nov. 3, 2024, 4:37 p.m. UTC 
I reworked the patch to factorize the bsd signal definitions.
I have split off the system definitions because the priority range of
GNU/Mach has diverged from the original BSD kernels.

Samuel Thibault (3):
  ada: Factorize bsd signal definitions
  ada: Fix GNU/Hurd priority range
  ada: Add GNU/Hurd x86_64 support

 gcc/ada/ChangeLog                             |  17 ++
 gcc/ada/Makefile.rtl                          |  48 +++++-
 ...-intnam__freebsd.ads => a-intnam__bsd.ads} |   2 +-
 gcc/ada/libgnarl/a-intnam__dragonfly.ads      | 136 ----------------
 gcc/ada/libgnat/system-gnu.ads                | 148 ++++++++++++++++++
 5 files changed, 206 insertions(+), 145 deletions(-)
 rename gcc/ada/libgnarl/{a-intnam__freebsd.ads => a-intnam__bsd.ads} (99%)
 delete mode 100644 gcc/ada/libgnarl/a-intnam__dragonfly.ads
 create mode 100644 gcc/ada/libgnat/system-gnu.ads

Comments

Marc Poulhiès Nov. 4, 2024, 3:28 p.m. UTC | #1 
Samuel Thibault <samuel.thibault@ens-lyon.org> writes:

> I reworked the patch to factorize the bsd signal definitions.
> I have split off the system definitions because the priority range of
> GNU/Mach has diverged from the original BSD kernels.

>  gcc/ada/ChangeLog                             |  17 ++

Hello,

The changes look good, I'll finish the review tomorrow.

But I can already point out that your 3 patches include changes to the
ChangeLog file, which should not be the case (it's automated from the
git log now).

Thanks!

Marc