[Lldb-commits] [lldb] r184333 - Sort out a number of mismatched integer types in order to cut down the number of compiler warnings.

Andy Gibbs andyg1001 at hotmail.co.uk
Wed Jun 19 12:04:54 PDT 2013


Author: andyg
Date: Wed Jun 19 14:04:53 2013
New Revision: 184333

URL: http://llvm.org/viewvc/llvm-project?rev=184333&view=rev
Log:
Sort out a number of mismatched integer types in order to cut down the number of compiler warnings.

Modified:
    lldb/trunk/include/lldb/DataFormatters/FormatNavigator.h
    lldb/trunk/include/lldb/DataFormatters/TypeSynthetic.h
    lldb/trunk/scripts/Python/python-typemaps.swig
    lldb/trunk/source/Breakpoint/BreakpointResolverFileLine.cpp
    lldb/trunk/source/Breakpoint/BreakpointResolverFileRegex.cpp
    lldb/trunk/source/Commands/CommandObjectArgs.cpp
    lldb/trunk/source/Commands/CommandObjectHelp.cpp
    lldb/trunk/source/Commands/CommandObjectSyntax.cpp
    lldb/trunk/source/Core/DataExtractor.cpp
    lldb/trunk/source/Core/Listener.cpp
    lldb/trunk/source/Core/StringList.cpp
    lldb/trunk/source/DataFormatters/TypeSynthetic.cpp
    lldb/trunk/source/Expression/ClangASTSource.cpp
    lldb/trunk/source/Expression/IRForTarget.cpp
    lldb/trunk/source/Interpreter/CommandInterpreter.cpp
    lldb/trunk/source/Interpreter/OptionValueFileSpecLIst.cpp
    lldb/trunk/source/Interpreter/OptionValuePathMappings.cpp
    lldb/trunk/source/Interpreter/Options.cpp
    lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp
    lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.cpp
    lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp
    lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp
    lldb/trunk/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp
    lldb/trunk/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp
    lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
    lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp
    lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp
    lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
    lldb/trunk/source/Symbol/ClangASTContext.cpp
    lldb/trunk/source/Symbol/Symtab.cpp
    lldb/trunk/source/Target/Process.cpp
    lldb/trunk/source/Target/Target.cpp
    lldb/trunk/source/Target/ThreadPlanStepUntil.cpp

