Re: [PATCH] Add support for Analog Devices Blackfin processor (part 3/6: gdb testsuite)

Jie Zhang
2006-01-04T06:41:30Z

Hi Jim,

Thanks a lot for reviewing these patches!

On 1/4/06, Jim Blandy <jimb@red-bean.com> wrote:
> On 12/27/05, Jie Zhang <jzhang918@gmail.com> wrote:
> + set asm-flags "-I${srcdir}/${subdir} -I${objdir}/${subdir}"
>
> What's the story behind these -I flags that seem to be sprouting up in
> a lot of the cases in asm-source.exp? It seems like they should
> either be necessary everywhere, or nowhere.
>
This is the easiest question for me, so I answer it first.

In asm-source.exp, first comes:

set asm-flags ""

After all target things, there is:

if { "${asm-flags}" == "" } {
set asm-flags "-I${srcdir}/${subdir} -I${objdir}/${subdir}"
set debug-flags "-gstabs"
}

So if target does not set asm-flags, the debug-flags will be set to
"-gstabs", which will override anything target set for it. If any
target want to use dwarf2 as debug information, like bfin and others,
it needs set asm-flags to the vanilla string to avoid the overriding.

Regards,
Jie