llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Pete Lawrence (PortalPete) <details> <summary>Changes</summary> 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. --- Full diff: https://github.com/llvm/llvm-project/pull/75870.diff 2 Files Affected: - (modified) lldb/include/lldb/Core/ValueObject.h (-8) - (modified) lldb/source/Core/ValueObject.cpp (-40) ``````````diff 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) `````````` </details> https://github.com/llvm/llvm-project/pull/75870 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
