[Lldb-commits] [PATCH] D15715: Don't pack the structs of the jit debug interface

Tamas Berghammer via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 22 06:51:04 PST 2015

tberghammer created this revision.
tberghammer added reviewers: tfiala, endlessroad, loladiro.
tberghammer added a subscriber: lldb-commits.

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.

LLVM implementation: https://github.com/llvm-mirror/llvm/blob/master/lib/ExecutionEngine/GDBRegistrationListener.cpp



Index: source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp
--- source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp
+++ source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp
@@ -90,7 +90,6 @@
 } jit_actions_t;
-#pragma pack(push, 4)
 template <typename ptr_t>
 struct jit_code_entry
@@ -107,7 +106,6 @@
     ptr_t    relevant_entry; // pointer
     ptr_t    first_entry; // pointer
-#pragma pack(pop)
 JITLoaderGDB::JITLoaderGDB (lldb_private::Process *process) :

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15715.43444.patch
Type: text/x-patch
Size: 570 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151222/e307e11a/attachment.bin>

More information about the lldb-commits mailing list