Always load function pointer into a stack variable

This patch makes sure that compiler won't optimize out loading function
into a stack variable.

	* ld-ifunc/ifunc-main.c (get_bar): New function.
	(main): Use it.

diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index 555d6bf..63f145e 100644 (file)
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,5 +1,10 @@
 2014-11-20  H.J. Lu  <hongjiu.lu@intel.com>
 
+       * ld-ifunc/ifunc-main.c (get_bar): New function.
+       (main): Use it.
+
+2014-11-20  H.J. Lu  <hongjiu.lu@intel.com>
+
        * ld-ifunc/ifunc.exp: Run ifunc-main.
        * ld-ifunc/ifunc-lib.c: New file.
        * ld-ifunc/ifunc-main.c: Likewise.

diff --git a/ld/testsuite/ld-ifunc/ifunc-main.c b/ld/testsuite/ld-ifunc/ifunc-main.c
index a320cfb..61e9934 100644 (file)
--- a/ld/testsuite/ld-ifunc/ifunc-main.c
+++ b/ld/testsuite/ld-ifunc/ifunc-main.c
@@ -3,12 +3,21 @@
 extern int foo(void);
 extern int bar(void);
 
-int (*foo_ptr)(void) = foo;
+typedef int (*func_p) (void);
+
+func_p foo_ptr = foo;
+
+func_p
+__attribute__((noinline))
+get_bar (void)
+{
+  return bar;
+}
 
 int
 main (void)
 {
-  int (*bar_ptr)(void) = bar;
+  func_p bar_ptr = get_bar ();
   if (bar_ptr != bar)
     __builtin_abort ();
   if (bar_ptr() != -1)