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.
This commit is contained in:
Alan Modra 2017-02-16 22:19:10 +10:30
parent a8c75b765e
commit 37d7d56cae
4 changed files with 12 additions and 5 deletions

View File

@ -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

View File

@ -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 ();
}

View File

@ -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)

View File

@ -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))