From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pj1-x1033.google.com (mail-pj1-x1033.google.com [IPv6:2607:f8b0:4864:20::1033]) by sourceware.org (Postfix) with ESMTPS id 3F1123858C56 for ; Tue, 21 Jun 2022 14:46:13 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 3F1123858C56 Received: by mail-pj1-x1033.google.com with SMTP id b12-20020a17090a6acc00b001ec2b181c98so12283739pjm.4 for ; Tue, 21 Jun 2022 07:46:13 -0700 (PDT) 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=CF6wBVcJGSBhVVVNAC67lQn3FYfG9dciFwV8KInLi2k=; b=iVjhVemaVxjTvQ6xvJuyklXDbqNPLVIwsCSw1wy7KgT/L5qv+BUQCfW7A9lXnu8bQm FeRAPhfjqTVS/hI3Sue9pW7mpGZSZKgF4iBFXfh1rPUnGftozBxkryIgENkCvk0qfmQx WzIu5MFdXYCwkVfWPiIkwaFDPvLoYiOhA73cpU/O/8DZvOrHq89nvptT8nFl7crqhRge 2ZrsqFN3dB58Wf4Gt4n/jS8l7dYPYGIFfFccysI99nOG6D0jO7A5CsWaTBBqZkyoMdzF VmW4Qu8kIS0xD3V+ngEbmNOBvBFvOHITXULbBh0dcdZkr02vLczYCA8dZaIPPLVoYC4K mdCA== X-Gm-Message-State: AJIora8KJMSIaocty8oFYiht0QET+5ZczcxbksUbJiAn9ZcSzm9ocw9Y id2Rr3qpi3sMouzjZszxvts4JiOuUb59dPriNW8= X-Google-Smtp-Source: AGRyM1v3cze6w/v2GRKW7JMb2LCmLmItvH0R4hAIIzCLy8i2PUCtL1k3bywfFJRbgkvKbZnbr4Zcr+rr4mtH4U5xu3g= X-Received: by 2002:a17:902:a502:b0:15e:c251:b769 with SMTP id s2-20020a170902a50200b0015ec251b769mr28885074plq.115.1655822772036; Tue, 21 Jun 2022 07:46:12 -0700 (PDT) MIME-Version: 1.0 References: <20220614192548.1929268-1-hjl.tools@gmail.com> In-Reply-To: From: "H.J. Lu" Date: Tue, 21 Jun 2022 07:45:36 -0700 Message-ID: Subject: Re: PING^1 [PATCH] i386: Disallow sibcall when calling ifunc functions with PIC register To: Uros Bizjak Cc: GCC Patches Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-3025.2 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, GIT_PATCH_0, 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.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Jun 2022 14:46:14 -0000 On Mon, Jun 20, 2022 at 7:51 AM Uros Bizjak wrote: > > On Mon, Jun 20, 2022 at 4:03 PM H.J. Lu wrote: > > > > On Tue, Jun 14, 2022 at 12:25 PM H.J. Lu wrote: > > > > > > Disallow siball when calling ifunc functions with PIC register so that > > > PIC register can be restored. > > > > > > gcc/ > > > > > > PR target/105960 > > > * config/i386/i386.cc (ix86_function_ok_for_sibcall): Return > > > false if PIC register is used when calling ifunc functions. > > > > > > gcc/testsuite/ > > > > > > PR target/105960 > > > * gcc.target/i386/pr105960.c: New test. > > LGTM. OK to backport to GCC 12 branch? Thanks. > Thanks, > Uros. > > > > --- > > > gcc/config/i386/i386.cc | 9 +++++++++ > > > gcc/testsuite/gcc.target/i386/pr105960.c | 19 +++++++++++++++++++ > > > 2 files changed, 28 insertions(+) > > > create mode 100644 gcc/testsuite/gcc.target/i386/pr105960.c > > > > > > diff --git a/gcc/config/i386/i386.cc b/gcc/config/i386/i386.cc > > > index 3d189e124e4..1ca7836e11e 100644 > > > --- a/gcc/config/i386/i386.cc > > > +++ b/gcc/config/i386/i386.cc > > > @@ -1015,6 +1015,15 @@ ix86_function_ok_for_sibcall (tree decl, tree exp) > > > } > > > } > > > > > > + if (decl && ix86_use_pseudo_pic_reg ()) > > > + { > > > + /* When PIC register is used, it must be restored after ifunc > > > + function returns. */ > > > + cgraph_node *node = cgraph_node::get (decl); > > > + if (node && node->ifunc_resolver) > > > + return false; > > > + } > > > + > > > /* Otherwise okay. That also includes certain types of indirect calls. */ > > > return true; > > > } > > > diff --git a/gcc/testsuite/gcc.target/i386/pr105960.c b/gcc/testsuite/gcc.target/i386/pr105960.c > > > new file mode 100644 > > > index 00000000000..db137a1642d > > > --- /dev/null > > > +++ b/gcc/testsuite/gcc.target/i386/pr105960.c > > > @@ -0,0 +1,19 @@ > > > +/* { dg-do compile } */ > > > +/* { dg-require-ifunc "" } */ > > > +/* { dg-options "-O2 -fpic" } */ > > > + > > > +__attribute__((target_clones("default","fma"))) > > > +static inline double > > > +expfull_ref(double x) > > > +{ > > > + return __builtin_pow(x, 0.1234); > > > +} > > > + > > > +double > > > +exp_ref(double x) > > > +{ > > > + return expfull_ref(x); > > > +} > > > + > > > +/* { dg-final { scan-assembler "jmp\[ \t\]*expfull_ref@PLT" { target { ! ia32 } } } } */ > > > +/* { dg-final { scan-assembler "call\[ \t\]*expfull_ref@PLT" { target ia32 } } } */ > > > -- > > > 2.36.1 > > > > > > > PING. > > > > -- > > H.J. -- H.J.