warnings: Fix a bunch of -Wreorder problems.

llvm-svn: 143381

lldb/include/lldb/Expression/ClangASTSource.h

@@ -44,11 +44,11 @@ public:
     ///     A reference to the LLDB object that handles entity lookup.
     //------------------------------------------------------------------
 	ClangASTSource (const lldb::TargetSP &target) :
-        m_ast_context (NULL),
-        m_active_lookups (),
         m_import_in_progress (false),
         m_lookups_enabled (false),
-        m_target (target)
+        m_target (target),
+        m_ast_context (NULL),
+        m_active_lookups ()
     {
     }
     

lldb/include/lldb/Expression/ClangExpressionDeclMap.h

@@ -665,11 +665,11 @@ private:
     {
     public:
         ParserVars(ClangExpressionDeclMap &decl_map) :
-            m_decl_map(decl_map),
             m_exe_ctx(NULL),
             m_sym_ctx(),
             m_persistent_vars(NULL),
-            m_enable_lookups(false)
+            m_enable_lookups(false),
+            m_decl_map(decl_map)
         {
         }
         

lldb/source/Commands/CommandObjectTarget.cpp

@@ -2618,8 +2618,8 @@ public:
         
         CommandOptions (CommandInterpreter &interpreter) :
             Options(interpreter),
-            m_use_global_module_list (false),
             m_format_array(),
+            m_use_global_module_list (false),
             m_module_addr (LLDB_INVALID_ADDRESS)
         {
         }

lldb/source/Core/RegularExpression.cpp

@@ -32,8 +32,8 @@ RegularExpression::RegularExpression() :
 RegularExpression::RegularExpression(const char* re, int flags) :
     m_re(),
     m_comp_err (1),
-    m_compile_flags(flags),
+    m_preg(),
-    m_preg()
+    m_compile_flags(flags)
 {
     memset(&m_preg,0,sizeof(m_preg));
     Compile(re);
@@ -46,8 +46,8 @@ RegularExpression::RegularExpression(const char* re, int flags) :
 RegularExpression::RegularExpression(const char* re) :
     m_re(),
     m_comp_err (1),
-    m_compile_flags(REG_EXTENDED),
+    m_preg(),
-    m_preg()
+    m_compile_flags(REG_EXTENDED)
 {
     memset(&m_preg,0,sizeof(m_preg));
     Compile(re);

lldb/source/Core/ValueObject.cpp

@@ -85,6 +85,7 @@ ValueObject::ValueObject (ValueObject &parent) :
     m_last_value_format(),
     m_last_synthetic_filter(),
     m_user_id_of_forced_summary(),
+    m_address_type_of_ptr_or_ref_children(eAddressTypeInvalid),
     m_value_is_valid (false),
     m_value_did_change (false),
     m_children_count_valid (false),
@@ -94,7 +95,6 @@ ValueObject::ValueObject (ValueObject &parent) :
     m_is_bitfield_for_scalar(false),
     m_is_expression_path_child(false),
     m_is_child_at_offset(false),
-    m_address_type_of_ptr_or_ref_children(eAddressTypeInvalid),
     m_trying_summary_already(false)
 {
     m_manager->ManageObject(this);
@@ -131,6 +131,7 @@ ValueObject::ValueObject (ExecutionContextScope *exe_scope,
     m_last_value_format(),
     m_last_synthetic_filter(),
     m_user_id_of_forced_summary(),
+    m_address_type_of_ptr_or_ref_children(child_ptr_or_ref_addr_type),
     m_value_is_valid (false),
     m_value_did_change (false),
     m_children_count_valid (false),
@@ -140,7 +141,6 @@ ValueObject::ValueObject (ExecutionContextScope *exe_scope,
     m_is_bitfield_for_scalar(false),
     m_is_expression_path_child(false),
     m_is_child_at_offset(false),
-    m_address_type_of_ptr_or_ref_children(child_ptr_or_ref_addr_type),
     m_trying_summary_already(false)
 {
     m_manager = new ValueObjectManager();

lldb/source/Expression/ASTResultSynthesizer.cpp

@@ -34,10 +34,10 @@ ASTResultSynthesizer::ASTResultSynthesizer(ASTConsumer *passthrough,
     m_ast_context (NULL),
     m_passthrough (passthrough),
     m_passthrough_sema (NULL),
-    m_sema (NULL),
-    m_desired_type (desired_type),
     m_scratch_ast_context (scratch_ast_context),
-    m_persistent_vars (persistent_vars)
+    m_persistent_vars (persistent_vars),
+    m_sema (NULL),
+    m_desired_type (desired_type)
 {
     if (!m_passthrough)
         return;

lldb/source/Expression/ClangUserExpression.cpp

@@ -54,9 +54,9 @@ ClangUserExpression::ClangUserExpression (const char *expr,
     m_objectivec (false),
     m_needs_object_ptr (false),
     m_const_object (false),
+    m_target (NULL),
     m_evaluated_statically (false),
-    m_const_result (),
+    m_const_result ()
-    m_target (NULL)
 {
 }
 

lldb/source/Expression/IRInterpreter.cpp

@@ -419,8 +419,8 @@ public:
     InterpreterStackFrame (TargetData &target_data,
                            Memory &memory,
                            lldb_private::ClangExpressionDeclMap &decl_map) :
-        m_target_data (target_data),
         m_memory (memory),
+        m_target_data (target_data),
         m_decl_map (decl_map)
     {
         m_byte_order = (target_data.isLittleEndian() ? lldb::eByteOrderLittle : lldb::eByteOrderBig);

lldb/source/Plugins/Disassembler/llvm/DisassemblerLLVM.cpp

@@ -80,8 +80,8 @@ InstructionLLVM::InstructionLLVM (const Address &addr,
                                   llvm::Triple::ArchType arch_type) :
     Instruction (addr, addr_class),
     m_disassembler (disassembler),
-    m_arch_type (arch_type),
+    m_inst (NULL),
-    m_inst (NULL)
+    m_arch_type (arch_type)
 {
 }
 

lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp

@@ -94,9 +94,9 @@ DynamicLoaderDarwinKernel::DynamicLoaderDarwinKernel (Process* process) :
     m_kext_summary_header_ptr_addr (),
     m_kext_summary_header_addr (),
     m_kext_summary_header (),
-    m_break_id (LLDB_INVALID_BREAK_ID),
     m_kext_summaries(),
-    m_mutex(Mutex::eMutexTypeRecursive)
+    m_mutex(Mutex::eMutexTypeRecursive),
+    m_break_id (LLDB_INVALID_BREAK_ID)
 {
 }
 

lldb/tools/driver/Driver.cpp

@@ -378,10 +378,10 @@ Driver::OptionData::OptionData () :
     m_debug_mode (false),
     m_print_version (false),
     m_print_help (false),
-    m_use_external_editor(false),
     m_wait_for(false),
     m_process_name(),
     m_process_pid(LLDB_INVALID_PROCESS_ID),
+    m_use_external_editor(false),
     m_seen_options()
 {
 }