This is the mail archive of the gdb-cvs@sourceware.org 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]

gdb and binutils branch master updated. cdc07690a4812fb41d3766d087cf7ec78184d6a8


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".

The branch, master has been updated
       via  cdc07690a4812fb41d3766d087cf7ec78184d6a8 (commit)
      from  3881fb67b7017669b3acaba919914954c16270a8 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=cdc07690a4812fb41d3766d087cf7ec78184d6a8

commit cdc07690a4812fb41d3766d087cf7ec78184d6a8
Author: Yao Qi <yao@codesourcery.com>
Date:   Fri Aug 15 19:06:02 2014 +0800

    Update comments in scan_partial_symbols and add_partial_subprogram
    
    I read comment of scan_partial_symbols about NEED_PC and how *LOWPC
    and *HIGHPC are updated:
    
       DW_AT_ranges).  If NEED_PC is set, then this function will set
       *LOWPC and *HIGHPC to the lowest and highest PC values found in CU
       and record the covered ranges in the addrmap.
    
    NEED_PC is only used in the callee of scan_partial_symbols,
    add_partial_subprogram,
    
      if (pdi->tag == DW_TAG_subprogram)
        {
          if (pdi->has_pc_info)
            {
              if (pdi->lowpc < *lowpc)
                *lowpc = pdi->lowpc;
              if (pdi->highpc > *highpc)
                *highpc = pdi->highpc;
    	  if (need_pc)
    
    *LOWPC and *HIGHPC is updated regardless of NEED_PC.  When NEED_PC is
    true, addrmap is updated.  It would be clear to rename NEED_PC to
    SET_ADDRMAP.  That is what this patch does.  Beside this, this patch
    also adjust comments in related functions.
    
    gdb:
    
    2014-08-24  Yao Qi  <yao@codesourcery.com>
    
    	* dwarf2read.c (scan_partial_symbols):  Update comments.
    	Rename argument 'need_pc' with 'set_addrmap'.
    	ï¼?add_partial_namespace): Rename argument 'need_pc' with
    	'set_addrmap'.
    	(add_partial_module): Likewise.
    	(add_partial_subprogram): Likewise.  Update comments.
    	(dwarf2_name): Fix typo.

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog    |   10 ++++++++++
 gdb/dwarf2read.c |   44 +++++++++++++++++++++++---------------------
 2 files changed, 33 insertions(+), 21 deletions(-)


hooks/post-receive
-- 
gdb and binutils


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