[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
> I think the point was to allow a platform to define asm version of
> some routines, and have fallback versions.
This works for platforms that have asm versions of the routines and
explicitly disable them by defining ALLEGRO_NO_ASM very early. Now for
platforms that don't have asm versions, asm.inl is clever enough to
automatically define ALLEGRO_NO_ASM. The new layout breaks this mechanism.
> If putting everything in the (platform independant) .inl files, that
> will mean it contains a lot of #ifdef THAT_PLATFORM stuff, which is
> not very nice...
No. If you take a look at asm.inl, you'll see that it is only a wrapper for
the platform-dependent asm file, which are located in the platform/
directory.
IMHO gfxasm.inl and mathasm.inl are useless because you can't include them
without including *each time* gfx.inl and fmaths.inl (respectively) before.
This oversight broke the compilation under BC++.
--
Eric Botcazou
ebotcazou@xxxxxxxxxx