[PATCH] [mach-o] wire up a dylib's install-name in more places

Tim Northover t.p.northover at gmail.com
Mon Jun 30 07:44:04 PDT 2014


Hi kledzik,

Hi,

This adds support for installName to the various transformations an object file may go through. I *think* LC_ID_DYLIB commands will be interepreted and written correctly after this.

I know we were trying to keep tests contained in a single file and avoid using "Inputs/", but I can't think of a way to combine the two yaml files into a single one and get lld to act on them correctly.

Does it look OK?

Cheers.

Tim.

http://reviews.llvm.org/D4348

Files:
  lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp
  lib/ReaderWriter/MachO/MachONormalizedFileBinaryWriter.cpp
  lib/ReaderWriter/MachO/MachONormalizedFileFromAtoms.cpp
  lib/ReaderWriter/MachO/MachONormalizedFileToAtoms.cpp
  lib/ReaderWriter/MachO/MachONormalizedFileYAML.cpp
  test/mach-o/Inputs/use-dylib-install-names.yaml
  test/mach-o/dylib-install-names.yaml
  test/mach-o/lit.local.cfg
  test/mach-o/use-simple-dylib.yaml
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4348.10975.patch
Type: text/x-patch
Size: 8829 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140630/a064c79b/attachment.bin>


More information about the llvm-commits mailing list