public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: libc-alpha@sourceware.org, Wilco Dijkstra <Wilco.Dijkstra@arm.com>
Subject: [PATCH 0/7] Remove _STRING_ARCH_unaligned
Date: Mon, 13 Feb 2023 10:55:51 -0300	[thread overview]
Message-ID: <20230213135558.3328727-1-adhemerval.zanella@linaro.org> (raw)

The _STRING_ARCH_unaligned advertise that the architecture allows
efficient unaligned memory accesses and it is used to optimize some
implementations.

However, some are complete unnecessary and only adds complexity
(getenv), while other can be removed since they add only marginal
improvement on legacy/compat code (crypt and nscd).

The iconv code can be also simplified by using compiler extension
(packaed struct) that allows code efficient code generation without
the need to provide aligned/unaligned variants.

Adhemerval Zanella (7):
  crypto: Remove _STRING_ARCH_unaligned usage
  stdlib: Simplify getenv
  nscd: Remove _STRING_ARCH_unaligned usage
  resolv: Remove _STRING_ARCH_unaligned usage
  iconv: Remove _STRING_ARCH_unaligned usage for get/set macros
  iconv: Remove _STRING_ARCH_unaligned usage
  string: Remove string_private.h

 crypt/md5.c                                 |  24 +-
 crypt/sha256.c                              |  28 +-
 crypt/sha512.c                              |  26 +-
 iconv/gconv_int.h                           |  28 ++
 iconv/gconv_simple.c                        | 282 ++------------------
 iconv/loop.c                                | 139 ++--------
 iconv/skeleton.c                            | 185 ++-----------
 iconvdata/iso-2022-jp-3.c                   |   2 +-
 iconvdata/unicode.c                         |   6 +-
 iconvdata/utf-16.c                          |   6 +-
 iconvdata/utf-32.c                          |   6 +-
 include/arpa/nameser.h                      |  36 ---
 include/string.h                            |   3 -
 nscd/nscd_gethst_r.c                        |   2 -
 nscd/nscd_getserv_r.c                       |   2 -
 nscd/nscd_helper.c                          |   6 -
 stdlib/getenv.c                             |  63 +----
 sysdeps/aarch64/string_private.h            |  20 --
 sysdeps/generic/string_private.h            |  21 --
 sysdeps/m68k/m680x0/m68020/string_private.h |  21 --
 sysdeps/s390/string_private.h               |  20 --
 sysdeps/s390/utf16-utf32-z9.c               |   4 +-
 sysdeps/s390/utf8-utf16-z9.c                |   2 +-
 sysdeps/s390/utf8-utf32-z9.c                |   2 +-
 sysdeps/x86/string_private.h                |  20 --
 25 files changed, 118 insertions(+), 836 deletions(-)
 delete mode 100644 sysdeps/aarch64/string_private.h
 delete mode 100644 sysdeps/generic/string_private.h
 delete mode 100644 sysdeps/m68k/m680x0/m68020/string_private.h
 delete mode 100644 sysdeps/s390/string_private.h
 delete mode 100644 sysdeps/x86/string_private.h

-- 
2.34.1


             reply	other threads:[~2023-02-13 13:56 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-13 13:55 Adhemerval Zanella [this message]
2023-02-13 13:55 ` [PATCH 1/7] crypto: Remove _STRING_ARCH_unaligned usage Adhemerval Zanella
2023-02-15 17:55   ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 2/7] stdlib: Simplify getenv Adhemerval Zanella
2023-02-15 17:50   ` Wilco Dijkstra
2023-02-16 13:49     ` Adhemerval Zanella Netto
2023-02-16 14:02       ` Andreas Schwab
2023-02-16 18:02       ` Wilco Dijkstra
2023-02-16 18:38         ` Adhemerval Zanella Netto
2023-02-16 18:47           ` Wilco Dijkstra
2023-02-16 19:29             ` Adhemerval Zanella Netto
2023-02-17 17:13               ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 3/7] nscd: Remove _STRING_ARCH_unaligned usage Adhemerval Zanella
2023-02-15 17:59   ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 4/7] resolv: " Adhemerval Zanella
2023-02-15 18:04   ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 5/7] iconv: Remove _STRING_ARCH_unaligned usage for get/set macros Adhemerval Zanella
2023-02-13 14:05   ` Andreas Schwab
2023-02-15 18:34   ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 6/7] iconv: Remove _STRING_ARCH_unaligned usage Adhemerval Zanella
2023-02-15 19:02   ` Wilco Dijkstra
2023-02-13 13:55 ` [PATCH 7/7] string: Remove string_private.h Adhemerval Zanella
2023-02-15 19:04   ` Wilco Dijkstra

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=20230213135558.3328727-1-adhemerval.zanella@linaro.org \
    --to=adhemerval.zanella@linaro.org \
    --cc=Wilco.Dijkstra@arm.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).