[Glide-devel] Re: Glide3 asm

[Daniel B. <db...@3d...>]

Hans de Goede writes: 

>> C code knows that "gc" is passed in "edx".  Whew! 
>> 
> 
> Ok, sorry about the harrasment. I want to move the Fedora Glide3 package 
> to CVs once all the nescesarry patches in this package are intergrated 
> into CVS and I don't want any bug reports because of this.

Well, that depends.  If the compiler decides to optimize the following
   __asm(""::"d"(gc));
by transforming it into a NOP, we're in big trouble...  Either we can
force EDX to contain GC or change the hoopti shtuff.  Not sure how smart
the compiler gets about ASM statements, though... 

> I'm not planning on changing it this was just an observation I wanted to 
> share, Glide3 is full of deadcode because of using defines which are 
> always true (GLIDE3, GLIDE3_ALPHA). A cleanup would be nice but I don't 
> have the time for this.

Yeah... 

> I've attched a file with a list of files I want to remove because they are 
> no longer used. can you take a look at this?

Do not remove the MinGW makefiles (*.mgw).
Also, removing all the files from ASIC/lib directories can be problematic.
The CVS usually prunes empty directories on checkout, and the build process
will fail (unless we modify the makefiles to re-create those dirs). 

Regards,
Daniel Borca