This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] PR debug/10055: fix input_filename clobber


On Sun, 2003-07-06 at 11:39, Michael Elizabeth Chastain wrote:
> This patch fixes PR gdb/10055, a regression from gcc 3.3.
> 
>   http://gcc.gnu.org/bugzilla/show_bug.cgi?id=10055
>   [3.4 regression] g++ emits "<internal>" as source filename with -gstabs+
> 
> The problem happens with g++ -gstabs+.  g++ emits a BINCL stab for
> "<internal>" rather than the name of the input file.  This makes
> debugging more difficult and causes dozens of FAIL results in the gdb
> test suite.
> 
> The problem is with the initialization order of input_filename.
> Here is a sketch of the call tree:

This patch is OK -- but remove the first-person comment.  Just write:

We cannot just assign to input_filename because it has already been
initialized and will be used later as an N_BINCL for stabs+ debugging.

Neil, do you have time to apply this to 3.4 for Michael?

Thanks,

-- 
Mark Mitchell <mark@codesourcery.com>
CodeSourcery, LLC


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]