Re: [PATCH] Pass -g to assembler under CONFIG_DEBUG_INFO

[Andrew Morton]

On Wed, 11 Jul 2007 15:43:27 -0700 (PDT)
Roland McGrath <roland@xxxxxxxxxx> wrote:

> > The -gdwarf2 had a specific reason. 
> 
> I have no objection to it.  
> 

OK, thanks.  So I ended up with the below as a probably-2.6.23 thing:


From: Roland McGrath <roland@xxxxxxxxxx>

The assembler for a while now supports -gdwarf to generate source line info
just like the C compiler does.  Source-level assembly debugging sounds like an
oxymoron, but it is handy to be able to see the right source file and read its
comments rather than just the disassembly.  This patch enables -gdwarf for
assembly files when CONFIG_DEBUG_INFO=y and the assembler supports the option.

Signed-off-by: Roland McGrath <roland@xxxxxxxxxx>
Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---

 Makefile |    1 +
 1 files changed, 1 insertion(+)

diff -puN Makefile~pass-g-to-assembler-under-config_debug_info Makefile
--- a/Makefile~pass-g-to-assembler-under-config_debug_info
+++ a/Makefile
@@ -498,6 +498,7 @@ endif
 
 ifdef CONFIG_DEBUG_INFO
 CFLAGS		+= -g
+AFLAGS		+= $(call as-option, -gdwarf)
 endif
 
 # Force gcc to behave correct even for buggy distributions
_


If there are problems it'll most likely be with oddball architectures
running older compilers, I guess.  If something blows up we might need to
make this an arch/foo/Makefile thing.

-
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/