diff options
author | David Stenberg <david.stenberg@ericsson.com> | 2019-04-09 07:46:09 +0000 |
---|---|---|
committer | David Stenberg <david.stenberg@ericsson.com> | 2019-04-09 07:46:09 +0000 |
commit | 93b497a61d205088ad14e3ec7e7abc91e33592fe (patch) | |
tree | 811c3607bc33d53d6589bbfbd445e3ace5d0967f /llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h | |
parent | e043dadcad2e84a8fecb57efbbbd3f55b1b3a422 (diff) | |
download | bcm5719-llvm-93b497a61d205088ad14e3ec7e7abc91e33592fe.tar.gz bcm5719-llvm-93b497a61d205088ad14e3ec7e7abc91e33592fe.zip |
[DebugInfo] Remove redundant DebugLocEntry::MergeValues() function, NFC
Summary:
The MergeValues() function would try to merge two entries if they shared
the same beginning label. Having the same beginning label means that the
former entry's range would be empty; however, after D55919 we no longer
create entries for empty ranges, so we can no longer land in a situation
where that check in MergeValues would succeed. Instead, the "merging" is
done by keeping the live values from the preceding empty ranges in
OpenRanges, and adding them to the first non-empty range.
Reviewers: aprantl, dblaikie, loladiro
Reviewed By: aprantl
Subscribers: llvm-commits
Tags: #debug-info, #llvm
Differential Revision: https://reviews.llvm.org/D59301
llvm-svn: 357974
Diffstat (limited to 'llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h')
-rw-r--r-- | llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h b/llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h index 256a41be237..5011c84d7f7 100644 --- a/llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h +++ b/llvm/lib/CodeGen/AsmPrinter/DebugLocEntry.h @@ -105,12 +105,6 @@ public: Values.push_back(std::move(Val)); } - /// If this and Next are describing different pieces of the same - /// variable, merge them by appending Next's values to the current - /// list of values. - /// Return true if the merge was successful. - bool MergeValues(const DebugLocEntry &Next); - /// Attempt to merge this DebugLocEntry with Next and return /// true if the merge was successful. Entries can be merged if they /// share the same Loc/Constant and if Next immediately follows this |