clayborg added inline comments.

================
Comment at: include/lldb/Symbol/PostfixExpression.h:97
 private:
-  uint32_t m_value;
+  int64_t m_value;
 };
----------------
Do we want to try and use lldb_private::Scalar here? Then this could handle 
signed/unsigned ints, floats and anything else that comes along? Might need to 
be renamed "ScalarNode"? All of the type promotion stuff is already built into 
those classes.


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

https://reviews.llvm.org/D61311



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to