<div class="gmail_quote">On 15 March 2010 22:32, Bill Wendling <span dir="ltr"><<a href="mailto:wendling@apple.com">wendling@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div><div></div><div class="h5">On Mar 15, 2010, at 10:06 AM, Aaron Gray wrote:<br>
<br>
> Hi,<br>
><br>
> This patch extracts 'class MachObjectWriter' into its own .h and .cpp files.<br>
><br>
> Tested on MSVC and Cygwin.<br>
><br>
</div></div>Hi Aaron,<br>
<br>
I'll let Chris & Daniel comment on the rest, but:<br>
<br>
Index: lib/CodeGen/TargetLoweringObjectFileImpl.cpp<br>
===================================================================<br>
--- lib/CodeGen/TargetLoweringObjectFileImpl.cpp        (revision 98543)<br>
+++ lib/CodeGen/TargetLoweringObjectFileImpl.cpp        (working copy)<br>
@@ -469,6 +469,7 @@<br>
   // Otherwise, return a new section.<br>
   return Entry = MCSectionMachO::Create(Segment, Section, TypeAndAttributes,<br>
                                         Reserved2, Kind, getContext());<br>
+  assert( Entry != 0 && "MCSectionMachO::Create returns zero");<br>
 }<br>
<br>
The assert is dead code.</blockquote><div><br></div><div>Yes, that is not very good, I thought I had got rid of that :(</div><div><br></div><div>Aaron</div><div><br></div></div>