From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27561 invoked by alias); 6 Nov 2014 10:50:58 -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 27536 invoked by uid 89); 6 Nov 2014 10:50:58 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,KAM_STOCKGEN,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=no version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 06 Nov 2014 10:50:57 +0000 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id sA6AotRs018758 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Thu, 6 Nov 2014 05:50:55 -0500 Received: from blade.nx (ovpn-116-104.ams2.redhat.com [10.36.116.104]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id sA6AossB004630 for ; Thu, 6 Nov 2014 05:50:55 -0500 Received: from blade.nx (localhost [127.0.0.1]) by blade.nx (Postfix) with ESMTP id 356672626A7 for ; Thu, 6 Nov 2014 10:50:54 +0000 (GMT) From: Gary Benson To: gdb-patches@sourceware.org Subject: [PATCH 2/3 v2] Interleave completion list building with symbol table expansion Date: Thu, 06 Nov 2014 10:51:00 -0000 Message-Id: <1415271046-4957-3-git-send-email-gbenson@redhat.com> In-Reply-To: <1415271046-4957-1-git-send-email-gbenson@redhat.com> References: <1415271046-4957-1-git-send-email-gbenson@redhat.com> X-IsSubscribed: yes X-SW-Source: 2014-11/txt/msg00116.txt.bz2 This commit makes default_make_symbol_completion_list_break_on build the list of completions as it expands the necessary symbol tables, rather than expanding all necessary symbol tables first and then building the completion lists second. This allows for the early termination of symbol table expansion if required. gdb/ChangeLog: * symtab.c (struct add_name_data) : New field. Updated comments. (add_symtab_completions): New function. (symtab_expansion_callback): Likewise. (default_make_symbol_completion_list_break_on): Set datum.code. Move minimal symbol scan before calling expand_symtabs_matching. Scan known primary symtabs for externs and statics before calling expand_symtabs_matching. Pass symtab_expansion_callback as expansion_notify argument to expand_symtabs_matching. Do not scan primary symtabs for externs and statics after calling expand_symtabs_matching. --- gdb/ChangeLog | 14 +++++++ gdb/symtab.c | 106 ++++++++++++++++++++++++++++++++++++--------------------- 2 files changed, 81 insertions(+), 39 deletions(-) diff --git a/gdb/symtab.c b/gdb/symtab.c index 836f3bc..ebb7b8f 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -4424,15 +4424,19 @@ completion_list_add_fields (struct symbol *sym, const char *sym_text, } } -/* Type of the user_data argument passed to add_macro_name or - symbol_completion_matcher. The contents are simply whatever is - needed by completion_list_add_name. */ +/* Type of the user_data argument passed to add_macro_name, + symbol_completion_matcher and symtab_expansion_callback. */ + struct add_name_data { + /* Arguments required by completion_list_add_name. */ const char *sym_text; int sym_text_len; const char *text; const char *word; + + /* Extra argument required for add_symtab_completions. */ + enum type_code code; }; /* A callback used with macro_for_each and macro_for_each_in_scope. @@ -4460,6 +4464,52 @@ symbol_completion_matcher (const char *name, void *user_data) return compare_symbol_name (name, datum->sym_text, datum->sym_text_len); } +/* Add matching symbols from SYMTAB to the current completion list. */ + +static void +add_symtab_completions (struct symtab *symtab, + const char *sym_text, int sym_text_len, + const char *text, const char *word, + enum type_code code) +{ + struct symbol *sym; + const struct block *b; + struct block_iterator iter; + int i; + + if (!symtab->primary) + return; + + for (i = GLOBAL_BLOCK; i <= STATIC_BLOCK; i++) + { + QUIT; + b = BLOCKVECTOR_BLOCK (BLOCKVECTOR (symtab), i); + ALL_BLOCK_SYMBOLS (b, iter, sym) + { + if (code == TYPE_CODE_UNDEF + || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN + && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) + COMPLETION_LIST_ADD_SYMBOL (sym, + sym_text, sym_text_len, + text, word); + } + } +} + +/* Callback to add completions to the current list when symbol tables + are expanded during completion list generation. */ + +static void +symtab_expansion_callback (struct symtab *symtab, void *user_data) +{ + struct add_name_data *datum = (struct add_name_data *) user_data; + + add_symtab_completions (symtab, + datum->sym_text, datum->sym_text_len, + datum->text, datum->word, + datum->code); +} + VEC (char_ptr) * default_make_symbol_completion_list_break_on (const char *text, const char *word, @@ -4560,17 +4610,12 @@ default_make_symbol_completion_list_break_on (const char *text, datum.sym_text_len = sym_text_len; datum.text = text; datum.word = word; - - /* Look through the partial symtabs for all symbols which begin - by matching SYM_TEXT. Expand all CUs that you find to the list. - The real names will get added by COMPLETION_LIST_ADD_SYMBOL below. */ - expand_symtabs_matching (NULL, symbol_completion_matcher, NULL, - ALL_DOMAIN, &datum); + datum.code = code; /* At this point scan through the misc symbol vectors and add each symbol you find to the list. Eventually we want to ignore anything that isn't a text symbol (everything else will be - handled by the psymtab code above). */ + handled by the psymtab code below). */ if (code == TYPE_CODE_UNDEF) { @@ -4585,6 +4630,18 @@ default_make_symbol_completion_list_break_on (const char *text, } } + /* Add completions for all currently loaded symbol tables. */ + ALL_PRIMARY_SYMTABS (objfile, s) + add_symtab_completions (s, sym_text, sym_text_len, text, word, code); + + /* Look through the partial symtabs for all symbols which begin + by matching SYM_TEXT. Expand all CUs that you find to the list. + symtab_expansion_callback is called for each expanded symtab, + causing those symtab's completions to be added to the list too. */ + expand_symtabs_matching (NULL, symbol_completion_matcher, + symtab_expansion_callback, ALL_DOMAIN, + &datum); + /* Search upwards from currently selected frame (so that we can complete on local vars). Also catch fields of types defined in this places which match our text string. Only complete on types @@ -4634,35 +4691,6 @@ default_make_symbol_completion_list_break_on (const char *text, completion_list_add_fields (sym, sym_text, sym_text_len, text, word); } - /* Go through the symtabs and check the externs and statics for - symbols which match. */ - - ALL_PRIMARY_SYMTABS (objfile, s) - { - QUIT; - b = BLOCKVECTOR_BLOCK (BLOCKVECTOR (s), GLOBAL_BLOCK); - ALL_BLOCK_SYMBOLS (b, iter, sym) - { - if (code == TYPE_CODE_UNDEF - || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN - && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) - COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word); - } - } - - ALL_PRIMARY_SYMTABS (objfile, s) - { - QUIT; - b = BLOCKVECTOR_BLOCK (BLOCKVECTOR (s), STATIC_BLOCK); - ALL_BLOCK_SYMBOLS (b, iter, sym) - { - if (code == TYPE_CODE_UNDEF - || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN - && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) - COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word); - } - } - /* Skip macros if we are completing a struct tag -- arguable but usually what is expected. */ if (current_language->la_macro_expansion == macro_expansion_c -- 1.7.1