public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: John David Anglin <dave.anglin@bell.net>
To: "Martin Liška" <mliska@suse.cz>, gcc-patches@gcc.gnu.org
Cc: Jeff Law <jeffreyalaw@gmail.com>
Subject: Re: [PATCH] 32-bit PA-RISC with HP-UX: remove deprecated ports
Date: Tue, 30 Aug 2022 15:51:51 -0400	[thread overview]
Message-ID: <fd3a31f5-3b82-7476-d891-39dc7a76a390@bell.net> (raw)
In-Reply-To: <d889b8b8-411d-a2a1-2492-b074cebeaadc@suse.cz>

On 2022-08-29 10:06 a.m., Martin Liška wrote:
> Thanks for the feedback, can you please check the updated version of the patch?
@@ -353,9 +347,6 @@ proc check_weak_available { } {
  # return 1 if weak undefined symbols are supported.

  proc check_effective_target_weak_undefined { } {
-    if { [istarget hppa*-*-hpux*] } {
-       return 0
-    }

This code needs to be retained for hppa64.

@@ -562,7 +553,6 @@ proc check_effective_target_trampolines { } {
      if { [istarget avr-*-*]
          || [istarget msp430-*-*]
          || [istarget nvptx-*-*]
-        || [istarget hppa2.0w-hp-hpux11.23]
          || [istarget hppa64-hp-hpux11.23]
          || [istarget pru-*-*]
          || [istarget bpf-*-*] } {

The above looks odd.  As far as I know, support for trampolines doesn't depend in any way on hpux11.23.
I suspect the hppa64 line can be removed as well.

@@ -1,4 +0,0 @@
-# The ada virtual array implementation requires that indexing be disabled on
-# hosts such as hpux that use a segmented memory architecture. Both the c
-# and ada files need to be compiled with this option for correct operation.
-ADA_CFLAGS = -mdisable-indexing -D_X_HPUX10
diff --git a/config/mmap.m4 b/config/mmap.m4
index fba0d9d3657..53e5215d833 100644
--- a/config/mmap.m4
+++ b/config/mmap.m4
@@ -42,7 +42,7 @@ else
     # Systems known to be in this category are Windows (all variants),
     # VMS, and Darwin.
     case "$host_os" in
-     *vms* | cygwin* | pe | mingw* | darwin* | ultrix* | hpux10* | hpux11.00)
+     *vms* | cygwin* | pe | mingw* | darwin* | ultrix* | hpux11.00)
          gcc_cv_func_mmap_dev_zero=no ;;
       *)
          gcc_cv_func_mmap_dev_zero=yes;;

Strictly, all hpux versions up to and including hpux11.00 don't have /dev/zero.

Dave

-- 
John David Anglin  dave.anglin@bell.net


WARNING: multiple messages have this Message-ID
From: John David Anglin <dave.anglin@bell.net>
To: "Martin Liška" <mliska@suse.cz>, gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] 32-bit PA-RISC with HP-UX: remove deprecated ports
Date: Tue, 30 Aug 2022 15:51:51 -0400	[thread overview]
Message-ID: <fd3a31f5-3b82-7476-d891-39dc7a76a390@bell.net> (raw)
Message-ID: <20220830195151.N6ndohK_hRLUhFYoPaKXqThhRbxdqIlzzp876efBQxo@z> (raw)
In-Reply-To: <d889b8b8-411d-a2a1-2492-b074cebeaadc@suse.cz>

On 2022-08-29 10:06 a.m., Martin Liška wrote:
> Thanks for the feedback, can you please check the updated version of the patch?
@@ -353,9 +347,6 @@ proc check_weak_available { } {
  # return 1 if weak undefined symbols are supported.

  proc check_effective_target_weak_undefined { } {
-    if { [istarget hppa*-*-hpux*] } {
-       return 0
-    }

This code needs to be retained for hppa64.

@@ -562,7 +553,6 @@ proc check_effective_target_trampolines { } {
      if { [istarget avr-*-*]
          || [istarget msp430-*-*]
          || [istarget nvptx-*-*]
-        || [istarget hppa2.0w-hp-hpux11.23]
          || [istarget hppa64-hp-hpux11.23]
          || [istarget pru-*-*]
          || [istarget bpf-*-*] } {

The above looks odd.  As far as I know, support for trampolines doesn't depend in any way on hpux11.23.
I suspect the hppa64 line can be removed as well.

@@ -1,4 +0,0 @@
-# The ada virtual array implementation requires that indexing be disabled on
-# hosts such as hpux that use a segmented memory architecture. Both the c
-# and ada files need to be compiled with this option for correct operation.
-ADA_CFLAGS = -mdisable-indexing -D_X_HPUX10
diff --git a/config/mmap.m4 b/config/mmap.m4
index fba0d9d3657..53e5215d833 100644
--- a/config/mmap.m4
+++ b/config/mmap.m4
@@ -42,7 +42,7 @@ else
     # Systems known to be in this category are Windows (all variants),
     # VMS, and Darwin.
     case "$host_os" in
-     *vms* | cygwin* | pe | mingw* | darwin* | ultrix* | hpux10* | hpux11.00)
+     *vms* | cygwin* | pe | mingw* | darwin* | ultrix* | hpux11.00)
          gcc_cv_func_mmap_dev_zero=no ;;
       *)
          gcc_cv_func_mmap_dev_zero=yes;;

Strictly, all hpux versions up to and including hpux11.00 don't have /dev/zero.

Dave

-- 
John David Anglin  dave.anglin@bell.net


  parent reply	other threads:[~2022-08-30 19:51 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-26  7:15 Martin Liška
2022-08-26  7:20 ` Andreas Schwab
2022-08-26  7:22   ` Martin Liška
2022-08-26 19:09 ` John David Anglin
2022-08-26 19:09   ` John David Anglin
2022-08-28 16:34 ` John David Anglin
2022-08-28 16:34   ` John David Anglin
2022-08-29 14:06   ` Martin Liška
2022-08-29 14:06     ` Martin Liška
2022-08-30 18:39     ` John David Anglin
2022-08-30 18:39       ` John David Anglin
2022-08-30 19:00     ` John David Anglin
2022-08-30 19:00       ` John David Anglin
2022-08-30 19:51     ` John David Anglin [this message]
2022-08-30 19:51       ` John David Anglin
2022-08-31  0:13   ` Jeff Law
2022-08-31  0:32     ` John David Anglin
2022-08-31  7:21       ` [PATCH][V3] " Martin Liška
2022-08-31 15:35         ` Jeff Law
2022-08-31 16:40           ` John David Anglin
2022-08-31 17:24             ` Jeff Law
2022-09-01 13:08         ` John David Anglin
2022-09-01 13:08           ` John David Anglin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=fd3a31f5-3b82-7476-d891-39dc7a76a390@bell.net \
    --to=dave.anglin@bell.net \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jeffreyalaw@gmail.com \
    --cc=mliska@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).