From aa2c978400f3b3ca6e9f2d18598a379589e77ba0 Mon Sep 17 00:00:00 2001 From: Jason Merrill Date: Mon, 11 May 2020 14:05:46 -0400 Subject: c++: Make references to __cxa_pure_virtual weak. If a program has no other dependencies on libstdc++, we shouldn't require it just for __cxa_pure_virtual, which is only there to give a prettier diagnostic before crashing the program; resolving the reference to NULL will also crash, just without the diagnostic. gcc/cp/ChangeLog 2020-05-11 Jason Merrill * decl.c (cxx_init_decl_processing): Call declare_weak for __cxa_pure_virtual. --- gcc/cp/ChangeLog | 5 +++++ gcc/cp/decl.c | 3 +++ gcc/testsuite/g++.dg/abi/pure-virtual1.C | 21 +++++++++++++++++++++ 3 files changed, 29 insertions(+) create mode 100644 gcc/testsuite/g++.dg/abi/pure-virtual1.C diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 1fdc3880fff..2cc9b8deb15 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,8 @@ +2020-05-11 Jason Merrill + + * decl.c (cxx_init_decl_processing): Call declare_weak for + __cxa_pure_virtual. + 2020-05-11 Jason Merrill * pt.c (instantiate_class_template_1): Call tsubst_expr for diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index dea1ba07c0e..1b6a5672334 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -4544,6 +4544,9 @@ cxx_init_decl_processing (void) abort_fndecl = build_library_fn_ptr ("__cxa_pure_virtual", void_ftype, ECF_NORETURN | ECF_NOTHROW | ECF_COLD); + if (flag_weak) + /* If no definition is available, resolve references to NULL. */ + declare_weak (abort_fndecl); /* Perform other language dependent initializations. */ init_class_processing (); diff --git a/gcc/testsuite/g++.dg/abi/pure-virtual1.C b/gcc/testsuite/g++.dg/abi/pure-virtual1.C new file mode 100644 index 00000000000..823328ea951 --- /dev/null +++ b/gcc/testsuite/g++.dg/abi/pure-virtual1.C @@ -0,0 +1,21 @@ +// 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" } + +struct A +{ + int i; + virtual void f() = 0; + A(): i(0) {} +}; + +struct B: A +{ + virtual void f() {} +}; + +int main() +{ + B b; +} -- cgit v1.2.3