commit 6ee298a346237f88a938e96c038802b729b51e9d
author Joshua Peraza <jperaza@chromium.org> Thu Aug 16 13:42:04 2018 -0700
committer Joshua Peraza <jperaza@chromium.org> Fri Aug 17 04:26:39 2018 +0000
tree b36b8d7fbf2e898c217767be7afe5c17c4a4c043
parent acf983292a94c968ccb8e5a0d6951d1f787d92da

Treat high_pc as an address for DW_FORM_GNU_addr_index

The high_pc is an address and has already been read from .debug_addr
before being passed into FuncHandler::ProcessAttributeUnsigned.

Bug:870908
Change-Id: I950098e360b5193f26bf767b8fa0a5f9d59e66ce
Reviewed-on: https://chromium-review.googlesource.com/1178760
Reviewed-by: Mark Mentovai <mark@chromium.org>

src/common/dwarf_cu_to_module.cc

diff --git a/src/common/dwarf_cu_to_module.cc b/src/common/dwarf_cu_to_module.cc
index f5a03b4..527a704 100644
--- a/src/common/dwarf_cu_to_module.cc
+++ b/src/common/dwarf_cu_to_module.cc
@@ -573,7 +573,8 @@
 
   if (!ranges_) {
     // Make high_pc_ an address, if it isn't already.
-    if (high_pc_form_ != dwarf2reader::DW_FORM_addr) {
+    if (high_pc_form_ != dwarf2reader::DW_FORM_addr &&
+        high_pc_form_ != dwarf2reader::DW_FORM_GNU_addr_index) {
       high_pc_ += low_pc_;
     }