From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <hjl.tools@gmail.com>
Received: from mail-pl1-x62c.google.com (mail-pl1-x62c.google.com
 [IPv6:2607:f8b0:4864:20::62c])
 by sourceware.org (Postfix) with ESMTPS id 586BB3858013
 for <gcc-patches@gcc.gnu.org>; Tue,  8 Mar 2022 17:51:32 +0000 (GMT)
DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 586BB3858013
Received: by mail-pl1-x62c.google.com with SMTP id e2so17745675pls.10
 for <gcc-patches@gcc.gnu.org>; Tue, 08 Mar 2022 09:51:32 -0800 (PST)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20210112;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=rpLezLsP8QE0nxaI1e2LTdRHB4sPvsqNeINYS/xRcjA=;
 b=ZHCOhHXFuLUQGNtPTmzTkX+9ZJH+YvAT7FsPkoYG0Qnu5ubRoxL0yj4CqAPpwUS8/v
 0FvyXZ+2B/mKPzVFlRHP4B7Z/D4wiLSR9l6uriWbXAf4+4ZfWyIEw/DjUzjfPpcAEHGx
 JBr3o2c9pmcB4685klLQk1U3DYBb6NkSKOB/rqIPq2h4uEyZwEiGqa4QH/lEjG9wIY/i
 iJX2vSWzTQ0FVmNpT61VUUYmqYZiWu/iJ7iCEcugKfqQ8MtX2kDuGASr2yJYpZu1t5dA
 OL9JOxq0meI+hBEQKmDk6i/yXK9z3ZYlUzHb8UvaAn1/nh/WQSO4VkRnExLZ0PgxW7B5
 oLKg==
X-Gm-Message-State: AOAM530C+IAULQ6mOdNqihzcWOAvXwB9p0a37HXz5KYJuVPiM5ZXwqRY
 IVm/COFRUMUkhNVVPi/2LjmThjz+Uk96bHtHCsQ=
X-Google-Smtp-Source: ABdhPJzHGdEwsiq1sLkEcgdzJpYFCYMyYuptCw4lAYRaZKzhnzzoRz83SHDszweiPgrUOkmeFqSW3aaRGrcukhAPqtw=
X-Received: by 2002:a17:90a:1b4a:b0:1bf:1112:5ef with SMTP id
 q68-20020a17090a1b4a00b001bf111205efmr5890259pjq.143.1646761891470; Tue, 08
 Mar 2022 09:51:31 -0800 (PST)
MIME-Version: 1.0
References: <20220307150628.68146-1-hjl.tools@gmail.com>
 <YicutzHS7vFn9x5n@tucnak>
 <Yic6wxdaLDVd3/oE@tucnak> <YidMOvVHLWBwgBxz@tucnak>
 <CAMe9rOq=+e1fD7edAwF+XMARCePvbJ-SN1=4BdaX3P=uXYUtLQ@mail.gmail.com>
 <Yid6OgTq8uH+Ze4O@tucnak>
 <CAMe9rOosJO5wz5=ZibD-sL_JL+uAoc+-m4B4BT8AK1g5jc619g@mail.gmail.com>
 <YieT0l9915XwHt7c@tucnak>
In-Reply-To: <YieT0l9915XwHt7c@tucnak>
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Tue, 8 Mar 2022 09:50:55 -0800
Message-ID: <CAMe9rOoCaqsEiD9H1Qv_a1Lq9A-PimXv_825YtKNE5FYFEmi3w@mail.gmail.com>
Subject: Re: [PATCH] x86: Define LIBGCC2_UNWIND_ATTRIBUTE on ia32 [PR104781]
To: Jakub Jelinek <jakub@redhat.com>
Cc: Uros Bizjak <ubizjak@gmail.com>, Richard Biener <rguenther@suse.de>, 
 Jeff Law <jeffreyalaw@gmail.com>, gcc-patches@gcc.gnu.org
Content-Type: text/plain; charset="UTF-8"
X-Spam-Status: No, score=-3020.6 required=5.0 tests=BAYES_00, DKIM_SIGNED,
 DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, KAM_SHORT,
 RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP,
 T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4
X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on
 server2.sourceware.org
X-BeenThere: gcc-patches@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-patches mailing list <gcc-patches.gcc.gnu.org>
List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>,
 <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help>
List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>,
 <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Tue, 08 Mar 2022 17:51:33 -0000

On Tue, Mar 8, 2022 at 9:35 AM Jakub Jelinek <jakub@redhat.com> wrote:
>
> On Tue, Mar 08, 2022 at 08:09:25AM -0800, H.J. Lu wrote:
> > > Ok.  So, what do you think about replacing the libgcc/ part of your patch
> > > with that
> > > /* __builtin_eh_return can't handle stack realignment, so disable SSE in
> > >    32-bit libgcc functions that call it.  */
> > > #ifndef __x86_64__
> > > #define LIBGCC2_UNWIND_ATTRIBUTE __attribute__((target ("no-sse")))
> > > #endif
> > > ?
> >
> > Yes, it should work.
>
> So, how do we move on with this?
> I can't self-approve my own patch, so can anyone please ack the following
> provided it passes bootstraps/regtests ({x86_64,i686}-linux) that are
> currently pending?
>
> That can go in independently from your patch, and if it is committed,
> your V3 patch with the libgcc/ hunks removed is preapproved for trunk.

I am checking in this:

https://gcc.gnu.org/pipermail/gcc-patches/2022-March/591392.html

Thanks.

> 2022-03-08  Jakub Jelinek  <jakub@redhat.com>
>
>         PR target/104781
>         * config/i386/i386.h (LIBGCC2_UNWIND_ATTRIBUTE): Define for ia32.
>
> --- gcc/config/i386/i386.h.jj   2022-02-25 12:06:45.535493490 +0100
> +++ gcc/config/i386/i386.h      2022-03-08 11:20:43.207043370 +0100
> @@ -2848,6 +2848,12 @@ extern enum attr_cpu ix86_schedule;
>  #define NUM_X86_64_MS_CLOBBERED_REGS 12
>  #endif
>
> +/* __builtin_eh_return can't handle stack realignment, so disable SSE in
> +   32-bit libgcc functions that call it.  */
> +#ifndef __x86_64__
> +#define LIBGCC2_UNWIND_ATTRIBUTE __attribute__((target ("no-sse")))
> +#endif
> +
>  /*
>  Local variables:
>  version-control: t
>
>
>         Jakub
>


-- 
H.J.