public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
From: Thomas Schwinge <thomas@codesourcery.com>
To: <newlib@sourceware.org>
Cc: <tdevries@suse.de>, <ams@codesourcery.com>,
	Thomas Schwinge <thomas@codesourcery.com>
Subject: [PATCH 2/2] nvptx: Remove newlib ELIX level 1 restriction
Date: Thu, 19 Jan 2023 22:53:17 +0100	[thread overview]
Message-ID: <20230119215317.2001873-2-thomas@codesourcery.com> (raw)
In-Reply-To: <20230119215317.2001873-1-thomas@codesourcery.com>

Such a hard-coded ELIX level restriction is only being applied for nvptx
newlib -- but we'd actually like higher levels' functions available there,
too.  (Users continue to be able to override this via newlib 'configure',
as for every other newlib target.)

This already enables GCC test cases that currently FAIL due to
'unresolved symbol strndup' ('gcc.dg/builtin-dynamic-object-size-0.c'), or
'unresolved symbol mempcpy' ('gcc.dg/torture/pr45636.c'), for example.

Co-authored-by: Andrew Stubbs <ams@codesourcery.com>
---
 newlib/NEWS           | 1 +
 newlib/configure.host | 5 -----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/newlib/NEWS b/newlib/NEWS
index 9aa82920d..ee456f9b8 100644
--- a/newlib/NEWS
+++ b/newlib/NEWS
@@ -13,6 +13,7 @@
 - use global stdio streams for all configurations
 - use global atexit data for all configurations
 - vectorized math routines added for amdgcn platform
+- nvptx: remove newlib ELIX level 1 restriction

 *** Major changes in newlib version 4.2.0:

diff --git a/newlib/configure.host b/newlib/configure.host
index 7157877ff..0a1ba282a 100644
--- a/newlib/configure.host
+++ b/newlib/configure.host
@@ -938,11 +938,6 @@ fi
 # Target-specific defaults
 case "${host_cpu}" in
   nvptx*)
-       if [ "x${newlib_elix_level}" = "x0" ]; then
-               newlib_elix_level=1;
-       elif [ "x${newlib_elix_level}" != "x1" ]; then
-               newlib_msg_warn="nvptx does not support elix level > 1";
-       fi
        if [[ -z ${newlib_global_stdio_streams} ]]; then
                newlib_global_stdio_streams="yes";
        fi
--
2.25.1

-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

  reply	other threads:[~2023-01-19 21:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <ae825c453f484ffd99c9be34af726089@mentor.com>
2023-01-19 21:53 ` [PATCH 1/2] nvptx: Don't use global constructor for SSP setup Thomas Schwinge
2023-01-19 21:53   ` Thomas Schwinge [this message]
2023-01-19 21:53 ` [PATCH] nvptx: Provide stub 'getpid', 'kill', 'stat' Thomas Schwinge

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=20230119215317.2001873-2-thomas@codesourcery.com \
    --to=thomas@codesourcery.com \
    --cc=ams@codesourcery.com \
    --cc=newlib@sourceware.org \
    --cc=tdevries@suse.de \
    /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).