Hi,

libitm.c/stackundo.c fails with -fpic since test1 and test2 may be
preempted with -fpic.  This patch makes those 2 functions static.
Tested on Linux/x86.  OK for trunk?

Thanks.


H.J.
----
diff --git a/libitm/ChangeLog b/libitm/ChangeLog
index 74e2940..e46819c 100644
--- a/libitm/ChangeLog
+++ b/libitm/ChangeLog
@@ -1,3 +1,9 @@
+2015-01-15  H.J. Lu  <hongjiu...@intel.com>
+
+       PR libitm/64360
+       * libitm.c/stackundo.c (test2): Make it static.
+       (test1): Likewise.
+
 2015-01-05  Jakub Jelinek  <ja...@redhat.com>
 
        Update copyright years.
diff --git a/libitm/testsuite/libitm.c/stackundo.c 
b/libitm/testsuite/libitm.c/stackundo.c
index 02759d7..c7d585e 100644
--- a/libitm/testsuite/libitm.c/stackundo.c
+++ b/libitm/testsuite/libitm.c/stackundo.c
@@ -1,10 +1,10 @@
-int __attribute__((noinline)) test2(int x[1000])
+static int __attribute__((noinline)) test2(int x[1000])
 {
   int i;
   return x[12];
 }
 
-int __attribute__((noinline)) test1()
+static int __attribute__((noinline)) test1()
 {
   int x[1000], i;
 

Reply via email to