ld testsuite function pointer comparisons vs. hppa

ld/testsuite/ld-elf/check-ptr-eq.c fails for hppa, since function
pointers may point at plabels.  It isn't valid to cast two function
pointers to void* and then compare the void pointers.

	* testsuite/ld-elf/check-ptr-eq.c (check_ptr_eq): Change params
	from void pointers to function pointers.
	* testsuite/ld-elf/pr18718.c: Update to suit.
	* testsuite/ld-elf/pr18720a.c: Update to suit.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 7fdcf3a93b7d567bb49d029a635ca64f9894dfaf..c5c55833aa6bc526189091bee29fecf55126ea35 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,10 @@
+2017-02-16  Alan Modra  <amodra@gmail.com>
+
+       * testsuite/ld-elf/check-ptr-eq.c (check_ptr_eq): Change params
+       from void pointers to function pointers.
+       * testsuite/ld-elf/pr18718.c: Update to suit.
+       * testsuite/ld-elf/pr18720a.c: Update to suit.
+
 2017-02-16  Alan Modra  <amodra@gmail.com>
 
        PR 21000

diff --git a/ld/testsuite/ld-elf/check-ptr-eq.c b/ld/testsuite/ld-elf/check-ptr-eq.c
index 874431fbc1e4eeed7d6796ab0ae745a704b500b7..c1d5ef699452f43a4a808fdbb8dae5f864302899 100644 (file)
--- a/ld/testsuite/ld-elf/check-ptr-eq.c
+++ b/ld/testsuite/ld-elf/check-ptr-eq.c
@@ -2,11 +2,11 @@ extern void abort (void);
 
 /* Since GCC 5 folds symbol address comparison, assuming each symbol has
    different address,  &foo == &bar is always false for GCC 5.  Use
-   check_ptr_eq to check if 2 addresses are the same.  */
+   check_ptr_eq to check if two functions are the same.  */
 
 void 
-check_ptr_eq (void *p1, void *p2)
+check_ptr_eq (void (*f1) (void), void (*f2) (void))
 {
-  if (p1 != p2)
+  if (f1 != f2)
     abort ();
 }

diff --git a/ld/testsuite/ld-elf/pr18718.c b/ld/testsuite/ld-elf/pr18718.c
index 9cf15e8c054010f135bf29449879de8ae6e9c4a0..426f46a42cb27a48d14541b5e5aefb661c6f0b21 100644 (file)
--- a/ld/testsuite/ld-elf/pr18718.c
+++ b/ld/testsuite/ld-elf/pr18718.c
@@ -2,7 +2,7 @@
 #include <bfd_stdint.h>
 
 extern void foo (void);
-extern void check_ptr_eq (void *, void *);
+extern void check_ptr_eq (void (*) (void), void (*) (void));
 
 void
 new_foo (void)

diff --git a/ld/testsuite/ld-elf/pr18720a.c b/ld/testsuite/ld-elf/pr18720a.c
index 752623be06cf79b9306931f2d55a485adc145a5f..b4a33696c6dc13584c39d1651e909b4dfeb3341e 100644 (file)
--- a/ld/testsuite/ld-elf/pr18720a.c
+++ b/ld/testsuite/ld-elf/pr18720a.c
@@ -3,7 +3,7 @@
 extern void bar (void);
 extern void foo (void);
 extern void foo_alias (void);
-extern void check_ptr_eq (void *, void *);
+extern void check_ptr_eq (void (*) (void), void (*) (void));
 
 #if defined(__GNUC__) && (__GNUC__ * 1000 + __GNUC_MINOR__) >= 4005
 __attribute__ ((noinline, noclone))