Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1731)

Issue 5824050: [google/4.6] Fix problem where -gfission emits duplicate strings

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by Cary
Modified:
12 years ago
Reviewers:
saugustine
CC:
gcc-patches_gcc.gnu.org
Base URL:
svn+ssh://gcc.gnu.org/svn/gcc/branches/google/gcc-4_6/gcc/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M dwarf2out.c View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2
Cary
This is for the google/gcc-4_6 branch only. Fix output_indirect_string so that it does not output ...
12 years ago (2012-03-14 18:15:25 UTC) #1
saugustine
12 years ago (2012-03-14 19:09:20 UTC) #2
On 2012/03/14 18:15:25, Cary wrote:
> This is for the google/gcc-4_6 branch only.
> 
> Fix output_indirect_string so that it does not output strings that
> have already been written to the .debug_str.dwo section.
> 
> This is a backport of  part of an unrelated change from upstream 4.7:
> http://gcc.gnu.org/ml/gcc-cvs/2011-05/msg00130.html
> 
> Tested: bootstrap, core, mantle, crust (in progress).
> 
> 
> 2012-03-14   Cary Coutant  <mailto:ccoutant@google.com>
> 
> 	* dwarf2out.c (output_indirect_string): Check for DW_FORM_strp
> 	instead of presence of label and non-zero refcount.
> 
> 
> Index: dwarf2out.c
> ===================================================================
> --- dwarf2out.c	(revision 185269)
> +++ dwarf2out.c	(working copy)
> @@ -23042,7 +23042,7 @@ output_indirect_string (void **h, void *
>  {
>    struct indirect_string_node *node = (struct indirect_string_node *) *h;
>  
> -  if (node->label && node->refcount)
> +  if (node->form == DW_FORM_strp)
>      {
>        switch_to_section (debug_str_section);
>        ASM_OUTPUT_LABEL (asm_out_file, node->label);
> 
> --
> This patch is available for review at http://codereview.appspot.com/5824050

This is OK for google branches.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b