Remove outdated TODOs in RegisterValue
authorAlex Langford <apl@fb.com>
Thu, 16 Aug 2018 22:48:46 +0000 (22:48 +0000)
committerAlex Langford <apl@fb.com>
Thu, 16 Aug 2018 22:48:46 +0000 (22:48 +0000)
These TODOs were for setting m_type in RegisterValue::SetValueFromString
in the case where reg_info's encoding was eEncodingUint or
eEncodingSint. m_type is set by SetUInt{8,16,32,64.128} during the
SetUInt call.

llvm-svn: 339959

lldb/source/Utility/RegisterValue.cpp

index 44dfead..2870ac2 100644 (file)
@@ -380,7 +380,6 @@ Status RegisterValue::SetValueFromString(const RegisterInfo *reg_info,
           "unsupported unsigned integer byte size: %u", byte_size);
       break;
     }
-    // TODO: Shouldn't we be setting m_type here?
     break;
 
   case eEncodingSint:
@@ -410,8 +409,6 @@ Status RegisterValue::SetValueFromString(const RegisterInfo *reg_info,
                                      byte_size);
       break;
     }
-
-    // TODO: Shouldn't we be setting m_type here?
     break;
 
   case eEncodingIEEE754: {