[PATCH] D15943: Add support for headerpad_max_install_names cmdline option

kledzik@apple.com via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 7 11:16:16 PST 2016


kledzik added inline comments.

================
Comment at: lib/ReaderWriter/MachO/MachONormalizedFile.h:289-290
@@ -288,3 +288,4 @@
 /// Takes in-memory normalized view and writes a mach-o object file.
-std::error_code writeBinary(const NormalizedFile &file, StringRef path);
+std::error_code writeBinary(const NormalizedFile &file, StringRef path,
+                            const MachOLinkingContext &ctx);
 
----------------
This is a design change.  writeBinary() has only depended on the NormalizedFile object - not the LinkingContext.  That is, for extra info like this, we add a new field to NormalizedFile.



http://reviews.llvm.org/D15943





More information about the llvm-commits mailing list