killing ASM_IDENTIFY_GCC [branch]

[Daniel Berlin]

Mark Mitchell <mark@codesourcery.com> writes:

> I'm not qualified to review this patch since there are too many funky
> targets I don't know about.

This patch will break STABS in gdb.

For one thing, it appears that line numbers for symbols will stop
working completely (don't even ask, actually noticing this made me
want to throw up).  Local symbols look like they'll be borked in some
says, as do a few other things.

There is a lot of annoying stuff in processing STABS in gdb.  I could work
around most of it by saying "if we aren't on a sun, and we have STABS
debug info, we are processing gcc compiled stuff" (since it's sun's dbx stuff that
all these problems/workarounds are implemented for). However, I can't
do this unless the sun ones are moved to DWARF2 as the default with
this change as well (so that stabs no longer occurs, and stabs that
does occur will have the symbol, or be sun compiled stuff).
Unless Sun's compiler also outputs gcc2_compiled these days, i don't know.

> 
> However, I support the basic idea of the patch, and would like to see
> it on the branch and the mainline.  I seem to remember that Jim
> expressed interest in this topic, so I'm hoping he will be able to
> review it.  (Anyone else is welcome to do as well.)
> 
> Once you get a review, please install on the branch and on the
> mainline.
> 
> And thank you for taking care of this chore.
> 
> --
> Mark Mitchell                   mark@codesourcery.com
> CodeSourcery, LLC               http://www.codesourcery.com

-- 
I stayed up all night playing poker with Tarot cards.  I got a
full house and four people died.