[LLVMdev] [PATCH] llvm-config: add svn revision to --version

David Greene dag at cray.com
Mon May 18 14:17:10 PDT 2009


On Monday 18 May 2009 13:35, Frits van Bommel wrote:
> Török Edwin wrote:
> > I think you don't need to support all trunk revisions.
> > People using trunk could be told to either update to a certain revision,
> > or stick to a released version.
>
> We don't support /all/ trunk revisions, but we don't like to force upgrades
> unless it's really needed so we try to at least support most recent
> revisions.
>
> One of the reasons is that not all of our developers use the same trunk
> revision, and recompiling LLVM trunk is not something everyone likes to do
> once a day. (It's nice if you can put it off for a couple of days without
> things breaking)

There's an easy way to fix these kinds of problems.  Deprecate interfaces for
one release.  I brought this up before but the idea was quickly shot down.
It's insane to litter your code with #ifdefs based on SVN revisions.

This is exactly the kind of situation that the LLVM community needs to 
address.  Changing APIs is fine.  Ripping out old versions without first
deprecating them for a release causes nightmares all around.

                           -Dave




More information about the llvm-dev mailing list