[Bug symtab/11992] Regression: C++ this scope sometimes does not work

[Bug symtab/11992] Regression: C++ this scope sometimes does not work

[tromey at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=11992

Tom Tromey <tromey at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
                 CC|                            |tromey at redhat dot com
         Resolution|                            |FIXED

--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> 2010-09-13 20:41:29 UTC ---
Subject: Bug 11992

CVSROOT:	/cvs/src
Module name:	src
Changes by:	swagiaal@sourceware.org	2010-09-13 20:41:06

Modified files:
	gdb            : ChangeLog c-exp.y 
	gdb/testsuite  : ChangeLog 
	gdb/testsuite/gdb.cp: koenig.cc koenig.exp 

Log message:
	Fix PR 11992: C++ 'this' scope sometimes does not work
	
	2010-09-13  Sami Wagiaalla  <swagiaal@redhat.com>
	
	PR symtab/11992:
	* c-exp.y (classify_name): Check is_a_member_of_this before returning
	UNKNOWN_CPP_NAME.
	
	2010-09-13  Sami Wagiaalla  <swagiaal@redhat.com>
	
	* gdb.cp/koenig.cc: created class for testing member lookup.
	* gdb.cp/koenig.exp: Added test for member lookup.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/ChangeLog.diff?cvsroot=src&r1=1.12178&r2=1.12179
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/c-exp.y.diff?cvsroot=src&r1=1.77&r2=1.78
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/ChangeLog.diff?cvsroot=src&r1=1.2446&r2=1.2447
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.cp/koenig.cc.diff?cvsroot=src&r1=1.3&r2=1.4
http://sourceware.org/cgi-bin/cvsweb.cgi/src/gdb/testsuite/gdb.cp/koenig.exp.diff?cvsroot=src&r1=1.4&r2=1.5



--- Comment #4 from Tom Tromey <tromey at redhat dot com> 2011-06-10 15:21:26 UTC ---
Fix checked in a while ago.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug symtab/11992] Regression: C++ this scope sometimes does not work

[swagiaal at redhat dot com]

------- Additional Comments From swagiaal at redhat dot com  2010-09-08 16:54 -------
Thanks for the triage Jan. I'll take a look at this


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at sourceware dot|swagiaal at redhat dot com
                   |org                         |
             Status|NEW                         |ASSIGNED


http://sourceware.org/bugzilla/show_bug.cgi?id=11992

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug symtab/11992] Regression: C++ this scope sometimes does not work

[jan dot kratochvil at redhat dot com]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dodji at redhat dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=11992

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug symtab/11992] Regression: C++ this scope sometimes does not work

[jan dot kratochvil at redhat dot com]

------- Additional Comments From jan dot kratochvil at redhat dot com  2010-09-08 12:57 -------
3c30632200b0021f6fd6673553c3d124de629f68 is the first bad commit
commit 3c30632200b0021f6fd6673553c3d124de629f68
Author: swagiaal <swagiaal>
Date:   Fri May 7 14:46:24 2010 +0000

    Add ADL support
    
    2010-05-07  Sami Wagiaalla  <swagiaal@redhat.com>
    
    	PR C++/7943:
    	* valops.c (find_overload_match): Handle fsym == NULL case.
    	Add int no_adl argument.
    	(find_oload_champ_namespace_loop): Call make_symbol_overload_list_adl
    	when appropriate.
    	Add int no_adl argument.
    	(find_oload_champ_namespace): 	Add int no_adl argument.
    	* parse.c (operator_length_standard): Return length for OP_ADL_FUNC
    	expression.
    	* expprint.c (op_name_standard): Added string for OP_ADL_FUNC case.
    	* eval.c (evaluate_subexp_standard): Added OP_ADL_FUNC case.
    	Evaluate arguments and use them to perform ADL lookup.
    	Pass no_adl argument to find_overload_match.
    	Disable adl lookup when evaluating a fully qualified OP_FUNCALL.
    	* cp-support.h: Added prototype for
    	make_symbol_overload_list_namespace.
    	* cp-support.c (make_symbol_overload_list_namespace): New function.
    	(make_symbol_overload_list_adl_namespace): New function.
    	(make_symbol_overload_list_adl): New function.
    	(make_symbol_overload_list_using): Moved code to add function to
    	overload set to make_symbol_overload_list_namespace.
    	* c-exp.y: create UNKNOWN_CPP_NAME token.
    	Add parse rule for ADL functions.
    	(classify_name): Recognize an UNKNOWN_CPP_NAME.
    
    2010-05-07  Sami Wagiaalla  <swagiaal@redhat.com>
    
    	* gdb.cp/koenig.exp: New test.
    	* gdb.cp/koenig.cc: New test program.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |swagiaal at redhat dot com
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1


http://sourceware.org/bugzilla/show_bug.cgi?id=11992

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.