6Fermer8
Kevin KoflerLe 17/09/2009 à 04:57
Lionel Debroux (./3) :
1) a code correctness fix + size optimization for bsearch() (which has been broken since before the creation of the TIGCC CVS repository in 2004, probably since the routine was integrated in TIGCC),2) a significant size & speed optimization on qsort()

So you converted the C code to assembly and micro-optimized a few instructions in it. I'm not sure I'll be merging that, as it reduces maintainability significantly for very little gain.

In addition, why did you put the assembly code in an asm block in a .c file? Assembly functions should be in .s files!

And finally, I cannot merge your bsearch.c because it uses code under the BSD license which is not compatible with the TIGCCLIB license. (The BSD license requires documentation of binary-only programs to carry the copyright notice for the used code, which is a requirement not in the TIGCCLIB license's GPL exception.)