Commit 5c1ec20c9a

Andrew Kelley <superjoe30@gmail.com>
2018-08-23 22:11:34
MacOS stack traces use the already mmapped executable
...rather than trying to find the executable on the file system. Also use a more robust PIE offset calculation based on the available metadata. And for the last function, use the data that tells the end rather than assuming 4K. Also they print in a consistent way with Linux stack traces.
1 parent 8f96553
Changed files (4)
std/c/darwin.zig
@@ -1,5 +1,6 @@
 extern "c" fn __error() *c_int;
 pub extern "c" fn _NSGetExecutablePath(buf: [*]u8, bufsize: *u32) c_int;
+pub extern "c" fn _dyld_get_image_header(image_index: u32) ?*mach_header;
 
 pub extern "c" fn __getdirentries64(fd: c_int, buf_ptr: [*]u8, buf_len: usize, basep: *i64) usize;
 
@@ -33,6 +34,12 @@ pub extern "c" fn sysctlnametomib(name: [*]const u8, mibp: ?*c_int, sizep: ?*usi
 pub extern "c" fn bind(socket: c_int, address: ?*const sockaddr, address_len: socklen_t) c_int;
 pub extern "c" fn socket(domain: c_int, type: c_int, protocol: c_int) c_int;
 
+/// The value of the link editor defined symbol _MH_EXECUTE_SYM is the address
+/// of the mach header in a Mach-O executable file type.  It does not appear in
+/// any file type other than a MH_EXECUTE file type.  The type of the symbol is
+/// absolute as the header is not part of any section.
+pub extern "c" var _mh_execute_header: if (@sizeOf(usize) == 8) mach_header_64 else mach_header;
+
 pub use @import("../os/darwin/errno.zig");
 
 pub const _errno = __error;
@@ -139,6 +146,353 @@ pub const Kevent = extern struct {
     udata: usize,
 };
 
+pub const mach_header = extern struct {
+    magic: u32,
+    cputype: cpu_type_t,
+    cpusubtype: cpu_subtype_t,
+    filetype: u32,
+    ncmds: u32,
+    sizeofcmds: u32,
+    flags: u32,
+};
+
+pub const mach_header_64 = extern struct {
+    magic: u32,
+    cputype: cpu_type_t,
+    cpusubtype: cpu_subtype_t,
+    filetype: u32,
+    ncmds: u32,
+    sizeofcmds: u32,
+    flags: u32,
+    reserved: u32,
+};
+
+pub const load_command = extern struct {
+    cmd: u32,
+    cmdsize: u32,
+};
+
+
+/// The symtab_command contains the offsets and sizes of the link-edit 4.3BSD
+/// "stab" style symbol table information as described in the header files
+/// <nlist.h> and <stab.h>.
+pub const symtab_command = extern struct {
+    cmd: u32, /// LC_SYMTAB
+    cmdsize: u32, /// sizeof(struct symtab_command)
+    symoff: u32, /// symbol table offset
+    nsyms: u32, /// number of symbol table entries
+    stroff: u32, /// string table offset
+    strsize: u32, /// string table size in bytes
+};
+
+/// The linkedit_data_command contains the offsets and sizes of a blob
+/// of data in the __LINKEDIT segment.  
+const linkedit_data_command = extern struct {
+    cmd: u32,/// LC_CODE_SIGNATURE, LC_SEGMENT_SPLIT_INFO, LC_FUNCTION_STARTS, LC_DATA_IN_CODE, LC_DYLIB_CODE_SIGN_DRS or LC_LINKER_OPTIMIZATION_HINT.
+    cmdsize: u32, /// sizeof(struct linkedit_data_command)
+    dataoff: u32 , /// file offset of data in __LINKEDIT segment
+    datasize: u32 , /// file size of data in __LINKEDIT segment 
+};
+
+/// The segment load command indicates that a part of this file is to be
+/// mapped into the task's address space.  The size of this segment in memory,
+/// vmsize, maybe equal to or larger than the amount to map from this file,
+/// filesize.  The file is mapped starting at fileoff to the beginning of
+/// the segment in memory, vmaddr.  The rest of the memory of the segment,
+/// if any, is allocated zero fill on demand.  The segment's maximum virtual
+/// memory protection and initial virtual memory protection are specified
+/// by the maxprot and initprot fields.  If the segment has sections then the
+/// section structures directly follow the segment command and their size is
+/// reflected in cmdsize.
+pub const segment_command = extern struct {
+    cmd: u32,/// LC_SEGMENT
+    cmdsize: u32,/// includes sizeof section structs
+    segname: [16]u8,/// segment name
+    vmaddr: u32,/// memory address of this segment
+    vmsize: u32,/// memory size of this segment
+    fileoff: u32,/// file offset of this segment
+    filesize: u32,/// amount to map from the file
+    maxprot: vm_prot_t,/// maximum VM protection
+    initprot: vm_prot_t,/// initial VM protection
+    nsects: u32,/// number of sections in segment
+    flags: u32,
+};
+
+/// The 64-bit segment load command indicates that a part of this file is to be
+/// mapped into a 64-bit task's address space.  If the 64-bit segment has
+/// sections then section_64 structures directly follow the 64-bit segment
+/// command and their size is reflected in cmdsize.
+pub const segment_command_64 = extern struct {
+    cmd: u32,               /// LC_SEGMENT_64
+    cmdsize: u32,           /// includes sizeof section_64 structs
+    segname: [16]u8,        /// segment name
+    vmaddr: u64,            /// memory address of this segment
+    vmsize: u64,            /// memory size of this segment
+    fileoff: u64,           /// file offset of this segment
+    filesize: u64,          /// amount to map from the file
+    maxprot: vm_prot_t,     /// maximum VM protection
+    initprot: vm_prot_t,    /// initial VM protection
+    nsects: u32,            /// number of sections in segment
+    flags: u32,             
+};
+
+/// A segment is made up of zero or more sections.  Non-MH_OBJECT files have
+/// all of their segments with the proper sections in each, and padded to the
+/// specified segment alignment when produced by the link editor.  The first
+/// segment of a MH_EXECUTE and MH_FVMLIB format file contains the mach_header
+/// and load commands of the object file before its first section.  The zero
+/// fill sections are always last in their segment (in all formats).  This
+/// allows the zeroed segment padding to be mapped into memory where zero fill
+/// sections might be. The gigabyte zero fill sections, those with the section
+/// type S_GB_ZEROFILL, can only be in a segment with sections of this type.
+/// These segments are then placed after all other segments.
+///
+/// The MH_OBJECT format has all of its sections in one segment for
+/// compactness.  There is no padding to a specified segment boundary and the
+/// mach_header and load commands are not part of the segment.
+///
+/// Sections with the same section name, sectname, going into the same segment,
+/// segname, are combined by the link editor.  The resulting section is aligned
+/// to the maximum alignment of the combined sections and is the new section's
+/// alignment.  The combined sections are aligned to their original alignment in
+/// the combined section.  Any padded bytes to get the specified alignment are
+/// zeroed.
+///
+/// The format of the relocation entries referenced by the reloff and nreloc
+/// fields of the section structure for mach object files is described in the
+/// header file <reloc.h>.
+pub const @"section" = extern struct {
+    sectname: [16]u8, /// name of this section
+    segname: [16]u8,  /// segment this section goes in
+    addr: u32,        /// memory address of this section
+    size: u32,        /// size in bytes of this section
+    offset: u32,      /// file offset of this section
+    @"align": u32,    /// section alignment (power of 2)
+    reloff: u32,      /// file offset of relocation entries
+    nreloc: u32,      /// number of relocation entries
+    flags: u32,       /// flags (section type and attributes
+    reserved1: u32,   /// reserved (for offset or index)
+    reserved2: u32,   /// reserved (for count or sizeof)
+};
+
+pub const section_64 = extern struct {
+    sectname: [16]u8, /// name of this section
+    segname: [16]u8,  /// segment this section goes in
+    addr: u64,        /// memory address of this section
+    size: u64,        /// size in bytes of this section
+    offset: u32,      /// file offset of this section
+    @"align": u32,    /// section alignment (power of 2)
+    reloff: u32,      /// file offset of relocation entries
+    nreloc: u32,      /// number of relocation entries
+    flags: u32,       /// flags (section type and attributes
+    reserved1: u32,   /// reserved (for offset or index)
+    reserved2: u32,   /// reserved (for count or sizeof)
+    reserved3: u32,   /// reserved
+};
+
+pub const nlist = extern struct {
+    n_strx: u32,
+    n_type: u8,
+    n_sect: u8,
+    n_desc: i16,
+    n_value: u32,
+};
+
+pub const nlist_64 = extern struct {
+    n_strx: u32,
+    n_type: u8,
+    n_sect: u8,
+    n_desc: u16,
+    n_value: u64,
+};
+
+/// After MacOS X 10.1 when a new load command is added that is required to be
+/// understood by the dynamic linker for the image to execute properly the
+/// LC_REQ_DYLD bit will be or'ed into the load command constant.  If the dynamic
+/// linker sees such a load command it it does not understand will issue a
+/// "unknown load command required for execution" error and refuse to use the
+/// image.  Other load commands without this bit that are not understood will
+/// simply be ignored.
+pub const LC_REQ_DYLD = 0x80000000;
+
+pub const LC_SEGMENT = 0x1; /// segment of this file to be mapped
+pub const LC_SYMTAB = 0x2; /// link-edit stab symbol table info
+pub const LC_SYMSEG = 0x3; /// link-edit gdb symbol table info (obsolete)
+pub const LC_THREAD = 0x4; /// thread
+pub const LC_UNIXTHREAD = 0x5; /// unix thread (includes a stack)
+pub const LC_LOADFVMLIB = 0x6; /// load a specified fixed VM shared library
+pub const LC_IDFVMLIB = 0x7; /// fixed VM shared library identification
+pub const LC_IDENT = 0x8; /// object identification info (obsolete)
+pub const LC_FVMFILE = 0x9; /// fixed VM file inclusion (internal use)
+pub const LC_PREPAGE      = 0xa; /// prepage command (internal use)
+pub const LC_DYSYMTAB = 0xb; /// dynamic link-edit symbol table info
+pub const LC_LOAD_DYLIB = 0xc; /// load a dynamically linked shared library
+pub const LC_ID_DYLIB = 0xd; /// dynamically linked shared lib ident
+pub const LC_LOAD_DYLINKER = 0xe; /// load a dynamic linker
+pub const LC_ID_DYLINKER = 0xf; /// dynamic linker identification
+pub const LC_PREBOUND_DYLIB = 0x10; /// modules prebound for a dynamically
+pub const LC_ROUTINES = 0x11; /// image routines
+pub const LC_SUB_FRAMEWORK = 0x12; /// sub framework
+pub const LC_SUB_UMBRELLA = 0x13; /// sub umbrella
+pub const LC_SUB_CLIENT = 0x14; /// sub client
+pub const LC_SUB_LIBRARY  = 0x15; /// sub library
+pub const LC_TWOLEVEL_HINTS = 0x16; /// two-level namespace lookup hints
+pub const LC_PREBIND_CKSUM  = 0x17; /// prebind checksum
+
+/// load a dynamically linked shared library that is allowed to be missing
+/// (all symbols are weak imported).
+pub const LC_LOAD_WEAK_DYLIB = (0x18 | LC_REQ_DYLD);
+
+pub const LC_SEGMENT_64 = 0x19; /// 64-bit segment of this file to be mapped
+pub const LC_ROUTINES_64 = 0x1a; /// 64-bit image routines
+pub const LC_UUID =  0x1b; /// the uuid
+pub const LC_RPATH = (0x1c | LC_REQ_DYLD); /// runpath additions
+pub const LC_CODE_SIGNATURE = 0x1d; /// local of code signature
+pub const LC_SEGMENT_SPLIT_INFO = 0x1e; /// local of info to split segments
+pub const LC_REEXPORT_DYLIB = (0x1f | LC_REQ_DYLD); /// load and re-export dylib
+pub const LC_LAZY_LOAD_DYLIB = 0x20; /// delay load of dylib until first use
+pub const LC_ENCRYPTION_INFO = 0x21; /// encrypted segment information
+pub const LC_DYLD_INFO =  0x22; /// compressed dyld information
+pub const LC_DYLD_INFO_ONLY = (0x22|LC_REQ_DYLD); /// compressed dyld information only
+pub const LC_LOAD_UPWARD_DYLIB = (0x23 | LC_REQ_DYLD); /// load upward dylib
+pub const LC_VERSION_MIN_MACOSX = 0x24; /// build for MacOSX min OS version
+pub const LC_VERSION_MIN_IPHONEOS = 0x25; /// build for iPhoneOS min OS version
+pub const LC_FUNCTION_STARTS = 0x26; /// compressed table of function start addresses
+pub const LC_DYLD_ENVIRONMENT = 0x27; /// string for dyld to treat like environment variable
+pub const LC_MAIN = (0x28|LC_REQ_DYLD); /// replacement for LC_UNIXTHREAD
+pub const LC_DATA_IN_CODE = 0x29; /// table of non-instructions in __text
+pub const LC_SOURCE_VERSION = 0x2A; /// source version used to build binary
+pub const LC_DYLIB_CODE_SIGN_DRS = 0x2B; /// Code signing DRs copied from linked dylibs
+pub const LC_ENCRYPTION_INFO_64 = 0x2C; /// 64-bit encrypted segment information
+pub const LC_LINKER_OPTION = 0x2D; /// linker options in MH_OBJECT files
+pub const LC_LINKER_OPTIMIZATION_HINT = 0x2E; /// optimization hints in MH_OBJECT files
+pub const LC_VERSION_MIN_TVOS = 0x2F; /// build for AppleTV min OS version
+pub const LC_VERSION_MIN_WATCHOS = 0x30; /// build for Watch min OS version
+pub const LC_NOTE = 0x31; /// arbitrary data included within a Mach-O file
+pub const LC_BUILD_VERSION = 0x32; /// build for platform min OS version
+
+pub const MH_MAGIC = 0xfeedface; /// the mach magic number
+pub const MH_CIGAM = 0xcefaedfe; /// NXSwapInt(MH_MAGIC)
+
+pub const MH_MAGIC_64 = 0xfeedfacf; /// the 64-bit mach magic number
+pub const MH_CIGAM_64 = 0xcffaedfe; /// NXSwapInt(MH_MAGIC_64)
+
+pub const MH_OBJECT = 0x1;  /// relocatable object file
+pub const MH_EXECUTE = 0x2;  /// demand paged executable file
+pub const MH_FVMLIB = 0x3;  /// fixed VM shared library file
+pub const MH_CORE =  0x4;  /// core file
+pub const MH_PRELOAD = 0x5;  /// preloaded executable file
+pub const MH_DYLIB = 0x6;  /// dynamically bound shared library
+pub const MH_DYLINKER = 0x7;  /// dynamic link editor
+pub const MH_BUNDLE = 0x8;  /// dynamically bound bundle file
+pub const MH_DYLIB_STUB = 0x9;  /// shared library stub for static linking only, no section contents
+pub const MH_DSYM =  0xa;  /// companion file with only debug sections
+pub const MH_KEXT_BUNDLE = 0xb;  /// x86_64 kexts
+
+// Constants for the flags field of the mach_header
+
+pub const MH_NOUNDEFS = 0x1;  /// the object file has no undefined references
+pub const MH_INCRLINK = 0x2;  /// the object file is the output of an incremental link against a base file and can't be link edited again
+pub const MH_DYLDLINK = 0x4;  /// the object file is input for the dynamic linker and can't be staticly link edited again
+pub const MH_BINDATLOAD = 0x8;  /// the object file's undefined references are bound by the dynamic linker when loaded.
+pub const MH_PREBOUND = 0x10;  /// the file has its dynamic undefined references prebound.
+pub const MH_SPLIT_SEGS = 0x20;  /// the file has its read-only and read-write segments split
+pub const MH_LAZY_INIT = 0x40;  /// the shared library init routine is to be run lazily via catching memory faults to its writeable segments (obsolete)
+pub const MH_TWOLEVEL = 0x80;  /// the image is using two-level name space bindings
+pub const MH_FORCE_FLAT = 0x100;  /// the executable is forcing all images to use flat name space bindings
+pub const MH_NOMULTIDEFS = 0x200;  /// this umbrella guarantees no multiple defintions of symbols in its sub-images so the two-level namespace hints can always be used.
+pub const MH_NOFIXPREBINDING = 0x400; /// do not have dyld notify the prebinding agent about this executable
+pub const MH_PREBINDABLE =  0x800;           /// the binary is not prebound but can have its prebinding redone. only used when MH_PREBOUND is not set.
+pub const MH_ALLMODSBOUND = 0x1000;  /// indicates that this binary binds to all two-level namespace modules of its dependent libraries. only used when MH_PREBINDABLE and MH_TWOLEVEL are both set. 
+pub const MH_SUBSECTIONS_VIA_SYMBOLS = 0x2000;/// safe to divide up the sections into sub-sections via symbols for dead code stripping
+pub const MH_CANONICAL =    0x4000;  /// the binary has been canonicalized via the unprebind operation
+pub const MH_WEAK_DEFINES = 0x8000;  /// the final linked image contains external weak symbols
+pub const MH_BINDS_TO_WEAK = 0x10000; /// the final linked image uses weak symbols
+
+pub const MH_ALLOW_STACK_EXECUTION = 0x20000;/// When this bit is set, all stacks in the task will be given stack execution privilege.  Only used in MH_EXECUTE filetypes.
+pub const MH_ROOT_SAFE = 0x40000;           /// When this bit is set, the binary declares it is safe for use in processes with uid zero
+                                         
+pub const MH_SETUID_SAFE = 0x80000;         /// When this bit is set, the binary declares it is safe for use in processes when issetugid() is true
+
+pub const MH_NO_REEXPORTED_DYLIBS = 0x100000; /// When this bit is set on a dylib, the static linker does not need to examine dependent dylibs to see if any are re-exported
+pub const MH_PIE = 0x200000;   /// When this bit is set, the OS will load the main executable at a random address.  Only used in MH_EXECUTE filetypes.
+pub const MH_DEAD_STRIPPABLE_DYLIB = 0x400000; /// Only for use on dylibs.  When linking against a dylib that has this bit set, the static linker will automatically not create a LC_LOAD_DYLIB load command to the dylib if no symbols are being referenced from the dylib.
+pub const MH_HAS_TLV_DESCRIPTORS = 0x800000; /// Contains a section of type S_THREAD_LOCAL_VARIABLES
+
+pub const MH_NO_HEAP_EXECUTION = 0x1000000; /// When this bit is set, the OS will run the main executable with a non-executable heap even on platforms (e.g. i386) that don't require it. Only used in MH_EXECUTE filetypes.
+
+pub const MH_APP_EXTENSION_SAFE = 0x02000000; /// The code was linked for use in an application extension.
+
+pub const MH_NLIST_OUTOFSYNC_WITH_DYLDINFO = 0x04000000; /// The external symbols listed in the nlist symbol table do not include all the symbols listed in the dyld info.
+
+
+/// The flags field of a section structure is separated into two parts a section
+/// type and section attributes.  The section types are mutually exclusive (it
+/// can only have one type) but the section attributes are not (it may have more
+/// than one attribute).
+/// 256 section types
+pub const SECTION_TYPE = 0x000000ff;
+pub const SECTION_ATTRIBUTES = 0xffffff00; ///  24 section attributes
+
+pub const S_REGULAR = 0x0; /// regular section
+pub const S_ZEROFILL = 0x1; /// zero fill on demand section
+pub const S_CSTRING_LITERALS = 0x2; /// section with only literal C string
+pub const S_4BYTE_LITERALS = 0x3; /// section with only 4 byte literals
+pub const S_8BYTE_LITERALS = 0x4; /// section with only 8 byte literals
+pub const S_LITERAL_POINTERS = 0x5; /// section with only pointers to
+
+
+pub const N_STAB = 0xe0; /// if any of these bits set, a symbolic debugging entry
+pub const N_PEXT = 0x10; /// private external symbol bit
+pub const N_TYPE = 0x0e; /// mask for the type bits
+pub const N_EXT = 0x01; /// external symbol bit, set for external symbols
+
+
+pub const N_GSYM = 0x20; /// global symbol: name,,NO_SECT,type,0
+pub const N_FNAME = 0x22; /// procedure name (f77 kludge): name,,NO_SECT,0,0
+pub const N_FUN = 0x24; /// procedure: name,,n_sect,linenumber,address
+pub const N_STSYM = 0x26; /// static symbol: name,,n_sect,type,address
+pub const N_LCSYM = 0x28; /// .lcomm symbol: name,,n_sect,type,address
+pub const N_BNSYM = 0x2e; /// begin nsect sym: 0,,n_sect,0,address
+pub const N_AST = 0x32; /// AST file path: name,,NO_SECT,0,0
+pub const N_OPT = 0x3c; /// emitted with gcc2_compiled and in gcc source
+pub const N_RSYM = 0x40; /// register sym: name,,NO_SECT,type,register
+pub const N_SLINE = 0x44; /// src line: 0,,n_sect,linenumber,address
+pub const N_ENSYM = 0x4e; /// end nsect sym: 0,,n_sect,0,address
+pub const N_SSYM = 0x60; /// structure elt: name,,NO_SECT,type,struct_offset
+pub const N_SO = 0x64; /// source file name: name,,n_sect,0,address
+pub const N_OSO = 0x66; /// object file name: name,,0,0,st_mtime
+pub const N_LSYM = 0x80; /// local sym: name,,NO_SECT,type,offset
+pub const N_BINCL = 0x82; /// include file beginning: name,,NO_SECT,0,sum
+pub const N_SOL = 0x84; /// #included file name: name,,n_sect,0,address
+pub const N_PARAMS =  0x86; /// compiler parameters: name,,NO_SECT,0,0
+pub const N_VERSION = 0x88; /// compiler version: name,,NO_SECT,0,0
+pub const N_OLEVEL =  0x8A; /// compiler -O level: name,,NO_SECT,0,0
+pub const N_PSYM = 0xa0; /// parameter: name,,NO_SECT,type,offset
+pub const N_EINCL = 0xa2; /// include file end: name,,NO_SECT,0,0
+pub const N_ENTRY = 0xa4; /// alternate entry: name,,n_sect,linenumber,address
+pub const N_LBRAC = 0xc0; /// left bracket: 0,,NO_SECT,nesting level,address
+pub const N_EXCL = 0xc2; /// deleted include file: name,,NO_SECT,0,sum
+pub const N_RBRAC = 0xe0; /// right bracket: 0,,NO_SECT,nesting level,address
+pub const N_BCOMM = 0xe2; /// begin common: name,,NO_SECT,0,0
+pub const N_ECOMM = 0xe4; /// end common: name,,n_sect,0,0
+pub const N_ECOML = 0xe8; /// end common (local name): 0,,n_sect,0,address
+pub const N_LENG = 0xfe; /// second stab entry with length information
+
+/// If a segment contains any sections marked with S_ATTR_DEBUG then all
+/// sections in that segment must have this attribute.  No section other than
+/// a section marked with this attribute may reference the contents of this
+/// section.  A section with this attribute may contain no symbols and must have
+/// a section type S_REGULAR.  The static linker will not copy section contents
+/// from sections with this attribute into its output file.  These sections
+/// generally contain DWARF debugging info.
+pub const S_ATTR_DEBUG = 0x02000000; /// a debug section
+
+pub const cpu_type_t = integer_t;
+pub const cpu_subtype_t = integer_t;
+pub const integer_t = c_int;
+pub const vm_prot_t = c_int;
+
 // sys/types.h on macos uses #pragma pack(4) so these checks are
 // to make sure the struct is laid out the same. These values were
 // produced from C code using the offsetof macro.
std/debug/index.zig
@@ -5,7 +5,6 @@ const io = std.io;
 const os = std.os;
 const elf = std.elf;
 const DW = std.dwarf;
-const macho = std.macho;
 const ArrayList = std.ArrayList;
 const builtin = @import("builtin");
 
@@ -19,9 +18,10 @@ pub const runtime_safety = switch (builtin.mode) {
 
 /// Tries to write to stderr, unbuffered, and ignores any error returned.
 /// Does not append a newline.
-/// TODO atomic/multithread support
 var stderr_file: os.File = undefined;
 var stderr_file_out_stream: io.FileOutStream = undefined;
+
+/// TODO multithreaded awareness
 var stderr_stream: ?*io.OutStream(io.FileOutStream.Error) = null;
 var stderr_mutex = std.Mutex.init();
 pub fn warn(comptime fmt: []const u8, args: ...) void {
@@ -30,6 +30,7 @@ pub fn warn(comptime fmt: []const u8, args: ...) void {
     const stderr = getStderrStream() catch return;
     stderr.print(fmt, args) catch return;
 }
+
 pub fn getStderrStream() !*io.OutStream(io.FileOutStream.Error) {
     if (stderr_stream) |st| {
         return st;
@@ -42,14 +43,15 @@ pub fn getStderrStream() !*io.OutStream(io.FileOutStream.Error) {
     }
 }
 
-var self_debug_info: ?*DebugInfo = null;
+/// TODO multithreaded awareness
+var self_debug_info: ?DebugInfo = null;
+
 pub fn getSelfDebugInfo() !*DebugInfo {
-    if (self_debug_info) |info| {
+    if (self_debug_info) |*info| {
         return info;
     } else {
-        const info = try openSelfDebugInfo(getDebugInfoAllocator());
-        self_debug_info = info;
-        return info;
+        self_debug_info = try openSelfDebugInfo(getDebugInfoAllocator());
+        return &self_debug_info.?;
     }
 }
 
@@ -127,6 +129,7 @@ pub fn panic(comptime format: []const u8, args: ...) noreturn {
     panicExtra(null, first_trace_addr, format, args);
 }
 
+/// TODO multithreaded awareness
 var panicking: u8 = 0; // TODO make this a bool
 
 pub fn panicExtra(trace: ?*const builtin.StackTrace, first_trace_addr: ?usize, comptime format: []const u8, args: ...) noreturn {
@@ -220,80 +223,147 @@ pub fn writeCurrentStackTrace(out_stream: var, allocator: *mem.Allocator, debug_
 
 pub fn printSourceAtAddress(debug_info: *DebugInfo, out_stream: var, address: usize, tty_color: bool) !void {
     switch (builtin.os) {
-        builtin.Os.windows => return error.UnsupportedDebugInfo,
-        builtin.Os.macosx => {
-            // TODO(bnoordhuis) It's theoretically possible to obtain the
-            // compilation unit from the symbtab but it's not that useful
-            // in practice because the compiler dumps everything in a single
-            // object file.  Future improvement: use external dSYM data when
-            // available.
-            const unknown = macho.Symbol{
-                .name = "???",
-                .address = address,
-            };
-            const symbol = debug_info.symbol_table.search(address) orelse &unknown;
-            try out_stream.print(WHITE ++ "{}" ++ RESET ++ ": " ++ DIM ++ "0x{x}" ++ " in ??? (???)" ++ RESET ++ "\n", symbol.name, address);
+        builtin.Os.macosx => return printSourceAtAddressMacOs(debug_info, out_stream, address, tty_color),
+        builtin.Os.linux => return printSourceAtAddressLinux(debug_info, out_stream, address, tty_color),
+        builtin.Os.windows => {
+            // TODO https://github.com/ziglang/zig/issues/721
+            return error.UnsupportedOperatingSystem;
         },
-        else => {
-            const compile_unit = findCompileUnit(debug_info, address) catch {
-                if (tty_color) {
-                    try out_stream.print("???:?:?: " ++ DIM ++ "0x{x} in ??? (???)" ++ RESET ++ "\n    ???\n\n", address);
-                } else {
-                    try out_stream.print("???:?:?: 0x{x} in ??? (???)\n    ???\n\n", address);
-                }
-                return;
-            };
-            const compile_unit_name = try compile_unit.die.getAttrString(debug_info, DW.AT_name);
-            if (getLineNumberInfo(debug_info, compile_unit, address - 1)) |line_info| {
-                defer line_info.deinit();
-                if (tty_color) {
-                    try out_stream.print(
-                        WHITE ++ "{}:{}:{}" ++ RESET ++ ": " ++ DIM ++ "0x{x} in ??? ({})" ++ RESET ++ "\n",
-                        line_info.file_name,
-                        line_info.line,
-                        line_info.column,
-                        address,
-                        compile_unit_name,
-                    );
-                    if (printLineFromFile(out_stream, line_info)) {
-                        if (line_info.column == 0) {
-                            try out_stream.write("\n");
-                        } else {
-                            {
-                                var col_i: usize = 1;
-                                while (col_i < line_info.column) : (col_i += 1) {
-                                    try out_stream.writeByte(' ');
-                                }
-                            }
-                            try out_stream.write(GREEN ++ "^" ++ RESET ++ "\n");
-                        }
-                    } else |err| switch (err) {
-                        error.EndOfFile => {},
-                        else => return err,
+        else => return error.UnsupportedOperatingSystem,
+    }
+}
+
+fn machoSearchSymbols(symbols: []const MachoSymbol, address: usize) ?*const MachoSymbol {
+    var min: usize = 0;
+    var max: usize = symbols.len - 1; // Exclude sentinel.
+    while (min < max) {
+        const mid = min + (max - min) / 2;
+        const curr = &symbols[mid];
+        const next = &symbols[mid + 1];
+        if (address >= next.address()) {
+            min = mid + 1;
+        } else if (address < curr.address()) {
+            max = mid;
+        } else {
+            return curr;
+        }
+    }
+    return null;
+}
+
+fn printSourceAtAddressMacOs(debug_info: *DebugInfo, out_stream: var, address: usize, tty_color: bool) !void {
+    const base_addr = @ptrToInt(&std.c._mh_execute_header);
+    const adjusted_addr = 0x100000000 + (address - base_addr);
+
+    const symbol = machoSearchSymbols(debug_info.symbols, adjusted_addr) orelse {
+        if (tty_color) {
+            try out_stream.print("???:?:?: " ++ DIM ++ "0x{x} in ??? (???)" ++ RESET ++ "\n\n\n", address);
+        } else {
+            try out_stream.print("???:?:?: 0x{x} in ??? (???)\n\n\n", address);
+        }
+        return;
+    };
+
+    const symbol_name = mem.toSliceConst(u8, debug_info.strings.ptr + symbol.nlist.n_strx);
+    if (getLineNumberInfoMacOs(debug_info, symbol.*, address)) |line_info| {
+        const compile_unit_name = "???";
+        try printLineInfo(debug_info, out_stream, line_info, address, symbol_name, compile_unit_name, tty_color);
+    } else |err| switch (err) {
+        error.MissingDebugInfo => {
+            if (tty_color) {
+                try out_stream.print("???:?:?: " ++ DIM ++ "0x{x} in {} (???)" ++ RESET ++ "\n\n\n", address, symbol_name);
+            } else {
+                try out_stream.print("???:?:?: 0x{x} in {} (???)\n\n\n", address, symbol_name);
+            }
+        },
+        else => return err,
+    }
+}
+
+pub fn printSourceAtAddressLinux(debug_info: *DebugInfo, out_stream: var, address: usize, tty_color: bool) !void {
+    const compile_unit = findCompileUnit(debug_info, address) catch {
+        if (tty_color) {
+            try out_stream.print("???:?:?: " ++ DIM ++ "0x{x} in ??? (???)" ++ RESET ++ "\n\n\n", address);
+        } else {
+            try out_stream.print("???:?:?: 0x{x} in ??? (???)\n\n\n", address);
+        }
+        return;
+    };
+    const compile_unit_name = try compile_unit.die.getAttrString(debug_info, DW.AT_name);
+    if (getLineNumberInfoLinux(debug_info, compile_unit, address - 1)) |line_info| {
+        defer line_info.deinit();
+        const symbol_name = "???";
+        try printLineInfo(debug_info, out_stream, line_info, address, symbol_name, compile_unit_name, tty_color);
+    } else |err| switch (err) {
+        error.MissingDebugInfo, error.InvalidDebugInfo => {
+            if (tty_color) {
+                try out_stream.print("???:?:?: " ++ DIM ++ "0x{x} in ??? ({})" ++ RESET ++ "\n\n\n", address, compile_unit_name);
+            } else {
+                try out_stream.print("???:?:?: 0x{x} in ??? ({})\n\n\n", address, compile_unit_name);
+            }
+        },
+        else => return err,
+    }
+}
+
+fn printLineInfo(
+    debug_info: *DebugInfo,
+    out_stream: var,
+    line_info: LineInfo,
+    address: usize,
+    symbol_name: []const u8,
+    compile_unit_name: []const u8,
+    tty_color: bool,
+) !void {
+    if (tty_color) {
+        try out_stream.print(
+            WHITE ++ "{}:{}:{}" ++ RESET ++ ": " ++ DIM ++ "0x{x} in {} ({})" ++ RESET ++ "\n",
+            line_info.file_name,
+            line_info.line,
+            line_info.column,
+            address,
+            symbol_name,
+            compile_unit_name,
+        );
+        if (printLineFromFile(out_stream, line_info)) {
+            if (line_info.column == 0) {
+                try out_stream.write("\n");
+            } else {
+                {
+                    var col_i: usize = 1;
+                    while (col_i < line_info.column) : (col_i += 1) {
+                        try out_stream.writeByte(' ');
                     }
-                } else {
-                    try out_stream.print(
-                        "{}:{}:{}: 0x{x} in ??? ({})\n",
-                        line_info.file_name,
-                        line_info.line,
-                        line_info.column,
-                        address,
-                        compile_unit_name,
-                    );
                 }
-            } else |err| switch (err) {
-                error.MissingDebugInfo, error.InvalidDebugInfo => {
-                    try out_stream.print("0x{x} in ??? ({})\n", address, compile_unit_name);
-                },
-                else => return err,
+                try out_stream.write(GREEN ++ "^" ++ RESET ++ "\n");
             }
-        },
+        } else |err| switch (err) {
+            error.EndOfFile => {},
+            else => return err,
+        }
+    } else {
+        try out_stream.print(
+            "{}:{}:{}: 0x{x} in {} ({})\n",
+            line_info.file_name,
+            line_info.line,
+            line_info.column,
+            address,
+            symbol_name,
+            compile_unit_name,
+        );
     }
 }
 
-pub fn openSelfDebugInfo(allocator: *mem.Allocator) !*DebugInfo {
-    switch (builtin.object_format) {
-        builtin.ObjectFormat.elf => {
+// TODO use this
+pub const OpenSelfDebugInfoError = error{
+    MissingDebugInfo,
+    OutOfMemory,
+    UnsupportedOperatingSystem,
+};
+
+pub fn openSelfDebugInfo(allocator: *mem.Allocator) !DebugInfo {
+    switch (builtin.os) {
+        builtin.Os.linux => {
             const st = try allocator.create(DebugInfo{
                 .self_exe_file = undefined,
                 .elf = undefined,
@@ -320,24 +390,79 @@ pub fn openSelfDebugInfo(allocator: *mem.Allocator) !*DebugInfo {
             try scanAllCompileUnits(st);
             return st;
         },
-        builtin.ObjectFormat.macho => {
-            var exe_file = try os.openSelfExe();
-            defer exe_file.close();
-
-            const st = try allocator.create(DebugInfo{ .symbol_table = try macho.loadSymbols(allocator, &io.FileInStream.init(&exe_file)) });
-            errdefer allocator.destroy(st);
-            return st;
-        },
-        builtin.ObjectFormat.coff => {
-            return error.TodoSupportCoffDebugInfo;
-        },
-        builtin.ObjectFormat.wasm => {
-            return error.TodoSupportCOFFDebugInfo;
-        },
-        builtin.ObjectFormat.unknown => {
-            return error.UnknownObjectFormat;
+        builtin.Os.macosx, builtin.Os.ios => return openSelfDebugInfoMacOs(allocator),
+        builtin.Os.windows => {
+            // TODO: https://github.com/ziglang/zig/issues/721
+            return error.UnsupportedOperatingSystem;
         },
+        else => return error.UnsupportedOperatingSystem,
+    }
+}
+
+fn openSelfDebugInfoMacOs(allocator: *mem.Allocator) !DebugInfo {
+    const hdr = &std.c._mh_execute_header;
+    assert(hdr.magic == std.c.MH_MAGIC_64);
+
+    const hdr_base = @ptrCast([*]u8, hdr);
+    var ptr = hdr_base + @sizeOf(std.c.mach_header_64);
+    var ncmd: u32 = hdr.ncmds;
+    const symtab = while (ncmd != 0) : (ncmd -= 1) {
+        const lc = @ptrCast(*std.c.load_command, ptr);
+        switch (lc.cmd) {
+            std.c.LC_SYMTAB => break @ptrCast(*std.c.symtab_command, ptr),
+            else => {},
+        }
+        ptr += lc.cmdsize; // TODO https://github.com/ziglang/zig/issues/1403
+    } else {
+        return error.MissingDebugInfo;
+    };
+    const syms = @ptrCast([*]std.c.nlist_64, hdr_base + symtab.symoff)[0..symtab.nsyms];
+    const strings = @ptrCast([*]u8, hdr_base + symtab.stroff)[0..symtab.strsize];
+
+    const symbols_buf = try allocator.alloc(MachoSymbol, syms.len);
+
+    var ofile: ?*std.c.nlist_64 = null;
+    var symbol_index: usize = 0;
+    var last_len: u64 = 0;
+    for (syms) |*sym| {
+        if (sym.n_type & std.c.N_STAB != 0) {
+            switch (sym.n_type) {
+                std.c.N_OSO => ofile = sym,
+                std.c.N_FUN => {
+                    if (sym.n_sect == 0) {
+                        last_len = sym.n_value;
+                    } else {
+                        symbols_buf[symbol_index] = MachoSymbol{
+                            .nlist = sym,
+                            .ofile = ofile,
+                        };
+                        symbol_index += 1;
+                    }
+                },
+                else => continue,
+            }
+        }
     }
+    const sentinel = try allocator.createOne(std.c.nlist_64);
+    sentinel.* = std.c.nlist_64{
+        .n_strx = 0,
+        .n_type = 36,
+        .n_sect = 0,
+        .n_desc = 0,
+        .n_value = symbols_buf[symbol_index - 1].nlist.n_value + last_len,
+    };
+
+    const symbols = allocator.shrink(MachoSymbol, symbols_buf, symbol_index);
+
+    // Even though lld emits symbols in ascending order, this debug code
+    // should work for programs linked in any valid way.
+    // This sort is so that we can binary search later.
+    std.sort.sort(MachoSymbol, symbols, MachoSymbol.addressLessThan);
+
+    return DebugInfo{
+        .symbols = symbols,
+        .strings = strings,
+    };
 }
 
 fn printLineFromFile(out_stream: var, line_info: *const LineInfo) !void {
@@ -372,13 +497,24 @@ fn printLineFromFile(out_stream: var, line_info: *const LineInfo) !void {
     }
 }
 
+const MachoSymbol = struct {
+    nlist: *std.c.nlist_64,
+    ofile: ?*std.c.nlist_64,
+
+    /// Returns the address from the macho file
+    fn address(self: MachoSymbol) u64 {
+        return self.nlist.n_value;
+    }
+
+    fn addressLessThan(lhs: MachoSymbol, rhs: MachoSymbol) bool {
+        return lhs.address() < rhs.address();
+    }
+};
+
 pub const DebugInfo = switch (builtin.os) {
     builtin.Os.macosx => struct {
-        symbol_table: macho.SymbolTable,
-
-        pub fn close(self: *DebugInfo) void {
-            self.symbol_table.deinit();
-        }
+        symbols: []const MachoSymbol,
+        strings: []const u8,
     },
     else => struct {
         self_exe_file: os.File,
@@ -803,12 +939,16 @@ fn parseDie(st: *DebugInfo, abbrev_table: *const AbbrevTable, is_64: bool) !Die
     return result;
 }
 
-fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_address: usize) !LineInfo {
-    const compile_unit_cwd = try compile_unit.die.getAttrString(st, DW.AT_comp_dir);
+fn getLineNumberInfoMacOs(di: *DebugInfo, symbol: MachoSymbol, target_address: usize) !LineInfo {
+    return error.MissingDebugInfo;
+}
 
-    const in_file = &st.self_exe_file;
-    const debug_line_end = st.debug_line.offset + st.debug_line.size;
-    var this_offset = st.debug_line.offset;
+fn getLineNumberInfoLinux(di: *DebugInfo, compile_unit: *const CompileUnit, target_address: usize) !LineInfo {
+    const compile_unit_cwd = try compile_unit.die.getAttrString(di, DW.AT_comp_dir);
+
+    const in_file = &di.self_exe_file;
+    const debug_line_end = di.debug_line.offset + di.debug_line.size;
+    var this_offset = di.debug_line.offset;
     var this_index: usize = 0;
 
     var in_file_stream = io.FileInStream.init(in_file);
@@ -827,11 +967,11 @@ fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_ad
             continue;
         }
 
-        const version = try in_stream.readInt(st.elf.endian, u16);
+        const version = try in_stream.readInt(di.elf.endian, u16);
         // TODO support 3 and 5
         if (version != 2 and version != 4) return error.InvalidDebugInfo;
 
-        const prologue_length = if (is_64) try in_stream.readInt(st.elf.endian, u64) else try in_stream.readInt(st.elf.endian, u32);
+        const prologue_length = if (is_64) try in_stream.readInt(di.elf.endian, u64) else try in_stream.readInt(di.elf.endian, u32);
         const prog_start_offset = (try in_file.getPos()) + prologue_length;
 
         const minimum_instruction_length = try in_stream.readByte();
@@ -850,7 +990,7 @@ fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_ad
 
         const opcode_base = try in_stream.readByte();
 
-        const standard_opcode_lengths = try st.allocator().alloc(u8, opcode_base - 1);
+        const standard_opcode_lengths = try di.allocator().alloc(u8, opcode_base - 1);
 
         {
             var i: usize = 0;
@@ -859,19 +999,19 @@ fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_ad
             }
         }
 
-        var include_directories = ArrayList([]u8).init(st.allocator());
+        var include_directories = ArrayList([]u8).init(di.allocator());
         try include_directories.append(compile_unit_cwd);
         while (true) {
-            const dir = try st.readString();
+            const dir = try di.readString();
             if (dir.len == 0) break;
             try include_directories.append(dir);
         }
 
-        var file_entries = ArrayList(FileEntry).init(st.allocator());
+        var file_entries = ArrayList(FileEntry).init(di.allocator());
         var prog = LineNumberProgram.init(default_is_stmt, include_directories.toSliceConst(), &file_entries, target_address);
 
         while (true) {
-            const file_name = try st.readString();
+            const file_name = try di.readString();
             if (file_name.len == 0) break;
             const dir_index = try readULeb128(in_stream);
             const mtime = try readULeb128(in_stream);
@@ -901,11 +1041,11 @@ fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_ad
                         return error.MissingDebugInfo;
                     },
                     DW.LNE_set_address => {
-                        const addr = try in_stream.readInt(st.elf.endian, usize);
+                        const addr = try in_stream.readInt(di.elf.endian, usize);
                         prog.address = addr;
                     },
                     DW.LNE_define_file => {
-                        const file_name = try st.readString();
+                        const file_name = try di.readString();
                         const dir_index = try readULeb128(in_stream);
                         const mtime = try readULeb128(in_stream);
                         const len_bytes = try readULeb128(in_stream);
@@ -963,7 +1103,7 @@ fn getLineNumberInfo(st: *DebugInfo, compile_unit: *const CompileUnit, target_ad
                         prog.address += inc_addr;
                     },
                     DW.LNS_fixed_advance_pc => {
-                        const arg = try in_stream.readInt(st.elf.endian, u16);
+                        const arg = try in_stream.readInt(di.elf.endian, u16);
                         prog.address += arg;
                     },
                     DW.LNS_set_prologue_end => {},
@@ -1142,7 +1282,7 @@ pub const global_allocator = &global_fixed_allocator.allocator;
 var global_fixed_allocator = std.heap.ThreadSafeFixedBufferAllocator.init(global_allocator_mem[0..]);
 var global_allocator_mem: [100 * 1024]u8 = undefined;
 
-// TODO make thread safe
+/// TODO multithreaded awareness
 var debug_info_allocator: ?*mem.Allocator = null;
 var debug_info_direct_allocator: std.heap.DirectAllocator = undefined;
 var debug_info_arena_allocator: std.heap.ArenaAllocator = undefined;
std/macho.zig
@@ -1,172 +0,0 @@
-const builtin = @import("builtin");
-const std = @import("index.zig");
-const io = std.io;
-const mem = std.mem;
-
-const MH_MAGIC_64 = 0xFEEDFACF;
-const MH_PIE = 0x200000;
-const LC_SYMTAB = 2;
-
-const MachHeader64 = packed struct {
-    magic: u32,
-    cputype: u32,
-    cpusubtype: u32,
-    filetype: u32,
-    ncmds: u32,
-    sizeofcmds: u32,
-    flags: u32,
-    reserved: u32,
-};
-
-const LoadCommand = packed struct {
-    cmd: u32,
-    cmdsize: u32,
-};
-
-const SymtabCommand = packed struct {
-    symoff: u32,
-    nsyms: u32,
-    stroff: u32,
-    strsize: u32,
-};
-
-const Nlist64 = packed struct {
-    n_strx: u32,
-    n_type: u8,
-    n_sect: u8,
-    n_desc: u16,
-    n_value: u64,
-};
-
-pub const Symbol = struct {
-    name: []const u8,
-    address: u64,
-
-    fn addressLessThan(lhs: Symbol, rhs: Symbol) bool {
-        return lhs.address < rhs.address;
-    }
-};
-
-pub const SymbolTable = struct {
-    allocator: *mem.Allocator,
-    symbols: []const Symbol,
-    strings: []const u8,
-
-    // Doubles as an eyecatcher to calculate the PIE slide, see loadSymbols().
-    // Ideally we'd use _mh_execute_header because it's always at 0x100000000
-    // in the image but as it's located in a different section than executable
-    // code, its displacement is different.
-    pub fn deinit(self: *SymbolTable) void {
-        self.allocator.free(self.symbols);
-        self.symbols = []const Symbol{};
-
-        self.allocator.free(self.strings);
-        self.strings = []const u8{};
-    }
-
-    pub fn search(self: *const SymbolTable, address: usize) ?*const Symbol {
-        var min: usize = 0;
-        var max: usize = self.symbols.len - 1; // Exclude sentinel.
-        while (min < max) {
-            const mid = min + (max - min) / 2;
-            const curr = &self.symbols[mid];
-            const next = &self.symbols[mid + 1];
-            if (address >= next.address) {
-                min = mid + 1;
-            } else if (address < curr.address) {
-                max = mid;
-            } else {
-                return curr;
-            }
-        }
-        return null;
-    }
-};
-
-pub fn loadSymbols(allocator: *mem.Allocator, in: *io.FileInStream) !SymbolTable {
-    var file = in.file;
-    try file.seekTo(0);
-
-    var hdr: MachHeader64 = undefined;
-    try readOneNoEof(in, MachHeader64, &hdr);
-    if (hdr.magic != MH_MAGIC_64) return error.MissingDebugInfo;
-    const is_pie = MH_PIE == (hdr.flags & MH_PIE);
-
-    var pos: usize = @sizeOf(@typeOf(hdr));
-    var ncmd: u32 = hdr.ncmds;
-    while (ncmd != 0) : (ncmd -= 1) {
-        try file.seekTo(pos);
-        var lc: LoadCommand = undefined;
-        try readOneNoEof(in, LoadCommand, &lc);
-        if (lc.cmd == LC_SYMTAB) break;
-        pos += lc.cmdsize;
-    } else {
-        return error.MissingDebugInfo;
-    }
-
-    var cmd: SymtabCommand = undefined;
-    try readOneNoEof(in, SymtabCommand, &cmd);
-
-    try file.seekTo(cmd.symoff);
-    var syms = try allocator.alloc(Nlist64, cmd.nsyms);
-    defer allocator.free(syms);
-    try readNoEof(in, Nlist64, syms);
-
-    try file.seekTo(cmd.stroff);
-    var strings = try allocator.alloc(u8, cmd.strsize);
-    errdefer allocator.free(strings);
-    try in.stream.readNoEof(strings);
-
-    var nsyms: usize = 0;
-    for (syms) |sym|
-        if (isSymbol(sym)) nsyms += 1;
-    if (nsyms == 0) return error.MissingDebugInfo;
-
-    var symbols = try allocator.alloc(Symbol, nsyms + 1); // Room for sentinel.
-    errdefer allocator.free(symbols);
-
-    var pie_slide: usize = 0;
-    var nsym: usize = 0;
-    for (syms) |sym| {
-        if (!isSymbol(sym)) continue;
-        const start = sym.n_strx;
-        const end = mem.indexOfScalarPos(u8, strings, start, 0).?;
-        const name = strings[start..end];
-        const address = sym.n_value;
-        symbols[nsym] = Symbol{ .name = name, .address = address };
-        nsym += 1;
-        if (is_pie and mem.eql(u8, name, "_SymbolTable_deinit")) {
-            pie_slide = @ptrToInt(SymbolTable.deinit) - address;
-        }
-    }
-
-    // Effectively a no-op, lld emits symbols in ascending order.
-    std.sort.sort(Symbol, symbols[0..nsyms], Symbol.addressLessThan);
-
-    // Insert the sentinel.  Since we don't know where the last function ends,
-    // we arbitrarily limit it to the start address + 4 KB.
-    const top = symbols[nsyms - 1].address + 4096;
-    symbols[nsyms] = Symbol{ .name = "", .address = top };
-
-    if (pie_slide != 0) {
-        for (symbols) |*symbol|
-            symbol.address += pie_slide;
-    }
-
-    return SymbolTable{
-        .allocator = allocator,
-        .symbols = symbols,
-        .strings = strings,
-    };
-}
-
-fn readNoEof(in: *io.FileInStream, comptime T: type, result: []T) !void {
-    return in.stream.readNoEof(@sliceToBytes(result));
-}
-fn readOneNoEof(in: *io.FileInStream, comptime T: type, result: *T) !void {
-    return readNoEof(in, T, (*[1]T)(result)[0..]);
-}
-
-fn isSymbol(sym: *const Nlist64) bool {
-    return sym.n_value != 0 and sym.n_desc == 0;
-}
CMakeLists.txt
@@ -485,7 +485,6 @@ set(ZIG_STD_FILES
     "json.zig"
     "lazy_init.zig"
     "linked_list.zig"
-    "macho.zig"
     "math/acos.zig"
     "math/acosh.zig"
     "math/asin.zig"