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

Chris Lattner sabre at nondot.org
Mon Jun 21 06:30:05 PDT 2004

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.


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



More information about the llvm-bugs mailing list