[PATCH] ld testsuite cleanup for hppa*-*-linux* and hppa64-*-hpux*
Nick Clifton
nickc@redhat.com
Sat Oct 4 09:41:00 GMT 2003
Hi Dave,
> The enclosed patch fixes a number of ld testsuite problems on
> hppa*-*-linux* and hppa64-*-hpux*. As the change to
> ld-elfvsb/main.c affects all ports, it needs review. Any other
> comments are welcome.
> 2003-09-11 John David Anglin <dave.anglin@nrc-cnrc.gc.ca>
>
> * configure.host (HOSTING_LIBS): Define for hppa*-*-linux*.
> * ld-elfvsb/elfvsb.exp: xfail non-pic shared library tests on
> hppa*-*-linux* and hppa*64*-*-hpux*.
> * ld-elfvsb/main.c (main_visibility_check): Cast value returned by
> visibility_funptr () to a function pointer.
> * ld-shared/shared.exp: xfail shared (non PIC), shared (non PIC, load
> offset), and shared (PIC main, non PIC so) tests on hppa*64*-*-hpux*
> and hppa*-*-linux*.
Approved - please apply.
> Index: testsuite/ld-elfvsb/main.c
> static int
> main_visibility_check ()
> {
> - return visibility_funptr () != visibility;
> + return (int (*) (void)) visibility_funptr () != visibility;
> }
Given that we are now using ISO-C all of this file could benefir from
being updated. Also with the case above, I would recommend adding
some parentheses to show what is being cast (ie the variable
visibility_funptr and not the result of the comparison).
Cheers
Nick
More information about the Binutils
mailing list