[Google 4.7] Fix is_static in gdb index generation

[saugustine, 2012-09-10 21:34:43]

The enclosed patch fixes a reversed conditional when calculating the
is_static flag while generating gdb index--is_static is the opposite
of the DW_AT_external flag.

OK for google 4.7?

Sterling

2012-09-10  Sterling Augustine  <saugustine@google.com>

	* dwarf2out.c (output_pubname): Correct conditional.

Index: dwarf2out.c
===================================================================
--- dwarf2out.c	(revision 191084)
+++ dwarf2out.c	(working copy)
@@ -9436,7 +9436,7 @@
 output_pubname (dw_offset die_offset, pubname_entry *entry)
 {
   dw_die_ref die = entry->die;
-  int is_static = get_AT_flag (die, DW_AT_external) ? 1 : 0;
+  int is_static = get_AT_flag (die, DW_AT_external) ? 0 : 1;
 
   dw2_asm_output_data (DWARF_OFFSET_SIZE, die_offset, "DIE offset");
 

--
This patch is available for review at http://codereview.appspot.com/6498114

[Cary, 2012-09-10 22:05:11]

> The enclosed patch fixes a reversed conditional when calculating the
> is_static flag while generating gdb index--is_static is the opposite
> of the DW_AT_external flag.
>
> OK for google 4.7?
>
> Sterling
>
> 2012-09-10  Sterling Augustine  <saugustine@google.com>
>
>         * dwarf2out.c (output_pubname): Correct conditional.

OK for google/gcc-4_7.

-cary

[saugustine, 2012-09-10 22:48:49]

On Mon, Sep 10, 2012 at 3:05 PM, Cary Coutant <ccoutant@google.com> wrote:
>> The enclosed patch fixes a reversed conditional when calculating the
>> is_static flag while generating gdb index--is_static is the opposite
>> of the DW_AT_external flag.
>>
>> OK for google 4.7?
>>
>> Sterling
>>
>> 2012-09-10  Sterling Augustine  <saugustine@google.com>
>>
>>         * dwarf2out.c (output_pubname): Correct conditional.
>
> OK for google/gcc-4_7.
>
> -cary

Committed as revision 191163.