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

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.


================
Comment at: lldb/source/Plugins/SymbolFile/CTF/CTFTypes.h:108
+          uint32_t size, std::vector<Value> values)
+      : CTFType(eEnum, uid, name), nelems(nelems), size(size) {
+    assert(values.size() == nelems);
----------------
Michael137 wrote:
> did you omit this by accident?
Yes, good eye. I need to see why my test didn't catch this. 


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

https://reviews.llvm.org/D156447