aprantl added inline comments.
================
Comment at:
lldb/source/Plugins/ObjectContainer/Universal-Mach-O/ObjectContainerUniversalMachO.cpp:205
+ const lldb::offset_t slice_file_offset =
+ fat_arch.GetOffset() + file_offset;
+ if (fat_arch.GetOffset() < file_size && file_size > slice_file_offset)
{
----------------
Should this use AddOverflow?
https://llvm.org/doxygen/MathExtras_8h.html
================
Comment at: lldb/test/API/macosx/universal64/TestUniversal64.py:21
+ # Make sure the binary and the dSYM are in the image list.
+ self.expect("image list ", patterns=['fat.out', 'fat.out.dSYM'])
+
----------------
would it be easy to verify in the test or the makefile that the dsym indeed has
a fat header?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147012/new/
https://reviews.llvm.org/D147012
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
