On Fri, 2009-08-07 at 14:01 +0200, Vladimir 'phcoder' Serbinenko wrote: > Apple's compiler is based GCC but binutils aren't and they pose the > most of problems. Actualy the most problematic bit was that I didn't > know that unless you prefix variable with L_ apple's assembler treats > it as global. Adding L_ in this code solves problem. Yves proposed to > add a LOCAL(x) macro to symbol.h this way the assembler code is also > more readable. I also did further adjustments in this patch to > decrease the number of ifdefs but had no time to test it and so > couldn't submit it
I would prefer if you don't commit any patches that introduce any more preprocessor conditionals for the Apple compiler or assembler. It would make it hard to debug problems reported by others, as different compilers would produce different binaries. If we are using some feature that is just nice but not needed (such as binary & in lnxboot.S to set the image size to 1024), it would be better to avoid using that feature for all compilers rather than for the compilers that don't support it. -- Regards, Pavel Roskin _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org http://lists.gnu.org/mailman/listinfo/grub-devel
