public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/94704] [8/9/10 Regression] class with empty base passed incorrectly with -std=c++17 on s390x/s390
Date: Mon, 27 Apr 2020 07:12:57 +0000	[thread overview]
Message-ID: <bug-94704-4-QdXBGXszhb@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-94704-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94704

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jakub Jelinek <jakub@gcc.gnu.org>:

https://gcc.gnu.org/g:9612a4833d761e3beda083a3e4dc92feba3b01bc

commit r10-7985-g9612a4833d761e3beda083a3e4dc92feba3b01bc
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Mon Apr 27 09:11:57 2020 +0200

    s390: Fix C++14 vs. C++17 ABI incompatibility on s390{,x} [PR94704]

    The following patch fixes the C++14 vs. C++17 ABI passing incompatibility
    on s390x-linux.

    Bootstrapped/regtested on s390x-linux without and with the patch, the
    difference being:
    -FAIL: tmpdir-g++.dg-struct-layout-1/t032
cp_compat_x_alt.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t032
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t032
cp_compat_x_tst.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t055
cp_compat_x_alt.o-cp_compat_y_alt.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t055
cp_compat_x_alt.o-cp_compat_y_tst.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t055
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t055
cp_compat_x_tst.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t056
cp_compat_x_alt.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t056
cp_compat_x_alt.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t056
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t056
cp_compat_x_tst.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t057
cp_compat_x_alt.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t057
cp_compat_x_alt.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t057
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t057
cp_compat_x_tst.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t058
cp_compat_x_alt.o-cp_compat_y_alt.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t058
cp_compat_x_alt.o-cp_compat_y_tst.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t058
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t058
cp_compat_x_tst.o-cp_compat_y_tst.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t059
cp_compat_x_alt.o-cp_compat_y_alt.o execute
     FAIL: tmpdir-g++.dg-struct-layout-1/t059
cp_compat_x_alt.o-cp_compat_y_tst.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t059
cp_compat_x_tst.o-cp_compat_y_alt.o execute
    -FAIL: tmpdir-g++.dg-struct-layout-1/t059
cp_compat_x_tst.o-cp_compat_y_tst.o execute
    when performing ALT_CXX_UNDER_TEST=g++ testing with a system GCC 10
compiler
    from a week ago.  So, the alt vs. alt FAILs are all expected (we know
before
    this patch there is an ABI incompatibility) and some alt vs. tst (or tst
vs.
    alt) FAILs too - that depends on if the particular x or y test is compiled
    with -std=c++14 or -std=c++17 - if x_tst is compiled with -std=c++14 and
    y_alt is compiled with -std=c++17, then it should FAIL, similarly if x_alt
    is compiled with -std=c++17 and y_tst is compiled with -std=c++14.

    2020-04-27  Jakub Jelinek  <jakub@redhat.com>

            PR target/94704
            * config/s390/s390.c (s390_function_arg_vector,
            s390_function_arg_float): Ignore cxx17_empty_base_field_p fields.

  parent reply	other threads:[~2020-04-27  7:12 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-22  7:05 [Bug target/94704] New: " jakub at gcc dot gnu.org
2020-04-22  7:06 ` [Bug target/94704] " jakub at gcc dot gnu.org
2020-04-22 12:10 ` jakub at gcc dot gnu.org
2020-04-22 12:41 ` jakub at gcc dot gnu.org
2020-04-22 16:38 ` krebbel at gcc dot gnu.org
2020-04-23 14:38 ` jakub at gcc dot gnu.org
2020-04-27  7:12 ` cvs-commit at gcc dot gnu.org [this message]
2020-04-27  7:14 ` jakub at gcc dot gnu.org
2020-04-28  8:27 ` cvs-commit at gcc dot gnu.org
2020-04-29 20:40 ` cvs-commit at gcc dot gnu.org
2021-11-05 23:18 ` timturnerc at yahoo dot com

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=bug-94704-4-QdXBGXszhb@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@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).