Author: xiaobai
Date: Thu Aug 16 15:48:46 2018
New Revision: 339959

URL: http://llvm.org/viewvc/llvm-project?rev=339959&view=rev
Log:
Remove outdated TODOs in RegisterValue

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.

Modified:
    lldb/trunk/source/Utility/RegisterValue.cpp

Modified: lldb/trunk/source/Utility/RegisterValue.cpp
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Utility/RegisterValue.cpp?rev=339959&r1=339958&r2=339959&view=diff
==============================================================================
--- lldb/trunk/source/Utility/RegisterValue.cpp (original)
+++ lldb/trunk/source/Utility/RegisterValue.cpp Thu Aug 16 15:48:46 2018
@@ -380,7 +380,6 @@ Status RegisterValue::SetValueFromString
           "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
                                      byte_size);
       break;
     }
-
-    // TODO: Shouldn't we be setting m_type here?
     break;
 
   case eEncodingIEEE754: {


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

Reply via email to