Re: [PATCH 12/14] FRV: Generate more useful debug info

[Andi Kleen]

dhowells@xxxxxxxxxx writes:

> diff -uNr /warthog/kernels/linux-2.6.10-rc1-bk10/Makefile linux-2.6.10-rc1-bk10-frv/Makefile
> --- /warthog/kernels/linux-2.6.10-rc1-bk10/Makefile	2004-11-01 11:45:20.000000000 +0000
> +++ linux-2.6.10-rc1-bk10-frv/Makefile	2004-11-01 11:48:36.397037723 +0000
> @@ -497,11 +497,18 @@
>  # Defaults vmlinux but it is usually overriden in the arch makefile
>  all: vmlinux
>  
> +
> +ifdef CONFIG_DEBUG_INFO
> +CFLAGS		+= -g -O1

Please don't do that. At least on i386/x86-64 we want the same
code with debug info as without. Otherwise how would you debug
a problem that only shows up at -O2. 

-Andi

P.S.: And it's quite unfriendly to put subscriber post only
mailing lists into cc of linux kernel threads. Dropped.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/