Re: pkg/45329 (x11/pixman not compiling on Solaris/sparc)

[Tim Zingelman <tez%netbsd.org@localhost>]

The following reply was made to PR pkg/45329; it has been noted by GNATS.

From: Tim Zingelman <tez%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/45329 (x11/pixman not compiling on Solaris/sparc)
Date: Wed, 12 Oct 2011 10:03:09 -0500

 I guess the danger of not having -fvisibility=hidden and not using
 pixman.sym is that ALL symbols in the oixman code will be exposed in
 the shared library, and this may result in name collisions.  The gcc4+
 man page says of this feature: Using this feature can very
 substantially improve linking and load times of shared object
 libraries, produce more optimized code, provide near-perfect API
 export and prevent symbol clashes.
 
 In this case I have no idea how important or not this is, but it is
 probably not a good idea to rip out the support.
 
 Either the pixman_fill_mmx needs to be conditional in pixman.sym or a
 non-mmx processor alternate pixman.sym is needed.
 Not sure the best way to do that... it also looks like configure
 disables MMX for gcc<3.4 so anybody using old gcc will also fail due
 to pixman_fill_mmx in pixman.sym.