From 12ecff373a5013fc166057e83c9d607b3255ae96 Mon Sep 17 00:00:00 2001 From: Ludovic Guegan Date: Tue, 13 Nov 2018 13:00:07 +0100 Subject: Skip empty DWARF code range when mapping lines BUG=777 Change-Id: Ic6d05eee3ff4660b6d087999a8cea04a1ee3e92b Reviewed-on: https://chromium-review.googlesource.com/c/1333507 Reviewed-by: Ted Mielczarek --- src/common/dwarf_cu_to_module.cc | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to '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 e5071ba7..4f870ad4 100644 --- a/src/common/dwarf_cu_to_module.cc +++ b/src/common/dwarf_cu_to_module.cc @@ -931,8 +931,9 @@ class FunctionRange { Module::Function *function; }; -// Fills an array of ranges with pointers to the functions which owns them. -// The array is sorted in ascending order and the ranges are non-overlapping. +// Fills an array of ranges with pointers to the functions which owns +// them. The array is sorted in ascending order and the ranges are non +// empty and non-overlapping. static void FillSortedFunctionRanges(vector &dest_ranges, vector *functions) { @@ -946,7 +947,9 @@ static void FillSortedFunctionRanges(vector &dest_ranges, ranges_it != ranges.cend(); ++ranges_it) { FunctionRange range(*ranges_it, func); - dest_ranges.push_back(range); + if (range.size != 0) { + dest_ranges.push_back(range); + } } } -- cgit v1.2.1