unofficial mirror of libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: Joseph Myers <joseph@codesourcery.com>
Cc: libc-alpha@sourceware.org, David Miller <davem@davemloft.net>
Subject: Re: [PATCH] Remove 32 bit sparc v8 support
Date: Mon, 11 Nov 2019 16:14:31 -0300	[thread overview]
Message-ID: <1afe4a2b-0b4b-38d9-cb22-4bdc21e0cfd4@linaro.org> (raw)
In-Reply-To: <alpine.DEB.2.21.1911111821390.15838@digraph.polyomino.org.uk>



On 11/11/2019 15:24, Joseph Myers wrote:
> On Mon, 11 Nov 2019, Adhemerval Zanella wrote:
> 
>>  sysdeps/sparc/sparc32/divrem.m4               | 234 -------
>>  sysdeps/sparc/sparc32/rem.S                   | 365 +---------
>>  sysdeps/sparc/sparc32/sdiv.S                  | 363 +---------
>>  sysdeps/sparc/sparc32/udiv.S                  | 343 +--------
>>  sysdeps/sparc/sparc32/urem.S                  | 345 +--------
> 
> Since you're replacing these files generated from divrem.m4 with 
> non-generated files moved from the v9 directory, the code in 
> build-many-glibcs.py to touch them on checkout should also be removed, as 
> there is no longer any constraint on how their timestamps are ordered with 
> respect to other files in the source tree.
> 

Something like:

diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py
index f174094ff3..cd9ac08f52 100755
--- a/scripts/build-many-glibcs.py
+++ b/scripts/build-many-glibcs.py
@@ -358,7 +358,7 @@ class Context(object):
         self.add_config(arch='sparc64',
                         os_name='linux-gnu',
                         glibcs=[{},
-                                {'arch': 'sparcv9',
+                                {'arch': 'sparc',
                                  'ccopts': '-m32 -mlong-double-128 -mcpu=v9'}],
                         extra_glibcs=[{'variant': 'disable-multi-arch',
                                        'cfg': ['--disable-multi-arch']},
@@ -847,11 +847,7 @@ class Context(object):
         # be touched because nothing in a build depends on the files
         # in question.
         for f in ('sysdeps/gnu/errlist.c',
-                  'sysdeps/mach/hurd/bits/errno.h',
-                  'sysdeps/sparc/sparc32/rem.S',
-                  'sysdeps/sparc/sparc32/sdiv.S',
-                  'sysdeps/sparc/sparc32/udiv.S',
-                  'sysdeps/sparc/sparc32/urem.S'):
+                  'sysdeps/mach/hurd/bits/errno.h'):
             to_touch = os.path.join(srcdir, f)
             subprocess.run(['touch', '-c', to_touch], check=True)
         for dirpath, dirnames, filenames in os.walk(srcdir):

?

  reply	other threads:[~2019-11-11 19:14 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-11 17:43 [PATCH] Remove 32 bit sparc v8 support Adhemerval Zanella
2019-11-11 18:24 ` Joseph Myers
2019-11-11 19:14   ` Adhemerval Zanella [this message]
2019-11-11 22:27     ` Joseph Myers
2019-11-12 12:17 ` [PATCH v2] " Adhemerval Zanella
2019-11-12 13:35   ` Romain Naour
2019-11-12 15:39   ` Andreas Larsson
2019-11-12 20:43     ` Adhemerval Zanella
2019-11-13 19:28       ` [PATCH 1/2] Remove 32 bit sparc v7 support Adhemerval Zanella
2019-11-13 19:28         ` [PATCH 2/2] sparc: Use atomic compiler builtins on sparc Adhemerval Zanella
2019-11-19 14:23           ` Andreas Larsson
2019-11-21 19:43             ` Adhemerval Zanella
2019-11-26 18:16             ` Adhemerval Zanella
2019-11-13 20:32         ` [PATCH 1/2] Remove 32 bit sparc v7 support Joseph Myers
2019-11-13 20:34           ` Adhemerval Zanella
2019-11-26 18:16             ` Adhemerval Zanella
2019-11-28 17:23               ` Joseph Myers
2019-11-28 17:33                 ` Joseph Myers
2019-11-28 19:29                   ` Adhemerval Zanella
2019-11-28 20:10                     ` Joseph Myers

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

  List information: https://www.gnu.org/software/libc/involved.html

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

  git send-email \
    --in-reply-to=1afe4a2b-0b4b-38d9-cb22-4bdc21e0cfd4@linaro.org \
    --to=adhemerval.zanella@linaro.org \
    --cc=davem@davemloft.net \
    --cc=joseph@codesourcery.com \
    --cc=libc-alpha@sourceware.org \
    /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).