[quake3-bugzilla] [Bug 4977] x86 qvm improvements

bugzilla-daemon at icculus.org bugzilla-daemon at icculus.org
Tue May 10 03:35:53 EDT 2011


Eugene C. <mine at ukr.net> changed:

           What    |Removed                     |Added
   Attachment #2697|0                           |1
        is obsolete|                            |

--- Comment #11 from Eugene C. <mine at ukr.net> 2011-05-10 03:35:50 EDT ---
Created attachment 2701
  --> https://bugzilla.icculus.org/attachment.cgi?id=2701
x86 qvm improvements

Added more correct and effective const jump optimization

As I can see stack pointer can be modified even often than actual data
reads/writes so its better to put checks for load/store/arg instructions IMO

Also found that optimizer is very sensible to jused[] array - my fixed version
already rejected few load/store const-optimizations (because of jump labels)
that was possible with untouched code 

So I'm thinking - maybe its better to disable optimizations completely for
vq3-qvms? VM_MAGIC_VER2 sure fixing that issue but it
- it doesn't help existing mods (where source code is often unavailable)
- it breaks (iirc) Q3 SDK license so mod developers must release sources (which
may unacceptable due to many reasons)
- modern CPUs is fast enough to handle that :)

Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

More information about the quake3-bugzilla mailing list