]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
PR116080: Fix tail call dejagnu checks
authorAndi Kleen <ak@gcc.gnu.org>
Thu, 25 Jul 2024 03:18:56 +0000 (20:18 -0700)
committerThomas Koenig <tkoenig@gcc.gnu.org>
Sun, 28 Jul 2024 17:06:00 +0000 (19:06 +0200)
- Run the target_effective tail_call checks without optimization to
match the actual test cases.
- Add an extra check for external tail calls to handle targets like
powerpc that cannot tail call between different object files.
This one will also cover templates.

gcc/testsuite/ChangeLog:

PR testsuite/116080
* g++.dg/musttail10.C: Use external tail call target check.
* g++.dg/musttail6.C: Dito.
* lib/target-supports.exp: Add external_tail_call. Disable
optimization for tail call checks.

gcc/testsuite/g++.dg/musttail10.C
gcc/testsuite/g++.dg/musttail6.C
gcc/testsuite/lib/target-supports.exp

index ff7fcc7d8755ae879c45c7d56361144fe3232091..bd75affa2220e191020a37dc512259b1826a932b 100644 (file)
@@ -8,7 +8,7 @@ double g() { [[gnu::musttail]] return f<int>(); } /* { dg-error "cannot tail-cal
 
 template <class T>
 __attribute__((noinline, noclone, noipa))
-T g1() { [[gnu::musttail]] return f<T>(); } /* { dg-error "target is not able" "" { target powerpc*-*-* } } */
+T g1() { [[gnu::musttail]] return f<T>(); } /* { dg-error "target is not able" "" { target { external_tail_call } } } */
 
 template <class T>
 __attribute__((noinline, noclone, noipa))
index 5c6f69407ddb6ee1d58d93116317a633e937ecb2..81f6d9f3ca7745d9febca67b6e047e8308b614c5 100644 (file)
@@ -1,6 +1,6 @@
 /* { dg-do compile { target { struct_tail_call } } } */
+/* { dg-require-effective-target external_tail_call } */
 /* A lot of architectures will not build this due to PR115606 and PR115607 */
-/* { dg-skip-if "powerpc does not support sibcall to templates" { powerpc*-*-* } } */
 /* { dg-options "-std=gnu++11" } */
 /* { dg-additional-options "-fdelayed-branch" { target sparc*-*-* } } */
 
index d368251ef9a46bd4e0f22ac4bac2ce7c6394112a..0a3946e82d4b4ebb8b0f426fc0653a80c7e6327e 100644 (file)
@@ -12741,7 +12741,15 @@ proc check_effective_target_tail_call { } {
     return [check_no_messages_and_pattern tail_call ",SIBCALL" rtl-expand {
        __attribute__((__noipa__)) void foo (void) { }
        __attribute__((__noipa__)) void bar (void) { foo(); }
-    } {-O2 -fdump-rtl-expand-all}] ;# The "SIBCALL" note requires a detailed dump.
+    } {-fdump-rtl-expand-all}] ;# The "SIBCALL" note requires a detailed dump.
+}
+
+# Return 1 if the target can perform tail-calls for externals
+proc check_effective_target_external_tail_call { } {
+    return [check_no_messages_and_pattern tail_call ",SIBCALL" rtl-expand {
+       extern __attribute__((__noipa__)) void foo (void);
+       __attribute__((__noipa__)) void bar (void) { foo(); }
+    } {-fdump-rtl-expand-all}] ;# The "SIBCALL" note requires a detailed dump.
 }
 
 # Return 1 if the target can perform tail-call optimizations for structures
@@ -12751,9 +12759,9 @@ proc check_effective_target_struct_tail_call { } {
     return [check_no_messages_and_pattern tail_call ",SIBCALL" rtl-expand {
        // C++
        struct foo { int a, b; };
-       __attribute__((__noipa__)) struct foo foo (void) { return {}; }
+       extern __attribute__((__noipa__)) struct foo foo (void);
        __attribute__((__noipa__)) struct foo bar (void) { return foo(); }
-    } {-O2 -fdump-rtl-expand-all}] ;# The "SIBCALL" note requires a detailed dump.
+    } {-fdump-rtl-expand-all}] ;# The "SIBCALL" note requires a detailed dump.
 }
 
 # Return 1 if the target's calling sequence or its ABI