From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21730 invoked by alias); 20 Jul 2015 07:11:24 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 21712 invoked by uid 89); 20 Jul 2015 07:11:23 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-wi0-f174.google.com Received: from mail-wi0-f174.google.com (HELO mail-wi0-f174.google.com) (209.85.212.174) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Mon, 20 Jul 2015 07:11:22 +0000 Received: by wibxm9 with SMTP id xm9so83515793wib.0 for ; Mon, 20 Jul 2015 00:11:19 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.180.188.139 with SMTP id ga11mr18667838wic.87.1437376279360; Mon, 20 Jul 2015 00:11:19 -0700 (PDT) Received: by 10.27.11.33 with HTTP; Mon, 20 Jul 2015 00:11:19 -0700 (PDT) In-Reply-To: References: Date: Mon, 20 Jul 2015 07:11:00 -0000 Message-ID: Subject: Re: [PATCH] D: support reading modules from DWARF From: Iain Buclaw To: Doug Evans Cc: gdb-patches@sourceware.org Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-07/txt/msg00548.txt.bz2 On 18 July 2015 at 20:01, Doug Evans wrote: > Iain Buclaw writes: >> Hi, >> >> D uses modules (DW_TAG_module), but to separate the namespace of every >> source file. Modules can be imported into each other, either publicly >> or privately (DW_TAG_imported_module). Or declarations can be >> selectively imported or renamed (DW_TAG_imported_decl). >> >> This patch pretty much just extends the existing support for >> namespaces/modules in C++/Fortran/Java to include language_d too. >> However unlike Fortran/C++, the separator for qualified names is a >> single dot. >> >> This will need to be followed up with a patch to support looking up >> symbols in D module 'namespaces'. However I'm currently unsure >> whether to either extend cp-namespace.c, or to go ahead with my >> current fork (d-namespace.c), which copies only what's needed, >> adjusting for D-specific symbol import logic. > > Hi. > > Re: cp-namespace.c: > Hard to say without seeing the patch, > but having spent a lot of time cleaning up cp-namespace.c > I'm not fond of complicating it again to handle more languages. > > So, and again I'd have to see the patch to be sure, > I'd say plan on going with d-namespace.c. Thanks, I see you have found the other patch I was referring to. Assuming that you are intimate with cp-namespace.c, I might have to pick your brains for a couple of things that should be possible, but are currently not. However with the current homework I've done so far, I suspect that I'll have to request an amendment for inclusion in the next DWARFv5 specification first before anything else. Regards Iain.