https://github.com/PortalPete created https://github.com/llvm/llvm-project/pull/75870
This a follow-up PR from this other one: https://github.com/llvm/llvm-project/pull/74413 Nothing calls into these two methods, so we (@DavidSpickett, @adrian-prantl, and I) agreed to remove them once we merged the previous PR. >From c27a7a110bda6dbb96757f662c5d5b05e8a0ef75 Mon Sep 17 00:00:00 2001 From: Pete Lawrence <plawre...@apple.com> Date: Mon, 18 Dec 2023 13:00:27 -1000 Subject: [PATCH] [lldb] Remove unused GetChildAtIndexPath(...) methods from ValueObject.cpp --- lldb/include/lldb/Core/ValueObject.h | 8 ------ lldb/source/Core/ValueObject.cpp | 40 ---------------------------- 2 files changed, 48 deletions(-) diff --git a/lldb/include/lldb/Core/ValueObject.h b/lldb/include/lldb/Core/ValueObject.h index a158199e7fab1a..3f8005ba696ce8 100644 --- a/lldb/include/lldb/Core/ValueObject.h +++ b/lldb/include/lldb/Core/ValueObject.h @@ -468,14 +468,6 @@ class ValueObject { virtual lldb::ValueObjectSP GetChildAtIndex(size_t idx, bool can_create = true); - // The method always creates missing children in the path, if necessary. - lldb::ValueObjectSP GetChildAtIndexPath(llvm::ArrayRef<size_t> idxs, - size_t *index_of_error = nullptr); - - lldb::ValueObjectSP - GetChildAtIndexPath(llvm::ArrayRef<std::pair<size_t, bool>> idxs, - size_t *index_of_error = nullptr); - // The method always creates missing children in the path, if necessary. lldb::ValueObjectSP GetChildAtNamePath(llvm::ArrayRef<llvm::StringRef> names); diff --git a/lldb/source/Core/ValueObject.cpp b/lldb/source/Core/ValueObject.cpp index b13bffa0ca809b..8814c01d93ca01 100644 --- a/lldb/source/Core/ValueObject.cpp +++ b/lldb/source/Core/ValueObject.cpp @@ -392,46 +392,6 @@ ValueObjectSP ValueObject::GetChildAtIndex(size_t idx, bool can_create) { return child_sp; } -lldb::ValueObjectSP -ValueObject::GetChildAtIndexPath(llvm::ArrayRef<size_t> idxs, - size_t *index_of_error) { - if (idxs.size() == 0) - return GetSP(); - ValueObjectSP root(GetSP()); - - size_t current_index = 0; - for (size_t idx : idxs) { - root = root->GetChildAtIndex(idx); - if (!root) { - if (index_of_error) - *index_of_error = current_index; - return root; - } - current_index += 1; - } - return root; -} - -lldb::ValueObjectSP ValueObject::GetChildAtIndexPath( - llvm::ArrayRef<std::pair<size_t, bool>> idxs, size_t *index_of_error) { - if (idxs.size() == 0) - return GetSP(); - ValueObjectSP root(GetSP()); - - size_t current_index = 0; - for (std::pair<size_t, bool> idx : idxs) { - root = root->GetChildAtIndex(idx.first, idx.second); - if (!root) { - if (index_of_error) - *index_of_error = current_index; - return root; - } - - current_index += 1; - } - return root; -} - lldb::ValueObjectSP ValueObject::GetChildAtNamePath(llvm::ArrayRef<llvm::StringRef> names) { if (names.size() == 0) _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
