One more nit since I noticed: + if (is64) { + ((MachO::section_64 *)data)->addr = Addr; + } else { + ((MachO::section *)data)->addr = Addr; + } No need for the braces here. I'm kindof dubious about this whole approach, but this matches what the ELF stuff is doing so... -eric http://reviews.llvm.org/D4005