sas accepted this revision. sas added a comment. This revision is now accepted and ready to land.
Looks like the numeric values of `TypeCode` are contiguous. It would probably be cleaner to a check after the cast to validate the value instead of having to add a line for each label in the `switch` statement. Something like this: if (type_code < TypeCodes::sint8 || type_code > TypeCodes::sint128) return false; You could also add `TypeCodesBegin = -1` and `TypeCodesEnd` as the first and last enumerations of `TypeCodes` so the above check would not have to be modified when you change the definition of `TypeCodes`. Other than that, looks good. https://reviews.llvm.org/D35036 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits