Fixed bug #15 SDL_blit_A.mmx-speed.patch.txt -- Speed improvements and a bugfix for the current GCC inline mmx asm code: - Changed some ops and removed some resulting useless ones. - Added some instruction parallelism (some gain) The resulting speed on my Xeon improved upto 35% depending on the function (measured in fps). - Fixed a bug where BlitRGBtoRGBSurfaceAlphaMMX() was setting the alpha component on the destination surfaces (to opaque-alpha) even when the surface had none. SDL_blit_A.mmx-msvc.patch.txt -- MSVC mmx intrinsics version of the same GCC asm code. MSVC compiler tries to parallelize the code and to avoid register stalls, but does not always do a very good job. Per-surface blending MSVC functions run quite a bit faster than their pure-asm counterparts (upto 55% faster for 16bit ones), but the per-pixel blending runs somewhat slower than asm. - BlitRGBtoRGBSurfaceAlphaMMX and BlitRGBtoRGBPixelAlphaMMX (and all variants) can now also handle formats other than (A)RGB8888. Formats like RGBA8888 and some quite exotic ones are allowed -- like RAGB8888, or actually anything having channels aligned on 8bit boundary and full 8bit alpha (for per-pixel alpha blending). The performance cost of this change is virtually 0 for per-surface alpha blending (no extra ops inside the loop) and a single non-MMX op inside the loop for per-pixel blending. In testing, the per-pixel alpha blending takes a ~2% performance hit, but it still runs much faster than the current code in CVS. If necessary, a separate function with this functionality can be made. This code requires Processor Pack for VC6.

+
−
/* Include the SDL main definition header */+
−
#include "SDL_main.h"+
−
#include <stdlib.h>+
−
#include <unistd.h>+
−
#ifdef main+
−
#undef main+
−
#endif+
−
#ifdef QWS+
−
#include <qpe/qpeapplication.h>+
−
#include <qapplication.h>+
−
#include <qpe/qpeapplication.h>+
−
#include <stdlib.h>+
−
+
−
// Workaround for OPIE to remove taskbar icon. Also fixes+
−
// some issues in Qtopia where there are left-over qcop files in /tmp/.+
−
// I'm guessing this will also clean up the taskbar in the Sharp version+
−
// of Qtopia.+
−
static inline void cleanupQCop() {+
−
  QString appname(qApp->argv()[0]);+
−
  int slash = appname.findRev("/");+
−
  if(slash != -1) {  appname = appname.mid(slash+1); }+
−
  QString cmd = QPEApplication::qpeDir() + "bin/qcop QPE/System 'closing(QString)' '"+appname+"'";+
−
  system(cmd.latin1());+
−
  cmd = "/tmp/qcop-msg-"+appname;+
−
  unlink(cmd.latin1());+
−
}+
−
+
−
static QPEApplication *app;+
−
#endif+
−
+
−
extern int SDL_main(int argc, char *argv[]);+
−
+
−
int main(int argc, char *argv[])+
−
{+
−
#ifdef QWS+
−
  // This initializes the Qtopia application. It needs to be done here+
−
  // because it parses command line options.+
−
  app = new QPEApplication(argc, argv);+
−
  QWidget dummy;+
−
  app->showMainWidget(&dummy);+
−
  atexit(cleanupQCop);+
−
#endif+
−
  // Exit here because if return is used, the application+
−
  // doesn't seem to quit correctly.+
−
  exit(SDL_main(argc, argv));+
−
}+
−