Re: [AD] CPU detection for Macintosh

[Chris <chris.kcat@xxxxxxxxxx>]

Evert Glebbeek wrote:
> I've attached a slightly reworked version of Pete's original patch. Most of 
> the changes are are in the documentation, although I took the liberty of 
> setting the CPU_FPU flag in the cpu_capabilities.

> +   Contains the CPU vendor name, if known. On Mac OSX systems this returns
> +   the PPC subtype name. On other platforms, this may be an empty string.
> +   You can read this variable after you have called check_cpu()
>     (which is automatically called by allegro_init()).

Perhaps this should say "On other, non-Intel, platforms" to remove 
ambiguity? Also, the line width seems to be off on the last new line.

> +    CPU_FAMILY_PENTIUM  - The processor is a Pentium or equivalent
> +    CPU_FAMILY_PPRO     - The processor is a Pentium Pro, II, III or equivalent

I think these might be a little deceiving. Perhaps they should be 
changed to CPU_FAMILY_I586 and CPU_FAMILY_I686 respectively, to keep in 
line with the CPU_FAMILY_I386 and CPU_FAMILY_I486 names, and since that 
is what those are, AFAIK?

> +   CPU_FAMILY_POWERPC:
> +      CPU_MODEL_POWERPC_x, for x=601-604, 620, 750, 7400, 7450</endblock>
> +   You can read this variable after you have called check_cpu() (which is 
> +   automatically called by allegro_init()). Make sure you check teh cpu_family
> +   and cpu_vendor so you know which models make sense to check.

*the

>        CPU_IA64     - Running on Intel 64 bit CPU
>        CPU_AMD64    - Running on AMD 64 bit CPU

I don't think we need these since we can already tell these things from 
the cpu family/model.