public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
From: Alexandre Oliva <aoliva@gcc.gnu.org>
To: gcc-cvs@gcc.gnu.org
Subject: [gcc(refs/users/aoliva/heads/testme)] Introduce -nolibstdc++ option
Date: Tue, 21 Jun 2022 00:08:31 +0000 (GMT)	[thread overview]
Message-ID: <20220621000831.4DAFA3841440@sourceware.org> (raw)

https://gcc.gnu.org/g:0d4d8a3e90d79b1659db9331a00db86f8ba35420

commit 0d4d8a3e90d79b1659db9331a00db86f8ba35420
Author: Alexandre Oliva <oliva@adacore.com>
Date:   Mon Jun 20 19:44:23 2022 -0300

    Introduce -nolibstdc++ option
    
    Using g++ to link without libstdc++, as in g++.dg/abi/pure-virtual1.C,
    is error prone, because there's no way to tell g++ to drop libstdc++
    without also dropping libc and any other libraries that the target
    implicitly links in.
    
    This has often led to the need for manual adjustments to this
    testcase.
    
    I figured adding support for -nolibstdc++, even though redundant,
    makes some sense.  One could presumably use gcc rather than g++ for
    linking, for the same effect, but sometimes changing the link command
    is harder than adding an option, as in our testsuite.
    
    
    for  gcc/ChangeLog
    
            * common.opt (nolibstdc++): New.
            * doc/invoke.texi (-nolibstdc++): Document it.
    
    for  gcc/cp/ChangeLog
    
            * g++spec.c (lang_specific_driver): Implement -nolibstdc++.
    
    for  gcc/testsuite/ChangeLog
    
            * g++.dg/abi/pure-virtual1.C: Use -nolibstdc++.
    
    TN: V526-020

Diff:
---
 gcc/common.opt                           | 3 +++
 gcc/cp/g++spec.cc                        | 1 +
 gcc/doc/invoke.texi                      | 6 +++++-
 gcc/testsuite/g++.dg/abi/pure-virtual1.C | 2 +-
 4 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/gcc/common.opt b/gcc/common.opt
index 32917aafcae..e00c6fc2fb0 100644
--- a/gcc/common.opt
+++ b/gcc/common.opt
@@ -3456,6 +3456,9 @@ Driver
 nolibc
 Driver
 
+nolibstdc++
+Driver
+
 nostdlib
 Driver
 
diff --git a/gcc/cp/g++spec.cc b/gcc/cp/g++spec.cc
index 8174d652776..539e6ca089d 100644
--- a/gcc/cp/g++spec.cc
+++ b/gcc/cp/g++spec.cc
@@ -160,6 +160,7 @@ lang_specific_driver (struct cl_decoded_option **in_decoded_options,
 	{
 	case OPT_nostdlib:
 	case OPT_nodefaultlibs:
+	case OPT_nolibstdc__:
 	  library = -1;
 	  break;
 
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 50f57877477..469b6d97e0d 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -652,7 +652,7 @@ Objective-C and Objective-C++ Dialects}.
 @item Linker Options
 @xref{Link Options,,Options for Linking}.
 @gccoptlist{@var{object-file-name}  -fuse-ld=@var{linker}  -l@var{library} @gol
--nostartfiles  -nodefaultlibs  -nolibc  -nostdlib @gol
+-nostartfiles  -nodefaultlibs  -nolibc  -nolibstdc++  -nostdlib @gol
 -e @var{entry}  --entry=@var{entry} @gol
 -pie  -pthread  -r  -rdynamic @gol
 -s  -static  -static-pie  -static-libgcc  -static-libstdc++ @gol
@@ -16787,6 +16787,10 @@ absence of a C library is assumed, for example @option{-lpthread} or
 @option{-lm} in some configurations.  This is intended for bare-board
 targets when there is indeed no C library available.
 
+@item -nolibstdc++
+@opindex nolibstdc++
+Do not link with standard C++ libraries implicitly.
+
 @item -nostdlib
 @opindex nostdlib
 Do not use the standard system startup files or libraries when linking.
diff --git a/gcc/testsuite/g++.dg/abi/pure-virtual1.C b/gcc/testsuite/g++.dg/abi/pure-virtual1.C
index 538e2cb097a..889c33e4952 100644
--- a/gcc/testsuite/g++.dg/abi/pure-virtual1.C
+++ b/gcc/testsuite/g++.dg/abi/pure-virtual1.C
@@ -1,7 +1,7 @@
 // Test that we don't need libsupc++ just for __cxa_pure_virtual.
 // { dg-do link }
 // { dg-require-weak }
-// { dg-additional-options "-fno-rtti -nodefaultlibs -lc" }
+// { dg-additional-options "-fno-rtti -nolibstdc++" }
 // { dg-additional-options "-Wl,-undefined,dynamic_lookup" { target *-*-darwin* } }
 // { dg-xfail-if "AIX weak" { powerpc-ibm-aix* } }


             reply	other threads:[~2022-06-21  0:08 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-21  0:08 Alexandre Oliva [this message]
2022-06-21  0:16 Alexandre Oliva
2022-06-22  5:22 Alexandre Oliva

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=20220621000831.4DAFA3841440@sourceware.org \
    --to=aoliva@gcc.gnu.org \
    --cc=gcc-cvs@gcc.gnu.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).