Stelian Pop wrote:
> Tested, works ok.
thanks.
> Slighly different patch attached.
>
> Linus, please apply.
um, please don't...
> ===== arch/i386/Makefile 1.44 vs edited =====
> --- 1.44/arch/i386/Makefile Fri Feb 7 19:59:54 2003
> +++ edited/arch/i386/Makefile Tue Feb 18 09:59:16 2003
> @@ -39,12 +39,13 @@
> cflags-$(CONFIG_MK6) += $(call check_gcc,-march=k6,-march=i586)
> cflags-$(CONFIG_MK7) += $(call check_gcc,-march=athlon,-march=i686 -malign-functions=4)
> cflags-$(CONFIG_MK8) += $(call check_gcc,-march=k8,$(call check_gcc,-march=athlon,-march=i686 -malign-functions=4))
> -cflags-$(CONFIG_MCRUSOE) += -march=i686 -malign-functions=0 -malign-jumps=0 -malign-loops=0
> +cflags-$(CONFIG_MCRUSOE) += -march=i686
> cflags-$(CONFIG_MWINCHIPC6) += $(call check_gcc,-march=winchip-c6,-march=i586)
> cflags-$(CONFIG_MWINCHIP2) += $(call check_gcc,-march=winchip2,-march=i586)
> cflags-$(CONFIG_MWINCHIP3D) += -march=i586
> cflags-$(CONFIG_MCYRIXIII) += $(call check_gcc,-march=c3,-march=i486)
> # The alignment flags change with gcc 3.2
> +cflags-$(CONFIG_MCRUSOE) += $(call check_gcc,-falign-functions=0 -falign-jumps=0 -falign-loops=0,-malign-functions=0 -malign-jumps=0 -malign-loops=0)
> cflags-$(CONFIG_MCYRIXIII) += $(call check_gcc,-falign-functions=0 -falign-jumps=0 -falign-loops=0,-malign-functions=0 -malign-jumps=0 -malign-loops=0)
> cflags-$(CONFIG_MVIAC3_2) += $(call check_gcc,-march=c3-2,-march=i686)
...because
(a) the Crusoe cflags are no longer together,
(b) Cyrix-3 cflags are suddenly bisected by Crusoe cflags, and
(c) you just dumped a crusoe-specific line in the middle of a bunch of
Via/Cyrix CPU settings.
None of those choices makes sense. If you want to improve upon my
patch, I would suggest consolidating the -f/-m check above the CPU
cflags section, and then reference the calculated value, rather than
wholly duplicating the calculation. The comment line "# The alignment
flags [...]" is obviously not the start of a new alignment-flags section.
Jeff
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Feb 23 2003 - 22:00:21 EST