[LLVMbugs] [PATCH] fix llvm::Interval name clash

[Chris Lattner]

On Mon, 21 Jun 2004, Vladimir Prus wrote:

>
> It turns out that there are two classes named llvm::Interval. The first is in
> llvm/Analysis/Interval.h and the second is in lib/CodeGen/LiveIntervals.h.

Oops.

> I'm not sure why it works with make-based build system, but I've just build
> llc with Boost.Build, and this name clash result in run-time failure -- since
> for shared libraries the linker will pick one of the symbols in case of name
> clash, and this time it's the wrong symbol.

Probably just luck.  Also the Interval class is not used much.

> That attached patch renamed the latter Interval to LiveInterval. The other
> possibility was to use anonymous namespace, but I prefered to make the
> simplest possible change.

Sounds great.  Applied!
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20040621/015347.html

-Chris

-- 
http://llvm.cs.uiuc.edu/
http://www.nondot.org/~sabre/Projects/