diff mbox

[Google,4.7] Fix is_static in gdb index generation (issue6498114)

Message ID 20120910213442.04F9B160A9D@sterling.mtv.corp.google.com
State New
Headers show

Commit Message

Sterling Augustine Sept. 10, 2012, 9:34 p.m. UTC 
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.


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

Comments

Cary Coutant Sept. 10, 2012, 10:05 p.m. UTC | #1 
> 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
Sterling Augustine Sept. 10, 2012, 10:48 p.m. UTC | #2 
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.
diff mbox

Patch

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");