clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

Just move the eSectionTypeDWARFGNUDebugAltLink enumerator to the end of the 
enum and this is good to go.



================
Comment at: include/lldb/lldb-enumerations.h:647
   eSectionTypeDWARFAppleObjC,
+  eSectionTypeDWARFGNUDebugAltLink,
   eSectionTypeELFSymbolTable,       // Elf SHT_SYMTAB section
----------------
Since this enumeration is in the public API, it should be added to the end of 
the enum. Anyone that builds a new liblldb.so and then uses it with an binary 
compiled before could run into issues. But I noticed 
eSectionTypeDWARFDebugCuIndex was added in the middle in August, but we need to 
avoid these kinds of API issues, so lets add this to the end just to be safe.


https://reviews.llvm.org/D40468



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

Reply via email to