Modified: lldb/trunk/include/lldb/DataFormatters/FormatNavigator.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/include/lldb/DataFormatters/FormatNavigator.h?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/include/lldb/DataFormatters/FormatNavigator.h (original)
+++ lldb/trunk/include/lldb/DataFormatters/FormatNavigator.h Wed Jun 19 14:04:53 2013
@@ -71,7 +71,7 @@ HasPrefix (const char* str1, const char*
 // those will not match any type because of the way we strip qualifiers from typenames
 // this method looks for the case where the user is adding a "class","struct","enum" or "union" Foo
 // and strips the unnecessary qualifier
-static ConstString
+static inline ConstString
 GetValidTypeName_Impl (const ConstString& type)
 {
     int strip_len = 0;

Modified: lldb/trunk/include/lldb/DataFormatters/TypeSynthetic.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/include/lldb/DataFormatters/TypeSynthetic.h?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/include/lldb/DataFormatters/TypeSynthetic.h (original)
+++ lldb/trunk/include/lldb/DataFormatters/TypeSynthetic.h Wed Jun 19 14:04:53 2013
@@ -295,20 +295,20 @@ namespace lldb_private {
             m_expression_paths.clear();
         }
         
-        int
+        size_t
         GetCount() const
         {
             return m_expression_paths.size();
         }
         
         const char*
-        GetExpressionPathAtIndex(int i) const
+        GetExpressionPathAtIndex(size_t i) const
         {
             return m_expression_paths[i].c_str();
         }
         
         bool
-        SetExpressionPathAtIndex (int i, const char* path)
+        SetExpressionPathAtIndex (size_t i, const char* path)
         {
             return SetExpressionPathAtIndex(i, std::string(path));
         }
@@ -329,7 +329,7 @@ namespace lldb_private {
         }
         
         bool
-        SetExpressionPathAtIndex (int i, const std::string& path)
+        SetExpressionPathAtIndex (size_t i, const std::string& path)
         {
             if (i >= GetCount())
                 return false;
@@ -399,7 +399,7 @@ namespace lldb_private {
             GetIndexOfChildWithName (const ConstString &name)
             {
                 const char* name_cstr = name.GetCString();
-                for (int i = 0; i < filter->GetCount(); i++)
+                for (size_t i = 0; i < filter->GetCount(); i++)
                 {
                     const char* expr_cstr = filter->GetExpressionPathAtIndex(i);
                     if (expr_cstr)

Modified: lldb/trunk/scripts/Python/python-typemaps.swig
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/scripts/Python/python-typemaps.swig?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/scripts/Python/python-typemaps.swig (original)
+++ lldb/trunk/scripts/Python/python-typemaps.swig Wed Jun 19 14:04:53 2013
@@ -333,7 +333,7 @@
     if (count >= $2)
         count = $2;
     PyObject* list = PyList_New(count);
-    for (int j = 0; j < count; j++)
+    for (uint32_t j = 0; j < count; j++)
     {
         if ($1[j] < UINT32_MAX)
         {

Modified: lldb/trunk/source/Breakpoint/BreakpointResolverFileLine.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Breakpoint/BreakpointResolverFileLine.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Breakpoint/BreakpointResolverFileLine.cpp (original)
+++ lldb/trunk/source/Breakpoint/BreakpointResolverFileLine.cpp Wed Jun 19 14:04:53 2013
@@ -87,7 +87,7 @@ BreakpointResolverFileLine::SearchCallba
     while (sc_list.GetSize() > 0)
     {
         SymbolContextList tmp_sc_list;
-        int current_idx = 0;
+        unsigned current_idx = 0;
         SymbolContext sc;
         bool first_entry = true;
         

Modified: lldb/trunk/source/Breakpoint/BreakpointResolverFileRegex.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Breakpoint/BreakpointResolverFileRegex.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Breakpoint/BreakpointResolverFileRegex.cpp (original)
+++ lldb/trunk/source/Breakpoint/BreakpointResolverFileRegex.cpp Wed Jun 19 14:04:53 2013
@@ -62,7 +62,7 @@ BreakpointResolverFileRegex::SearchCallb
     std::vector<uint32_t> line_matches;
     context.target_sp->GetSourceManager().FindLinesMatchingRegex(cu_file_spec, m_regex, 1, UINT32_MAX, line_matches); 
     uint32_t num_matches = line_matches.size();
-    for (int i = 0; i < num_matches; i++)
+    for (uint32_t i = 0; i < num_matches; i++)
     {
         uint32_t start_idx = 0;
         bool exact = false;

Modified: lldb/trunk/source/Commands/CommandObjectArgs.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectArgs.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Commands/CommandObjectArgs.cpp (original)
+++ lldb/trunk/source/Commands/CommandObjectArgs.cpp Wed Jun 19 14:04:53 2013
@@ -121,7 +121,7 @@ CommandObjectArgs::DoExecute (Args& args
     }
     
     const size_t num_args = args.GetArgumentCount ();
-    int arg_index;
+    size_t arg_index;
     
     if (!num_args)
     {
@@ -256,7 +256,7 @@ CommandObjectArgs::DoExecute (Args& args
 
     for (arg_index = 0; arg_index < num_args; ++arg_index)
     {
-        result.GetOutputStream ().Printf ("%d (%s): ", arg_index, args.GetArgumentAtIndex (arg_index));
+        result.GetOutputStream ().Printf ("%zu (%s): ", arg_index, args.GetArgumentAtIndex (arg_index));
         value_list.GetValueAtIndex (arg_index)->Dump (&result.GetOutputStream ());
         result.GetOutputStream ().Printf("\n");
     }

Modified: lldb/trunk/source/Commands/CommandObjectHelp.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectHelp.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Commands/CommandObjectHelp.cpp (original)
+++ lldb/trunk/source/Commands/CommandObjectHelp.cpp Wed Jun 19 14:04:53 2013
@@ -94,7 +94,7 @@ CommandObjectHelp::DoExecute (Args& comm
             CommandObject *sub_cmd_obj = cmd_obj;
             // Loop down through sub_command dictionaries until we find the command object that corresponds
             // to the help command entered.
-            for (int i = 1; i < argc && all_okay; ++i)
+            for (size_t i = 1; i < argc && all_okay; ++i)
             {
                 std::string sub_command = command.GetArgumentAtIndex(i);
                 matches.Clear();

Modified: lldb/trunk/source/Commands/CommandObjectSyntax.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectSyntax.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Commands/CommandObjectSyntax.cpp (original)
+++ lldb/trunk/source/Commands/CommandObjectSyntax.cpp Wed Jun 19 14:04:53 2013
@@ -65,7 +65,7 @@ CommandObjectSyntax::DoExecute (Args& co
     {
         cmd_obj = m_interpreter.GetCommandObject (command.GetArgumentAtIndex(0));
         bool all_okay = true;
-        for (int i = 1; i < argc; ++i)
+        for (size_t i = 1; i < argc; ++i)
         {
             std::string sub_command = command.GetArgumentAtIndex (i);
             if (!cmd_obj->IsMultiwordObject())

Modified: lldb/trunk/source/Core/DataExtractor.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Core/DataExtractor.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Core/DataExtractor.cpp (original)
+++ lldb/trunk/source/Core/DataExtractor.cpp Wed Jun 19 14:04:53 2013
@@ -738,7 +738,7 @@ DataExtractor::GetFloat (offset_t *offse
         {
             const uint8_t *src_data = (const uint8_t *)src;
             uint8_t *dst_data = (uint8_t *)&val;
-            for (int i=0; i<sizeof(float_type); ++i)
+            for (size_t i=0; i<sizeof(float_type); ++i)
                 dst_data[sizeof(float_type) - 1 - i] = src_data[i];
         }
         else
@@ -762,7 +762,7 @@ DataExtractor::GetDouble (offset_t *offs
         {
             const uint8_t *src_data = (const uint8_t *)src;
             uint8_t *dst_data = (uint8_t *)&val;
-            for (int i=0; i<sizeof(float_type); ++i)
+            for (size_t i=0; i<sizeof(float_type); ++i)
                 dst_data[sizeof(float_type) - 1 - i] = src_data[i];
         }
         else
@@ -787,7 +787,7 @@ DataExtractor::GetLongDouble (offset_t *
         {
             const uint8_t *src_data = (const uint8_t *)src;
             uint8_t *dst_data = (uint8_t *)&val;
-            for (int i=0; i<sizeof(float_type); ++i)
+            for (size_t i=0; i<sizeof(float_type); ++i)
                 dst_data[sizeof(float_type) - 1 - i] = src_data[i];
         }
         else

Modified: lldb/trunk/source/Core/Listener.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Core/Listener.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Core/Listener.cpp (original)
+++ lldb/trunk/source/Core/Listener.cpp Wed Jun 19 14:04:53 2013
@@ -44,7 +44,7 @@ Listener::~Listener()
     
     size_t num_managers = m_broadcaster_managers.size();
     
-    for (int i = 0; i < num_managers; i++)
+    for (size_t i = 0; i < num_managers; i++)
         m_broadcaster_managers[i]->RemoveListener(*this);
         
     if (log)

Modified: lldb/trunk/source/Core/StringList.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Core/StringList.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Core/StringList.cpp (original)
+++ lldb/trunk/source/Core/StringList.cpp Wed Jun 19 14:04:53 2013
@@ -235,7 +235,7 @@ StringList::CopyList(const char* item_pr
                      const char* items_sep)
 {
     StreamString strm;
-    for (int i = 0; i < GetSize(); i++)
+    for (size_t i = 0; i < GetSize(); i++)
     {
         if (i && items_sep && items_sep[0])
             strm << items_sep;

Modified: lldb/trunk/source/DataFormatters/TypeSynthetic.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/DataFormatters/TypeSynthetic.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/DataFormatters/TypeSynthetic.cpp (original)
+++ lldb/trunk/source/DataFormatters/TypeSynthetic.cpp Wed Jun 19 14:04:53 2013
@@ -39,7 +39,7 @@ TypeFilterImpl::GetDescription()
                 SkipsPointers() ? " (skip pointers)" : "",
                 SkipsReferences() ? " (skip references)" : "");
     
-    for (int i = 0; i < GetCount(); i++)
+    for (size_t i = 0; i < GetCount(); i++)
     {
         sstr.Printf("    %s\n",
                     GetExpressionPathAtIndex(i));

Modified: lldb/trunk/source/Expression/ClangASTSource.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Expression/ClangASTSource.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Expression/ClangASTSource.cpp (original)
+++ lldb/trunk/source/Expression/ClangASTSource.cpp Wed Jun 19 14:04:53 2013
@@ -861,7 +861,7 @@ FindObjCMethodDeclsWithOrigin (unsigned
         
         clang::Selector sel = decl_name.getObjCSelector();
         
-        int num_args = sel.getNumArgs();
+        unsigned num_args = sel.getNumArgs();
         
         for (unsigned i = 0;
              i != num_args;

Modified: lldb/trunk/source/Expression/IRForTarget.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Expression/IRForTarget.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Expression/IRForTarget.cpp (original)
+++ lldb/trunk/source/Expression/IRForTarget.cpp Wed Jun 19 14:04:53 2013
@@ -1435,7 +1435,7 @@ IRForTarget::MaterializeInitializer (uin
                         
             size_t element_size = m_target_data->getTypeAllocSize(array_element_type);
             
-            for (int i = 0; i < array_initializer->getNumOperands(); ++i)
+            for (unsigned i = 0; i < array_initializer->getNumOperands(); ++i)
             {
                 Value *operand_value = array_initializer->getOperand(i);
                 Constant *operand_constant = dyn_cast<Constant>(operand_value);
@@ -1454,7 +1454,7 @@ IRForTarget::MaterializeInitializer (uin
         StructType *struct_initializer_type = struct_initializer->getType();
         const StructLayout *struct_layout = m_target_data->getStructLayout(struct_initializer_type);
 
-        for (int i = 0;
+        for (unsigned i = 0;
              i < struct_initializer->getNumOperands();
              ++i)
         {

Modified: lldb/trunk/source/Interpreter/CommandInterpreter.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/CommandInterpreter.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/CommandInterpreter.cpp (original)
+++ lldb/trunk/source/Interpreter/CommandInterpreter.cpp Wed Jun 19 14:04:53 2013
@@ -1019,7 +1019,7 @@ CommandInterpreter::GetAliasHelp (const
     if (option_arg_vector_sp)
     {
         OptionArgVector *options = option_arg_vector_sp.get();
-        for (int i = 0; i < options->size(); ++i)
+        for (size_t i = 0; i < options->size(); ++i)
         {
             OptionArgPair cur_option = (*options)[i];
             std::string opt = cur_option.first;
@@ -1314,7 +1314,7 @@ CommandInterpreter::BuildAliasResult (co
         {
             OptionArgVector *option_arg_vector = option_arg_vector_sp.get();
 
-            for (int i = 0; i < option_arg_vector->size(); ++i)
+            for (size_t i = 0; i < option_arg_vector->size(); ++i)
             {
                 OptionArgPair option_pair = (*option_arg_vector)[i];
                 OptionArgValue value_pair = option_pair.second;
@@ -2280,7 +2280,7 @@ CommandInterpreter::BuildAliasCommandArg
         
         used[0] = true;
 
-        for (int i = 0; i < option_arg_vector->size(); ++i)
+        for (size_t i = 0; i < option_arg_vector->size(); ++i)
         {
             OptionArgPair option_pair = (*option_arg_vector)[i];
             OptionArgValue value_pair = option_pair.second;
@@ -2345,7 +2345,7 @@ CommandInterpreter::BuildAliasCommandArg
             }
         }
 
-        for (int j = 0; j < cmd_args.GetArgumentCount(); ++j)
+        for (size_t j = 0; j < cmd_args.GetArgumentCount(); ++j)
         {
             if (!used[j] && !wants_raw_input)
                 new_args.AppendArgument (cmd_args.GetArgumentAtIndex (j));
@@ -2509,7 +2509,7 @@ CommandInterpreter::HandleCommands (cons
         m_debugger.SetAsyncExecution (false);
     }
 
-    for (int idx = 0; idx < num_lines; idx++)
+    for (size_t idx = 0; idx < num_lines; idx++)
     {
         const char *cmd = commands.GetStringAtIndex(idx);
         if (cmd[0] == '\0')
@@ -2550,7 +2550,7 @@ CommandInterpreter::HandleCommands (cons
                 error_msg = "<unknown error>.\n";
             if (stop_on_error)
             {
-                result.AppendErrorWithFormat("Aborting reading of commands after command #%d: '%s' failed with %s",
+                result.AppendErrorWithFormat("Aborting reading of commands after command #%zu: '%s' failed with %s",
                                          idx, cmd, error_msg);
                 result.SetStatus (eReturnStatusFailed);
                 m_debugger.SetAsyncExecution (old_async_execution);
@@ -2558,7 +2558,7 @@ CommandInterpreter::HandleCommands (cons
             }
             else if (print_results)
             {
-                result.AppendMessageWithFormat ("Command #%d '%s' failed with %s",
+                result.AppendMessageWithFormat ("Command #%zu '%s' failed with %s",
                                                 idx + 1, 
                                                 cmd, 
                                                 error_msg);
@@ -2583,10 +2583,10 @@ CommandInterpreter::HandleCommands (cons
                 // status in our real result before returning.  This is an error if the continue was not the
                 // last command in the set of commands to be run.
                 if (idx != num_lines - 1)
-                    result.AppendErrorWithFormat("Aborting reading of commands after command #%d: '%s' continued the target.\n", 
+                    result.AppendErrorWithFormat("Aborting reading of commands after command #%zu: '%s' continued the target.\n", 
                                                  idx + 1, cmd);
                 else
-                    result.AppendMessageWithFormat ("Command #%d '%s' continued the target.\n", idx + 1, cmd);
+                    result.AppendMessageWithFormat ("Command #%zu '%s' continued the target.\n", idx + 1, cmd);
                     
                 result.SetStatus(tmp_result.GetStatus());
                 m_debugger.SetAsyncExecution (old_async_execution);

Modified: lldb/trunk/source/Interpreter/OptionValueFileSpecLIst.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/OptionValueFileSpecLIst.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/OptionValueFileSpecLIst.cpp (original)
+++ lldb/trunk/source/Interpreter/OptionValueFileSpecLIst.cpp Wed Jun 19 14:04:53 2013
@@ -150,7 +150,7 @@ OptionValueFileSpecList::SetValueFromCSt
                     {
                         // Sort and then erase in reverse so indexes are always valid
                         std::sort(remove_indexes.begin(), remove_indexes.end());
-                        for (int j=num_remove_indexes-1; j<num_remove_indexes; ++j)
+                        for (size_t j=num_remove_indexes-1; j<num_remove_indexes; ++j)
                         {
                             m_current_value.Remove (j);
                         }

Modified: lldb/trunk/source/Interpreter/OptionValuePathMappings.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/OptionValuePathMappings.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/OptionValuePathMappings.cpp (original)
+++ lldb/trunk/source/Interpreter/OptionValuePathMappings.cpp Wed Jun 19 14:04:53 2013
@@ -151,7 +151,7 @@ OptionValuePathMappings::SetValueFromCSt
                     {
                         // Sort and then erase in reverse so indexes are always valid
                         std::sort(remove_indexes.begin(), remove_indexes.end());
-                        for (int j=num_remove_indexes-1; j<num_remove_indexes; ++j)
+                        for (size_t j=num_remove_indexes-1; j<num_remove_indexes; ++j)
                         {
                             m_path_mappings.Remove (j, m_notify_changes);
                         }

Modified: lldb/trunk/source/Interpreter/Options.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/Options.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/Options.cpp (original)
+++ lldb/trunk/source/Interpreter/Options.cpp Wed Jun 19 14:04:53 2013
@@ -206,7 +206,7 @@ Options::BuildValidOptionSets ()
         }
         else
         {
-            for (int j = 0; j < LLDB_MAX_NUM_OPTION_SETS; j++)
+            for (uint32_t j = 0; j < LLDB_MAX_NUM_OPTION_SETS; j++)
             {
                 if (this_usage_mask & (1 << j))
                 {
@@ -224,7 +224,7 @@ Options::BuildValidOptionSets ()
         
         for (int i = 0; i < num_options; ++i)
         {
-            for (int j = 0; j < num_option_sets; j++)
+            for (uint32_t j = 0; j < num_option_sets; j++)
             {
                 if (opt_defs[i].usage_mask & 1 << j)
                 {
@@ -759,7 +759,7 @@ Options::HandleOptionCompletion
     cur_opt_std_str.erase(char_pos);
     const char *cur_opt_str = cur_opt_std_str.c_str();
 
-    for (int i = 0; i < opt_element_vector.size(); i++)
+    for (size_t i = 0; i < opt_element_vector.size(); i++)
     {
         int opt_pos = opt_element_vector[i].opt_pos;
         int opt_arg_pos = opt_element_vector[i].opt_arg_pos;
@@ -833,7 +833,7 @@ Options::HandleOptionCompletion
                             // The options definitions table has duplicates because of the
                             // way the grouping information is stored, so only add once.
                             bool duplicate = false;
-                            for (int k = 0; k < matches.GetSize(); k++)
+                            for (size_t k = 0; k < matches.GetSize(); k++)
                             {
                                 if (matches.GetStringAtIndex(k) == full_name)
                                 {
@@ -944,7 +944,7 @@ Options::HandleOptionArgumentCompletion
     if (completion_mask & CommandCompletions::eSourceFileCompletion
         || completion_mask & CommandCompletions::eSymbolCompletion)
     {
-        for (int i = 0; i < opt_element_vector.size(); i++)
+        for (size_t i = 0; i < opt_element_vector.size(); i++)
         {
             int cur_defs_index = opt_element_vector[i].opt_defs_index;
             int cur_arg_pos    = opt_element_vector[i].opt_arg_pos;

Modified: lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp (original)
+++ lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp Wed Jun 19 14:04:53 2013
@@ -902,7 +902,7 @@ DynamicLoaderMacOSXDYLD::AddModulesUsing
         if (objc_runtime != NULL && !objc_runtime->HasReadObjCLibrary())
         {
             size_t num_modules = loaded_module_list.GetSize();
-            for (int i = 0; i < num_modules; i++)
+            for (size_t i = 0; i < num_modules; i++)
             {
                 if (objc_runtime->IsModuleObjCLibrary (loaded_module_list.GetModuleAtIndex (i)))
                 {
@@ -1031,7 +1031,7 @@ DynamicLoaderMacOSXDYLD::ReadImageInfos
     {
         lldb::offset_t info_data_offset = 0;
         DataExtractor info_data_ref(info_data.GetBytes(), info_data.GetByteSize(), endian, addr_size);
-        for (int i = 0; i < image_infos.size() && info_data_ref.ValidOffset(info_data_offset); i++)
+        for (size_t i = 0; i < image_infos.size() && info_data_ref.ValidOffset(info_data_offset); i++)
         {
             image_infos[i].address = info_data_ref.GetPointer(&info_data_offset);
             lldb::addr_t path_addr = info_data_ref.GetPointer(&info_data_offset);

Modified: lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV1.cpp Wed Jun 19 14:04:53 2013
@@ -168,7 +168,7 @@ AppleObjCRuntimeV1::CreateObjectChecker(
                     "   struct __objc_object *obj = (struct __objc_object*)$__lldb_arg_obj; \n"
                     "   (int)strlen(obj->isa->name);                                        \n"
                     "}                                                                      \n",
-                    name) < sizeof(buf->contents));
+                    name) < (int)sizeof(buf->contents));
 
     return new ClangUtilityFunction(buf->contents, name);
 }

Modified: lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCRuntimeV2.cpp Wed Jun 19 14:04:53 2013
@@ -529,7 +529,7 @@ AppleObjCRuntimeV2::CreateObjectChecker(
                           name);
     }
     
-    assert (len < sizeof(check_function_code));
+    assert (len < (int)sizeof(check_function_code));
 
     return new ClangUtilityFunction(check_function_code, name);
 }

Modified: lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp (original)
+++ lldb/trunk/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleObjCTrampolineHandler.cpp Wed Jun 19 14:04:53 2013
@@ -344,7 +344,7 @@ AppleObjCTrampolineHandler::AppleObjCVTa
     m_code_start_addr = 0;
     m_code_end_addr = 0;
 
-    for (int i = 0; i < num_descriptors; i++)
+    for (size_t i = 0; i < num_descriptors; i++)
     {
         lldb::addr_t start_offset = offset;
         uint32_t voffset = desc_extractor.GetU32 (&offset);
@@ -363,7 +363,7 @@ AppleObjCTrampolineHandler::AppleObjCVTa
     // Let's compute the blocks and if they are all the same add the size to the code end address:
     lldb::addr_t code_size = 0;
     bool all_the_same = true;
-    for (int i = 0; i < num_descriptors - 1; i++)
+    for (size_t i = 0; i < num_descriptors - 1; i++)
     {
         lldb::addr_t this_size = m_descriptors[i + 1].code_start - m_descriptors[i].code_start;
         if (code_size == 0)
@@ -686,7 +686,7 @@ AppleObjCTrampolineHandler::AppleObjCTra
     // turn the g_dispatch_functions char * array into a template table, and populate the DispatchFunction map
     // from there.
 
-    for (int i = 0; i != llvm::array_lengthof(g_dispatch_functions); i++)
+    for (size_t i = 0; i != llvm::array_lengthof(g_dispatch_functions); i++)
     {
         ConstString name_const_str(g_dispatch_functions[i].name);
         const Symbol *msgSend_symbol = m_objc_module_sp->FindFirstSymbolWithNameAndType (name_const_str, eSymbolTypeCode);

Modified: lldb/trunk/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp (original)
+++ lldb/trunk/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp Wed Jun 19 14:04:53 2013
@@ -984,7 +984,7 @@ PlatformDarwin::GetDeveloperDirectory()
                 {
                     const char *cmd_output_ptr = command_output.c_str();
                     developer_dir_path[sizeof (developer_dir_path) - 1] = '\0';
-                    int i;
+                    size_t i;
                     for (i = 0; i < sizeof (developer_dir_path) - 1; i++)
                     {
                         if (cmd_output_ptr[i] == '\r' || cmd_output_ptr[i] == '\n' || cmd_output_ptr[i] == '\0')
@@ -1043,7 +1043,7 @@ PlatformDarwin::SetThreadCreationBreakpo
     };
 
     FileSpecList bp_modules;
-    for (int i = 0; i < sizeof(g_bp_modules)/sizeof(const char *); i++)
+    for (size_t i = 0; i < sizeof(g_bp_modules)/sizeof(const char *); i++)
     {
         const char *bp_module = g_bp_modules[i];
         bp_modules.Append(FileSpec(bp_module, false));

Modified: lldb/trunk/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp (original)
+++ lldb/trunk/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp Wed Jun 19 14:04:53 2013
@@ -399,7 +399,7 @@ PlatformRemoteGDBServer::Attach (lldb_pr
                                                                 "connect://%s:%u", 
                                                                 GetHostname (), 
                                                                 port);
-                        assert (connect_url_len < sizeof(connect_url));
+                        assert (connect_url_len < (int)sizeof(connect_url));
                         error = process_sp->ConnectRemote (NULL, connect_url);
                         if (error.Success())
                             error = process_sp->Attach(attach_info);

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp Wed Jun 19 14:04:53 2013
@@ -1328,7 +1328,7 @@ GDBRemoteCommunicationClient::SendAttach
     {
         char packet[64];
         const int packet_len = ::snprintf (packet, sizeof(packet), "vAttach;%" PRIx64, pid);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
             if (response.IsErrorResponse())
@@ -1359,7 +1359,7 @@ GDBRemoteCommunicationClient::AllocateMe
                                            permissions & lldb::ePermissionsReadable ? "r" : "",
                                            permissions & lldb::ePermissionsWritable ? "w" : "",
                                            permissions & lldb::ePermissionsExecutable ? "x" : "");
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -1382,7 +1382,7 @@ GDBRemoteCommunicationClient::Deallocate
         m_supports_alloc_dealloc_memory = eLazyBoolYes;
         char packet[64];
         const int packet_len = ::snprintf(packet, sizeof(packet), "_m%" PRIx64, (uint64_t)addr);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -1408,7 +1408,7 @@ GDBRemoteCommunicationClient::Detach (bo
         {
             char packet[64];
             const int packet_len = ::snprintf(packet, sizeof(packet), "qSupportsDetachAndStayStopped:");
-            assert (packet_len < sizeof(packet));
+            assert (packet_len < (int)sizeof(packet));
             StringExtractorGDBRemote response;
             if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
             {
@@ -1453,7 +1453,7 @@ GDBRemoteCommunicationClient::GetMemoryR
         m_supports_memory_region_info = eLazyBoolYes;
         char packet[64];
         const int packet_len = ::snprintf(packet, sizeof(packet), "qMemoryRegionInfo:%" PRIx64, (uint64_t)addr);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -1556,7 +1556,7 @@ GDBRemoteCommunicationClient::GetWatchpo
     {
         char packet[64];
         const int packet_len = ::snprintf(packet, sizeof(packet), "qWatchpointSupportInfo:");
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -1703,7 +1703,7 @@ GDBRemoteCommunicationClient::SetDisable
 {
     char packet[32];
     const int packet_len = ::snprintf (packet, sizeof (packet), "QSetDisableASLR:%i", enable ? 1 : 0);
-    assert (packet_len < sizeof(packet));
+    assert (packet_len < (int)sizeof(packet));
     StringExtractorGDBRemote response;
     if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
     {
@@ -1786,7 +1786,7 @@ GDBRemoteCommunicationClient::GetProcess
     {
         char packet[32];
         const int packet_len = ::snprintf (packet, sizeof (packet), "qProcessInfoPID:%" PRIu64, pid);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -2001,7 +2001,7 @@ GDBRemoteCommunicationClient::GetUserNam
     {
         char packet[32];
         const int packet_len = ::snprintf (packet, sizeof (packet), "qUserName:%i", uid);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -2031,7 +2031,7 @@ GDBRemoteCommunicationClient::GetGroupNa
     {
         char packet[32];
         const int packet_len = ::snprintf (packet, sizeof (packet), "qGroupName:%i", gid);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (SendPacketAndWaitForResponse (packet, packet_len, response, false))
         {
@@ -2165,7 +2165,7 @@ GDBRemoteCommunicationClient::SetCurrent
         packet_len = ::snprintf (packet, sizeof(packet), "Hg-1");
     else
         packet_len = ::snprintf (packet, sizeof(packet), "Hg%" PRIx64, tid);
-    assert (packet_len + 1 < sizeof(packet));
+    assert (packet_len + 1 < (int)sizeof(packet));
     StringExtractorGDBRemote response;
     if (SendPacketAndWaitForResponse(packet, packet_len, response, false))
     {
@@ -2191,7 +2191,7 @@ GDBRemoteCommunicationClient::SetCurrent
     else
         packet_len = ::snprintf (packet, sizeof(packet), "Hc%" PRIx64, tid);
 
-    assert (packet_len + 1 < sizeof(packet));
+    assert (packet_len + 1 < (int)sizeof(packet));
     StringExtractorGDBRemote response;
     if (SendPacketAndWaitForResponse(packet, packet_len, response, false))
     {
@@ -2219,7 +2219,7 @@ GDBRemoteCommunicationClient::GetThreadS
     {
         char packet[256];
         int packet_len = ::snprintf(packet, sizeof(packet), "qThreadStopInfo%" PRIx64, tid);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         if (SendPacketAndWaitForResponse(packet, packet_len, response, false))
         {
             if (response.IsNormalResponse())
@@ -2259,7 +2259,7 @@ GDBRemoteCommunicationClient::SendGDBSto
                                        addr, 
                                        length);
 
-    assert (packet_len + 1 < sizeof(packet));
+    assert (packet_len + 1 < (int)sizeof(packet));
     StringExtractorGDBRemote response;
     if (SendPacketAndWaitForResponse(packet, packet_len, response, true))
     {

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServer.cpp Wed Jun 19 14:04:53 2013
@@ -186,7 +186,7 @@ GDBRemoteCommunicationServer::SendErrorR
 {
     char packet[16];
     int packet_len = ::snprintf (packet, sizeof(packet), "E%2.2x", err);
-    assert (packet_len < sizeof(packet));
+    assert (packet_len < (int)sizeof(packet));
     return SendPacketNoLock (packet, packet_len);
 }
 
@@ -713,7 +713,7 @@ GDBRemoteCommunicationServer::Handle_qLa
                             uint16_t port = (intptr_t)accept_thread_result;
                             char response[256];
                             const int response_len = ::snprintf (response, sizeof(response), "pid:%" PRIu64 ";port:%u;", debugserver_pid, port);
-                            assert (response_len < sizeof(response));
+                            assert (response_len < (int)sizeof(response));
                             //m_port_to_pid_map[port] = debugserver_launch_info.GetProcessID();
                             success = SendPacketNoLock (response, response_len) > 0;
                         }

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp Wed Jun 19 14:04:53 2013
@@ -158,7 +158,7 @@ GDBRemoteRegisterContext::GetPrimordialR
         packet_len = ::snprintf (packet, sizeof(packet), "p%x;thread:%4.4" PRIx64 ";", reg, m_thread.GetProtocolID());
     else
         packet_len = ::snprintf (packet, sizeof(packet), "p%x", reg);
-    assert (packet_len < (sizeof(packet) - 1));
+    assert (packet_len < ((int)sizeof(packet) - 1));
     if (gdb_comm.SendPacketAndWaitForResponse(packet, response, false))
         return PrivateSetRegisterValue (reg, response);
 
@@ -199,7 +199,7 @@ GDBRemoteRegisterContext::ReadRegisterBy
                         packet_len = ::snprintf (packet, sizeof(packet), "g;thread:%4.4" PRIx64 ";", m_thread.GetProtocolID());
                     else
                         packet_len = ::snprintf (packet, sizeof(packet), "g");
-                    assert (packet_len < (sizeof(packet) - 1));
+                    assert (packet_len < ((int)sizeof(packet) - 1));
                     if (gdb_comm.SendPacketAndWaitForResponse(packet, response, false))
                     {
                         if (response.IsNormalResponse())
@@ -515,7 +515,7 @@ GDBRemoteRegisterContext::ReadAllRegiste
                 packet_len = ::snprintf (packet, sizeof(packet), "g;thread:%4.4" PRIx64, m_thread.GetProtocolID());
             else
                 packet_len = ::snprintf (packet, sizeof(packet), "g");
-            assert (packet_len < (sizeof(packet) - 1));
+            assert (packet_len < ((int)sizeof(packet) - 1));
 
             if (gdb_comm.SendPacketAndWaitForResponse(packet, packet_len, response, false))
             {

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp Wed Jun 19 14:04:53 2013
@@ -260,7 +260,7 @@ ProcessGDBRemote::BuildDynamicRegisterIn
          ++reg_num)
     {
         const int packet_len = ::snprintf (packet, sizeof(packet), "qRegisterInfo%x", reg_num);
-        assert (packet_len < sizeof(packet));
+        assert (packet_len < (int)sizeof(packet));
         StringExtractorGDBRemote response;
         if (m_gdb_comm.SendPacketAndWaitForResponse(packet, packet_len, response, false))
         {
@@ -1942,7 +1942,7 @@ ProcessGDBRemote::DoReadMemory (addr_t a
 
     char packet[64];
     const int packet_len = ::snprintf (packet, sizeof(packet), "m%" PRIx64 ",%" PRIx64, (uint64_t)addr, (uint64_t)size);
-    assert (packet_len + 1 < sizeof(packet));
+    assert (packet_len + 1 < (int)sizeof(packet));
     StringExtractorGDBRemote response;
     if (m_gdb_comm.SendPacketAndWaitForResponse(packet, packet_len, response, true))
     {

Modified: lldb/trunk/source/Symbol/ClangASTContext.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Symbol/ClangASTContext.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Symbol/ClangASTContext.cpp (original)
+++ lldb/trunk/source/Symbol/ClangASTContext.cpp Wed Jun 19 14:04:53 2013
@@ -1874,7 +1874,7 @@ ClangASTContext::AddMethodToCXXRecordTyp
     
     llvm::SmallVector<ParmVarDecl *, 12> params;
     
-    for (int param_index = 0;
+    for (unsigned param_index = 0;
          param_index < num_params;
          ++param_index)
     {
@@ -2817,7 +2817,7 @@ ClangASTContext::AddMethodToObjCObjectTy
     {
         llvm::SmallVector<ParmVarDecl *, 12> params;
             
-        for (int param_index = 0; param_index < num_args; ++param_index)
+        for (unsigned param_index = 0; param_index < num_args; ++param_index)
         {
             params.push_back (ParmVarDecl::Create (*ast,
                                                    objc_method_decl,
@@ -3920,10 +3920,10 @@ ClangASTContext::GetIndexOfFieldWithName
                                           uint32_t *bitfield_bit_size_ptr,
                                           bool *is_bitfield_ptr)
 {
-    auto count = ClangASTContext::GetNumFields(ast, clang_type);
+    unsigned count = ClangASTContext::GetNumFields(ast, clang_type);
     lldb::clang_type_t field_clang_type_internal;
     std::string field_name;
-    for (auto index = 0; index < count; index++)
+    for (unsigned index = 0; index < count; index++)
     {
         field_clang_type_internal = ClangASTContext::GetFieldAtIndex(ast, clang_type, index, field_name, bit_offset_ptr, bitfield_bit_size_ptr, is_bitfield_ptr);
         if ( strcmp(field_name.c_str(), name) == 0 )

Modified: lldb/trunk/source/Symbol/Symtab.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Symbol/Symtab.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Symbol/Symtab.cpp (original)
+++ lldb/trunk/source/Symbol/Symtab.cpp Wed Jun 19 14:04:53 2013
@@ -721,7 +721,7 @@ Symtab::AppendSymbolIndexesMatchingRegEx
     uint32_t prev_size = indexes.size();
     uint32_t sym_end = m_symbols.size();
 
-    for (int i = 0; i < sym_end; i++)
+    for (uint32_t i = 0; i < sym_end; i++)
     {
         if (symbol_type == eSymbolTypeAny || m_symbols[i].GetType() == symbol_type)
         {
@@ -745,7 +745,7 @@ Symtab::AppendSymbolIndexesMatchingRegEx
     uint32_t prev_size = indexes.size();
     uint32_t sym_end = m_symbols.size();
 
-    for (int i = 0; i < sym_end; i++)
+    for (uint32_t i = 0; i < sym_end; i++)
     {
         if (symbol_type == eSymbolTypeAny || m_symbols[i].GetType() == symbol_type)
         {

Modified: lldb/trunk/source/Target/Process.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/Process.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Target/Process.cpp (original)
+++ lldb/trunk/source/Target/Process.cpp Wed Jun 19 14:04:53 2013
@@ -3226,7 +3226,7 @@ Process::CompleteAttach ()
     size_t num_modules = target_modules.GetSize();
     ModuleSP new_executable_module_sp;
     
-    for (int i = 0; i < num_modules; i++)
+    for (size_t i = 0; i < num_modules; i++)
     {
         ModuleSP module_sp (target_modules.GetModuleAtIndexUnlocked (i));
         if (module_sp && module_sp->IsExecutable())

Modified: lldb/trunk/source/Target/Target.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/Target.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Target/Target.cpp (original)
+++ lldb/trunk/source/Target/Target.cpp Wed Jun 19 14:04:53 2013
@@ -1195,7 +1195,7 @@ Target::ModuleIsExcludedForNonModuleSpec
         // black list.
         if (num_modules > 0)
         {
-            for (int i  = 0; i < num_modules; i++)
+            for (size_t i  = 0; i < num_modules; i++)
             {
                 if (!ModuleIsExcludedForNonModuleSpecificSearches (matchingModules.GetModuleAtIndex(i)))
                     return false;

Modified: lldb/trunk/source/Target/ThreadPlanStepUntil.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/ThreadPlanStepUntil.cpp?rev=184333&r1=184332&r2=184333&view=diff
==============================================================================
--- lldb/trunk/source/Target/ThreadPlanStepUntil.cpp (original)
+++ lldb/trunk/source/Target/ThreadPlanStepUntil.cpp Wed Jun 19 14:04:53 2013
@@ -80,7 +80,7 @@ ThreadPlanStepUntil::ThreadPlanStepUntil
         m_stack_id = m_thread.GetStackFrameAtIndex(frame_idx)->GetStackID();
 
         // Now set breakpoints on all our return addresses:
-        for (int i = 0; i < num_addresses; i++)
+        for (size_t i = 0; i < num_addresses; i++)
         {
             Breakpoint *until_bp = target_sp->CreateBreakpoint (address_list[i], true).get();
             if (until_bp != NULL)





More information about the lldb-commits mailing list