<div dir="ltr">lgtm</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 26, 2014 at 1:12 PM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Reid,<br>
<br>
Here is an updated patch with LLVMCloneModule() moved into the Transform<br>
library.  I wasn't sure where the best place for the declaration was,<br>
so I left it in Core.h.  I guess another option would be to add a new<br>
header called: include/llvm-c/Transforms/Util.h and put it there.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Tom<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Sep 26, 2014 at 12:44:38PM -0700, Reid Kleckner wrote:<br>
> Seems like a reasonable idea, but this patch won't work because it makes<br>
> the IR library depend on the Transform library.<br>
><br>
> On Fri, Sep 26, 2014 at 11:49 AM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br>
><br>
> > Hi,<br>
> ><br>
> > This patch adds the LLVMCloneModule function to the C API.<br>
> ><br>
> > Please Review.<br>
> ><br>
> > Thanks,<br>
> > Tom<br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> ><br>
> ><br>
</div></div></blockquote></div><br></div>