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;