From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8969 invoked by alias); 11 Jun 2010 14:53:20 -0000 Received: (qmail 8887 invoked by uid 48); 11 Jun 2010 14:53:08 -0000 Date: Fri, 11 Jun 2010 14:53:00 -0000 Message-ID: <20100611145308.8886.qmail@sourceware.org> X-Bugzilla-Reason: CC References: Subject: [Bug debug/44126] wrong location description for DW_AT_vtable_elem_location In-Reply-To: Reply-To: gcc-bugzilla@gcc.gnu.org To: gcc-bugs@gcc.gnu.org From: "tromey at gcc dot gnu dot org" Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2010-06/txt/msg01318.txt.bz2 ------- Comment #4 from tromey at gcc dot gnu dot org 2010-06-11 14:53 ------- I think the problem with this patch is that it leaves gdb no way to determine which approach it should use. This is important because there is a lot of existing code compiled with the incorrect approach. Currently gdb has a heuristic where it uses the incorrect style if it sees DW_AT_containing_type in the virtual function's DIE. This appears to be a GNU extension (you can see it at the end of the patch, in the context -- but there is another instance in dwarf2out.c). Maybe removing that would help. I'm not sure if it is needed and/or useful for some other reason, maybe a little exploration of the history would help. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44126