On Mon, 2 Aug 2010, Chris Young wrote: > On Sun, 1 Aug 2010 21:14:12 +0100 (GMT Daylight Time), Jeffrey Lee wrote: > >> If anyone's interested in experimenting on other platforms, the #define >> that controls the use of ARMul_ImmedTable is at the top of armemu.h. > > Not using the table is marginally faster here on PPC. It's probably > best to move that define into the makefile, same as SOUND_SUPPORT etc, > so it can be usefully toggled on a per-platform basis.
At the moment the main function of the makefile is to differentiate between the different OS's that are supported. But you'd want to enable/disable the use of ARMul_ImmedTable on a per-CPU basis, not a per-OS basis. So rather than add a whole mess of extra logic to the makefile (e.g. to try and detect what CPU type the X11 version is be being compiled for) I decided to take the easy route and just add the check to a header. As long as everyone's using sensible compilers that indicate the target machine's CPU type in a standard way then we shouldn't run into any problems. But having said that, I've got no qualms with it being moved to the makefile, or perhaps some new header to deal with per-CPU tweaks, if that's what people want. > Thank you for your hard work on this. No problem. I'm mainly doing it for my own enjoyment, not yours ;) Cheers, - Jeffrey ------------------------------------------------------------------------------ The Palm PDK Hot Apps Program offers developers who use the Plug-In Development Kit to bring their C/C++ apps to Palm for a share of $1 Million in cash or HP Products. Visit us here for more details: http://p.sf.net/sfu/dev2dev-palm _______________________________________________ arcem-devel mailing list arcem-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/arcem-devel
