[PATCH] C API: Add LLVMCloneModule()

Reid Kleckner rnk at google.com
Fri Sep 26 13:14:34 PDT 2014


lgtm

On Fri, Sep 26, 2014 at 1:12 PM, Tom Stellard <tom at stellard.net> wrote:

> Hi Reid,
>
> Here is an updated patch with LLVMCloneModule() moved into the Transform
> library.  I wasn't sure where the best place for the declaration was,
> so I left it in Core.h.  I guess another option would be to add a new
> header called: include/llvm-c/Transforms/Util.h and put it there.
>
> -Tom
>
>
> On Fri, Sep 26, 2014 at 12:44:38PM -0700, Reid Kleckner wrote:
> > Seems like a reasonable idea, but this patch won't work because it makes
> > the IR library depend on the Transform library.
> >
> > On Fri, Sep 26, 2014 at 11:49 AM, Tom Stellard <tom at stellard.net> wrote:
> >
> > > Hi,
> > >
> > > This patch adds the LLVMCloneModule function to the C API.
> > >
> > > Please Review.
> > >
> > > Thanks,
> > > Tom
> > >
> > > _______________________________________________
> > > llvm-commits mailing list
> > > llvm-commits at cs.uiuc.edu
> > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> > >
> > >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140926/86fe6c76/attachment.html>


More information about the llvm-commits mailing list