[PATCH] [lld] [MachO] Begin to flesh out normalizedToAtoms

Joey Gouly joey.gouly at gmail.com
Thu Jan 2 16:41:36 PST 2014



================
Comment at: lib/ReaderWriter/MachO/MachONormalizedFileYAML.cpp:282
@@ -280,1 +281,3 @@
+        io, sect.content);
+    io.mapOptional("content",         content->normalizedContent);
     io.mapOptional("relocations",     sect.relocations);
----------------
kledzik at apple.com wrote:
> I think you can just cast the address of sect.content to be of type ContentBytes and it should all just work with no need for NormalizedContent.
mapOptional wants a reference to the content, so I cant use the address of content, and as far as I can tell, I can't change the type of a reference.

Unless I'm mistaken, I can't do this without copying.


http://llvm-reviews.chandlerc.com/D2464



More information about the llvm-commits mailing list