[all-commits] [llvm/llvm-project] 1227fa: Remove unused ValueObjectDynamicValue::SetOwningSP...

jimingham via All-commits all-commits at lists.llvm.org
Thu Oct 28 16:07:13 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1227fa7e9040469d648cc7709a298901002f4c27
      https://github.com/llvm/llvm-project/commit/1227fa7e9040469d648cc7709a298901002f4c27
  Author: Jim Ingham <jingham at apple.com>
  Date:   2021-10-28 (Thu, 28 Oct 2021)

  Changed paths:
    M lldb/include/lldb/Core/ValueObjectDynamicValue.h
    M lldb/source/Core/ValueObjectDynamicValue.cpp

  Log Message:
  -----------
  Remove unused ValueObjectDynamicValue::SetOwningSP & backing ivar

This has no uses and the ValueObjectDynamicValue already tracks
its ownership through the parent it is passed when made.  I can't
find any vestiges of the use of this API, maybe it was from some
earlier design?

Resetting the backing ivar was the only job the destructor did, so I
set that to default as well.

Differential Revision: https://reviews.llvm.org/D112677




More information about the All-commits mailing list