[Lldb-commits] [PATCH] D151811: [lldb] Take StringRef name in GetIndexOfChildWithName (NFC)

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jun 1 20:23:35 PDT 2023


kastiglione added inline comments.


================
Comment at: lldb/source/Core/ValueObjectSyntheticFilter.cpp:338
   if (!did_find && m_synth_filter_up != nullptr) {
     uint32_t index = m_synth_filter_up->GetIndexOfChildWithName(name);
     if (index == UINT32_MAX)
----------------
Michael137 wrote:
> Could pass name_ref directly now, instead of doing the implicit conversion
This is a different `GetIndexOfChildWithName`, in the `SyntheticChildrenFrontEnd` class. It still takes a `ConstString`, changing it is a follow up.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151811/new/

https://reviews.llvm.org/D151811



More information about the lldb-commits mailing list