teemperor accepted this revision. teemperor added a comment. This revision is now accepted and ready to land.
Some small rename I forgot to point out, but LGTM modulo that rename. Thanks for cleaning this up! ================ Comment at: lldb/include/lldb/Symbol/Symtab.h:186 private: + UniqueCStringMap<uint32_t> &GetFunctypeMap(lldb::FunctionNameType type) { + auto map = m_name_to_symbol_indices.find(type); ---------------- `GetNameToSymbolIndexMap` might be more accurate now (I should have pointed out when changing the variable name). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103652/new/ https://reviews.llvm.org/D103652 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits