[Lldb-commits] [PATCH] D57273: Make Type::GetByteSize optional

[Adrian Prantl via Phabricator via lldb-commits]

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


================
Comment at: include/lldb/Symbol/Type.h:221
+  uint64_t m_byte_size : 63;
+  unsigned m_byte_size_has_value : 1;
   Declaration m_decl;
----------------
clayborg wrote:
> Does this need to be uint64_t to share the same space as m_byte_size?
I don't think so, but it might be more readable.


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

https://reviews.llvm.org/D57273