[Lldb-commits] [PATCH] D156447: [lldb] Split CTF parsing and type creation (NFC)

[Alex Langford via Phabricator via lldb-commits]

bulbazord added inline comments.


================
Comment at: lldb/source/Plugins/SymbolFile/CTF/CTFTypes.h:1
+//===-- CTFType.h -----------------------------------------------*- C++ -*-===//
+//
----------------



================
Comment at: lldb/source/Plugins/SymbolFile/CTF/CTFTypes.h:9-10
+
+#ifndef LLDB_SOURCE_PLUGINS_SYMBOLFILE_CTF_CTFTYPE_H
+#define LLDB_SOURCE_PLUGINS_SYMBOLFILE_CTF_CTFTYPE_H
+
----------------



================
Comment at: lldb/source/Plugins/SymbolFile/CTF/CTFTypes.h:101
+  struct Value {
+    Value(llvm::StringRef name, uint32_t value) : name(name), value(value){};
+    llvm::StringRef name;
----------------



================
Comment at: lldb/source/Plugins/SymbolFile/CTF/SymbolFileCTF.cpp:931
 
-void SymbolFileCTF::AddTypeForUID(lldb::user_id_t type_uid, lldb::TypeSP type) {
-  assert(type_uid == m_types.size() + 1);
-  m_types.emplace_back(type);
-}
+lldb_private::Type *SymbolFileCTF::ResolveTypeUID(lldb::user_id_t type_uid) {
+  auto find_result = m_types.find(type_uid);
----------------
Why do we return a `Type *` instead of a `TypeSP`?


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

https://reviews.llvm.org/D156447