use __builtin_alloca, drop non-standard alloca.h
authorAlexandre Oliva <oliva@adacore.com>
Fri, 16 Aug 2019 03:38:36 +0000 (03:38 +0000)
committerAlexandre Oliva <aoliva@gcc.gnu.org>
Fri, 16 Aug 2019 03:38:36 +0000 (03:38 +0000)
Since alloca.h is not ISO C, most of our alloca-using tests seem to
rely on __builtin_alloca instead of including the header and calling
alloca.  This patch extends this practice to some of the exceptions I
found in gcc.target, marking them as requiring a functional alloca
while at that.

for  gcc/testsuite/ChangeLog

* gcc.target/arc/interrupt-6.c: Use __builtin_alloca, require
effective target support for alloca, drop include of alloca.h.
* gcc.target/i386/pr80969-3.c: Likewise.
* gcc.target/sparc/setjmp-1.c: Likewise.
* gcc.target/x86_64/abi/ms-sysv/gen.cc: Likewise.
* gcc.target/x86_64/abi/ms-sysv/ms-sysv.c: Likewise.

From-SVN: r274558

gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/arc/interrupt-6.c
gcc/testsuite/gcc.target/i386/pr80969-3.c
gcc/testsuite/gcc.target/sparc/setjmp-1.c
gcc/testsuite/gcc.target/x86_64/abi/ms-sysv/gen.cc
gcc/testsuite/gcc.target/x86_64/abi/ms-sysv/ms-sysv.c

index 55c59822dde4087e6057a6fe0d21074903034547..055ff045e0fc931d6851e6a040f0a58d35d109ce 100644 (file)
@@ -1,5 +1,12 @@
 2019-08-16  Alexandre Oliva <oliva@adacore.com>
 
+       * gcc.target/arc/interrupt-6.c: Use __builtin_alloca, require
+       effective target support for alloca, drop include of alloca.h.
+       * gcc.target/i386/pr80969-3.c: Likewise.
+       * gcc.target/sparc/setjmp-1.c: Likewise.
+       * gcc.target/x86_64/abi/ms-sysv/gen.cc: Likewise.
+       * gcc.target/x86_64/abi/ms-sysv/ms-sysv.c: Likewise.
+
        * gcc.misc-tests/options.exp: Match /ld and -ld besides
        /collect2.
 
index d82bd67edd83448f9e7dc3b37520760b9985139e..9cb0565f55c9abf09c79d6f2bfee7c6038ca811a 100644 (file)
@@ -1,8 +1,7 @@
 /* { dg-do compile } */
 /* { dg-skip-if "Not available for ARCv1" { arc700 || arc6xx } } */
 /* { dg-options "-O2 -mirq-ctrl-saved=r0-ilink" } */
-
-#include <alloca.h>
+/* { dg-require-effective-target alloca } */
 
 /* Check if ilink is recognized. Check how FP and BLINK are saved.
    BLINK is saved last on the stack because the IRQ autosave will do
@@ -14,7 +13,7 @@ extern int bar (void *);
 void  __attribute__ ((interrupt("ilink")))
 foo(void)
 {
-  int *p = alloca (10);
+  int *p = __builtin_alloca (10);
   bar (p);
 }
 /* { dg-final { scan-assembler-not ".*fp,\\\[sp" } } */
index d902a771cc8f015815ec19b87c447c021a9f46cf..318e06cd94c6cd14d72dfff57582b74acd271491 100644 (file)
@@ -2,11 +2,10 @@
 /* { dg-do compile { target { { ! x32 } && { ! avx512f_runtime } } } } */
 /* { dg-options "-Ofast -mabi=ms -mavx512f" } */
 /* { dg-require-effective-target avx512f } */
+/* { dg-require-effective-target alloca } */
 
 /* Test with alloca (and DRAP).  */
 
-#include <alloca.h>
-
 int a[56];
 volatile int b = -12345;
 volatile const int d = 42;
@@ -19,7 +18,7 @@ void (*volatile const foo_noinfo)(int *, int, int) = foo;
 
 int main (int argc, char *argv[]) {
   int c;
-  int *e = alloca (d);
+  int *e = __builtin_alloca (d);
   foo_noinfo (e, d, 0);
   for (; b; b++) {
     c = b;
index d0fecb3632707798cb98b824015350c18ee732ff..699d7f7b8ff458843d5f5f01971b23ec62651c25 100644 (file)
@@ -4,9 +4,9 @@
 /* { dg-do run { target *-*-solaris2.* *-*-linux* *-*-*bsd* } } */\r
 /* { dg-require-effective-target fpic } */\r
 /* { dg-options "-fPIC" } */\r
+/* { dg-require-effective-target alloca } */\r
 \r
 #include <stdio.h>\r
-#include <alloca.h>\r
 #include <setjmp.h>\r
 #include <string.h>\r
 #include <stdlib.h>\r
@@ -26,7 +26,7 @@ int main (void)
 {\r
   setjmp (jb);\r
 \r
-  char *p = alloca (256);\r
+  char *p = __builtin_alloca (256);\r
   memset (p, 0, 256);\r
   sprintf (p, "%d\n", foo);\r
 \r
index 701531480f629e084756031442117479683e5ea8..818a8875a6dd79d82822f52474a54fdf6baf14fd 100644 (file)
@@ -291,7 +291,7 @@ void fn::print_def (ostream &out) const
   if (get_msabi () && get_alloca ())
     {
       const char *size_str = m_args.empty () ? "42" : "a";
-      out << "  void *alloca_mem = alloca (8 + " << size_str << ");" << endl
+      out << "  void *alloca_mem = __builtin_alloca (8 + " << size_str << ");" << endl
          << "  *(long*)alloca_mem = FLAG_ALLOCA;" << endl;
     }
   if (get_msabi () && get_varargs ())
index 5fdd1e20674b0e5b847961f584969b9d4b2d5816..abfcee6f56a41becc8fb6600a113fda0e8975aab 100644 (file)
@@ -49,6 +49,7 @@ see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
 /* { dg-do run } */
 /* { dg-additional-sources "do-test.S" } */
 /* { dg-additional-options "-Wall" } */
+/* { dg-require-effective-target alloca } */
 
 #include <stdio.h>
 #include <string.h>
@@ -56,7 +57,6 @@ see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
 #include <signal.h>
 #include <unistd.h>
 #include <stdint.h>
-#include <alloca.h>
 #include <stdarg.h>
 #include <assert.h>
 #include <errno.h>