On Wed, 2003-10-15 at 16:15, Felix Kühling wrote:
> On Wed, 15 Oct 2003 21:53:17 +0200
> Michel Dänzer <[EMAIL PROTECTED]> wrote:
> 
> > On Wed, 2003-10-15 at 18:50, Felix Kühling wrote:
> > > 
> > > Oops, xmlconfig.o gets linked into all drivers, even those that don't
> > > use it. BTW, the same is true for all object files in
> > > xc/lib/GL/mesa/src/drv/common. How should files in the common directory
> > > be handled that are not used by all drivers?
> > 
> > Change the drivers which don't use everything to explicitly link only
> > what they need? Or change common/Imakefile.inc to allow customization
> > via #defines or whatever?
> 
> I looked through the Imakefiles in order to resolve this. I found that
> hwlog.o is not needed at all any more. vblank.o and xmlconfig.o are only
> needed by mga, r128, r200 and radeon so far. The other objects (mm.o,
> texmem.o and utils.o) seem to be used by all drivers. I'm not 100% sure
> about texmem though.
> 
> The attached patch splits COMMONOBJS into COMMONOBJS, VBLANKOBJS and
> XMLCONFIGOBJS so that the individual driver Imakefiles can select those
> objects from the common subdir that they really need. It also removes
> hwlog.o from COMMONOBJS completely. If noone objects I'm going to commit
> this.

The SiS driver doesn't do texmem.  I don't think it needs the texmem
objects, and I don't forsee it using the current incarnation of texmem.
(not an objection, since the patch is an improvement anyway, but if
you're already in the area...)

-- 
Eric Anholt                                [EMAIL PROTECTED]          
http://people.freebsd.org/~anholt/         [EMAIL PROTECTED]




-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
SourceForge.net hosts over 70,000 Open Source Projects.
See the people who have HELPED US provide better services:
Click here: http://sourceforge.net/supporters.php
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to