[llvm] r189130 - CMake: don't install tablegen

Joerg Sonnenberger joerg at britannica.bec.de
Sat Aug 24 07:29:45 PDT 2013


On Fri, Aug 23, 2013 at 10:15:41PM -0700, Chandler Carruth wrote:
> On Fri, Aug 23, 2013 at 10:11 PM, Joerg Sonnenberger <
> joerg at britannica.bec.de> wrote:
> 
> > On Fri, Aug 23, 2013 at 06:28:10PM -0000, Hans Wennborg wrote:
> > > Author: hans
> > > Date: Fri Aug 23 13:28:10 2013
> > > New Revision: 189130
> > >
> > > URL: http://llvm.org/viewvc/llvm-project?rev=189130&view=rev
> > > Log:
> > > CMake: don't install tablegen
> > >
> > > Since it's an llvm-internal tool, we shouldn't install it.
> >
> > Is it really llvm-internal? The support for libOption makes me think
> > it is not.
> >
> 
> I don't think we can or should realistically try to support external users
> of libOption... It's really a utility library for building drivers like
> Clang and LLD, not something that we can reasonably support fully generic
> use of it. Similarly for the tablegen libraries and a few others.

Sure, but the thing I am thinking about (mclinker) effectively falls
into the same category.

The point I am trying to make is that lib/Option is a very valid use
case of tablegen outside llvm itself. That's unlike most other functions
of it.

Joerg



More information about the llvm-commits mailing list