Re: configure.in support for FreeBSD ia64/sparc64/powerpc

[Dan Nicolaescu <dann@ics.uci.edu>]

Giorgos Keramidas <keramida@freebsd.org> writes:

  > On Mon, 29 Dec 2008 12:39:36 -0800 (PST), Dan Nicolaescu <dann@ics.uci.edu> wrote:
  > >Ulrich Mueller <ulm@gentoo.org> writes:
  > >>> We know for sure that the original code works.
  > >>
  > >> It doesn't work on FreeBSD.
  > >
  > > You were proposing to change the Solaris code.
  > 
  > Err, I think there's a misunderstanding here.  The src/alloc.c change
  > is related to sparc64 not Solaris (not _only_ Solaris, that is):

  >     --- a/src/alloc.c	Fri Oct 27 15:45:51 2006 +0000
  >     +++ b/src/alloc.c	Sat Oct 28 01:49:45 2006 +0300
  >     @@ -4560,7 +4560,11 @@ mark_stack ()
  >           needed on ia64 too.  See mach_dep.c, where it also says inline
  >           assembler doesn't work with relevant proprietary compilers.
  >     */
  >      #ifdef sparc
  >     +#ifdef __sparc64__
  >     +  asm ("flushw");
  >     +#else
  >        asm ("ta 3");
  >     +#endif
  >      #endif
  > 
  > Now, I don't think anybody would argue that there are probably far more
  > Solaris installations of sparc64 than FreeBSD ones.

Exactly, that code has been running for all other systems just fine for
many years.  Why the insistence to change without proper testing? 
Why not just make the conditional
 defined (__sparc64__) && defined (__FreeBSD__) ?