[PATCH] C API: Add LLVMCloneModule()

Tom Stellard tom at stellard.net
Wed Oct 1 10:25:35 PDT 2014


On Fri, Sep 26, 2014 at 01:14:34PM -0700, Reid Kleckner wrote:
> lgtm
> 

r218775

> 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
> > > >
> > > >
> >



More information about the llvm-commits mailing list