[Lldb-commits] [lldb] r256244 - Don't pack the structs of the jit debug interface

Tamas Berghammer via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 22 07:23:33 PST 2015


Author: tberghammer
Date: Tue Dec 22 09:23:33 2015
New Revision: 256244

URL: http://llvm.org/viewvc/llvm-project?rev=256244&view=rev
Log:
Don't pack the structs of the jit debug interface

None of the documentation mentions that the entries are packed structs
and also none of the other implementation I found pack them.

Differential revision: http://reviews.llvm.org/D15715

Modified:
    lldb/trunk/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp

Modified: lldb/trunk/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp?rev=256244&r1=256243&r2=256244&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp (original)
+++ lldb/trunk/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp Tue Dec 22 09:23:33 2015
@@ -90,7 +90,6 @@ typedef enum
     JIT_UNREGISTER_FN
 } jit_actions_t;
 
-#pragma pack(push, 4)
 template <typename ptr_t>
 struct jit_code_entry
 {
@@ -107,7 +106,6 @@ struct jit_descriptor
     ptr_t    relevant_entry; // pointer
     ptr_t    first_entry; // pointer
 };
-#pragma pack(pop)
 
 JITLoaderGDB::JITLoaderGDB (lldb_private::Process *process) :
     JITLoader(process),




More information about the lldb-commits mailing list