This is the mail archive of the mailing list for the GDB 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: [testcase patch] 7.2 regression on expand psymtabs

Hi Keith,

original post:

On Sun, 21 Nov 2010 22:30:20 +0100, Jan Kratochvil wrote:
> there is a regression by the patch:
> 	42284fdf9d8cdb20c8e833bdbdb2b56977fea525
> 	dwarf2_physname patchset:
> 	[RFA] dwarf2_physname FINAL
> gdb/testsuite/
> 2010-11-21  Jan Kratochvil  <>
> 	* gdb.cp/expand-psymtabs-cxx.exp: New file.
> 	* gdb.cp/ New file.

I have found this testfile by me went FAIL->PASS by:

Re: [patch 0/3] Re: [RFA] c++/11734 revisited (and c++/12273)

= two commits:

commit 9e2e8e2f9323fe17ceafb49d8eca9e3117947066
Author: Keith Seitz <>
Date:   Wed Mar 16 21:12:11 2011 +0000

        * linespec.c (find_methods): Canonicalize NAME before looking
        up the symbol.
        (name_end): New function.
        (keep_name_info): New function.
        (decode_line_1): Use keep_name_info.
        (decode_compound): Likewise.
        * cli/cli-utils.h (remove_trailing_whitespace): New function.
        * cli/cli-utils.c (remove_trailing_whitespace): Likewise.
        PR c++/12273
        * linespec.c (locate_first_half): Keep overload information, too.
        (decode_compound): Use a string to represent break characters
        to escape the loop.
        If P points to a break character, do not increment it.
        For C++ and Java, keep overload information and relevant keywords.
        If we cannot find a symbol, search the minimal symbols.
        PR c++/11734
        * linespec.c (decode_compound): Rename SAVED_ARG to
        Make a copy of THE_REAL_SAVED_ARG in SAVED_ARG and strip
        Pass a valid block to lookup_symbol.
        (lookup_prefix_sym): Likewise.
        (find_method): Construct search name based on SYM_CLASS instead
        of SAVED_ARG.
        * psymtab.c (lookup_partial_symbol): Add language parameter.
        (lookup_symbol_aux_psymtabs): Likewise.
        Don't assume that the psymtab we found was the right one. Search
        for the desired symbol in the symtab to be certain.
        (psymtab_search_name): New function.
        (lookup_partial_symbol): Use psymtab_search_name.
        Add language parameter.
        (read_symtabs_for_function): Add language parameter and pass to
        (find_symbol_file_from_partial): Likewise.

commit bb108bccb87a6e3b166a2ba74cc3aaf192c687ff
Author: Keith Seitz <>
Date:   Wed Mar 16 21:08:56 2011 +0000

	PR c++/12273
	* gdb.cp/cmpd-minsyms.exp: New test.
	* gdb.cp/ New file.
	PR c++/11734
	* gdb.cp/ovsrch.exp: New test.
	* gdb.cp/ovsrch.h: New file.
	* gdb.cp/ New file.
	* gdb.cp/ New file.
	* gdb.cp/ New file.
	* gdb.cp/ New file.

I think these testcases do not cover the issue tested by this posted testfile.
Therefore going to check it in unless you tell otherwise.  A redundant
testfile would not be such a tragedy anyway.


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