[ARM][Testsuite]make dump_stack function call conditional in cold-lc.c
authorRenlin Li <renlin.li@arm.com>
Tue, 13 Jun 2017 13:57:45 +0000 (13:57 +0000)
committerRenlin Li <renlin@gcc.gnu.org>
Tue, 13 Jun 2017 13:57:45 +0000 (13:57 +0000)
After r249013, die () and dump_stack () are both in cold section. This makes
the compiler generate bl instruction for the function call, instead of
honoring the -mlong-calls option.

This patch changes the dump_stack function call conditional, which fixes the
regression.

gcc/testsuite/

* gcc.target/arm/cold-lc.c: Update coding style, call dump_stack
conditionally.

From-SVN: r249157

gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/arm/cold-lc.c

index d9a9fb2fa654e2e62dd1a6321c33f172c2eb8c38..6b33a67c08c13963633811f3b0f5355411203bb4 100644 (file)
@@ -1,3 +1,8 @@
+2017-06-13  Renlin Li  <renlin.li@arm.com>
+
+       * gcc.target/arm/cold-lc.c: Update coding style, call dump_stack
+       conditionally.
+
 2017-06-13  Martin Jambor  <mjambor@suse.cz>
 
        PR tree-optimization/80803
index 467a6966bd3bda54ae9929ec0b674c6849606c3d..f0cd6df34682401bebbb373280d013a0a7c11dff 100644 (file)
@@ -11,13 +11,14 @@ extern int show_stack (struct task_struct *, unsigned long *);
 
 void dump_stack (void)
 {
-    unsigned long stack;
-    show_stack ((current_thread_info ()->task), &stack);
+  unsigned long stack;
+  show_stack ((current_thread_info ()->task), &stack);
 }
 
 void die (char *str, void *fp, int nr)
 {
+  if (nr)
     dump_stack ();
-    while (1);
+  while (1);
 }