[LLVMbugs] [PATCH] fix llvm::Interval name clash
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