public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r12-7084] testsuite: Fix up testsuite/gcc.c-torture/execute/builtins/lib/chk.c for powerpc [PR104380]
@ 2022-02-07 16:40 Jakub Jelinek
  0 siblings, 0 replies; only message in thread
From: Jakub Jelinek @ 2022-02-07 16:40 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:5d2a330dee37ed10daa424d33ab1bcd5727c187b

commit r12-7084-g5d2a330dee37ed10daa424d33ab1bcd5727c187b
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Mon Feb 7 17:39:11 2022 +0100

    testsuite: Fix up testsuite/gcc.c-torture/execute/builtins/lib/chk.c for powerpc [PR104380]
    
    > > The following testcase FAILs when configured with
    > > --with-long-double-format=ieee .  Only happens in the -std=c* modes, not the
    > > GNU modes; while the glibc headers have __asm redirects of
    > > vsnprintf and __vsnprinf_chk to __vsnprintfieee128 and
    > > __vsnprintf_chkieee128, the vsnprintf fortification extern inline gnu_inline
    > > always_inline wrapper calls __builtin_vsnprintf_chk and we actually emit
    > > a call to __vsnprinf_chk (i.e. with IBM extended long double) instead of
    > > __vsnprintf_chkieee128.
    > >
    > > rs6000_mangle_decl_assembler_name already had cases for *printf and *scanf,
    > > so this just adds another case for *printf_chk.  *scanf_chk doesn't exist.
    > > __ prefixing isn't done because *printf_chk already starts with __.
    
    Unfortunately, while I've tested the testcase also with -mabi=ieeelongdouble
    by hand, the full bootstrap/regtest was on GCCFarm where glibc is too old
    to test with --with-long-double-format=ieee.
    I've done full bootstrap/regtest with that option during the weekend and
    the patch regressed:
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O1
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O2
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O2 -flto -fno-use-linker-plugin -flto-partition=none
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -O3 -g
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -Og -g
    FAIL: gcc.c-torture/execute/builtins/snprintf-chk.c execution,  -Os
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O1
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O2
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O2 -flto -fno-use-linker-plugin -flto-partition=none
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -O3 -g
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -Og -g
    FAIL: gcc.c-torture/execute/builtins/sprintf-chk.c execution,  -Os
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O1
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O2
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O2 -flto -fno-use-linker-plugin -flto-partition=none
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -O3 -g
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -Og -g
    FAIL: gcc.c-torture/execute/builtins/vsnprintf-chk.c execution,  -Os
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O1
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O2
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O2 -flto -fno-use-linker-plugin -flto-partition=none
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O2 -flto -fuse-linker-plugin -fno-fat-lto-objects
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -O3 -g
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -Og -g
    FAIL: gcc.c-torture/execute/builtins/vsprintf-chk.c execution,  -Os
    
    The problem is that the execute/builtins/ testsuite wants to override some
    of the library functions and with the change we (correctly) call
    __*printf_chkieee128 and so lib/chk.c is no longer called but the glibc
    APIs are.
    
    2022-02-07  Jakub Jelinek  <jakub@redhat.com>
    
            PR target/104380
            * gcc.c-torture/execute/builtins/lib/chk.c (__sprintf_chkieee128,
            __vsprintf_chkieee128, __snprintf_chkieee128,
            __vsnprintf_chkieee128): New aliases to non-ieee128 suffixed functions
            for powerpc -mabi=ieeelongdouble.

Diff:
---
 gcc/testsuite/gcc.c-torture/execute/builtins/lib/chk.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/gcc/testsuite/gcc.c-torture/execute/builtins/lib/chk.c b/gcc/testsuite/gcc.c-torture/execute/builtins/lib/chk.c
index fd9e6015faa..240cc1db57e 100644
--- a/gcc/testsuite/gcc.c-torture/execute/builtins/lib/chk.c
+++ b/gcc/testsuite/gcc.c-torture/execute/builtins/lib/chk.c
@@ -517,3 +517,14 @@ vsnprintf (char *str, __SIZE_TYPE__ len, const char *fmt, va_list ap)
   return ret;
 }
 #endif
+
+#if defined(__powerpc__) && defined(__LONG_DOUBLE_IEEE128__)
+__typeof (__sprintf_chk) __sprintf_chkieee128
+  __attribute__((alias ("__sprintf_chk")));
+__typeof (__vsprintf_chk) __vsprintf_chkieee128
+  __attribute__((alias ("__vsprintf_chk")));
+__typeof (__snprintf_chk) __snprintf_chkieee128
+  __attribute__((alias ("__snprintf_chk")));
+__typeof (__vsnprintf_chk) __vsnprintf_chkieee128
+  __attribute__((alias ("__vsnprintf_chk")));
+#endif


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-02-07 16:40 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-07 16:40 [gcc r12-7084] testsuite: Fix up testsuite/gcc.c-torture/execute/builtins/lib/chk.c for powerpc [PR104380] Jakub Jelinek

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).