[llvm-commits] [llvm] r117174 - in /llvm/trunk: include/llvm/CodeGen/LinkAllCodegenComponents.h include/llvm/CodeGen/Passes.h lib/CodeGen/CMakeLists.txt lib/CodeGen/LiveIntervalUnion.cpp lib/CodeGen/LiveIntervalUnion.h lib/CodeGen/RegAllocBase.h lib/CodeGen/RegAllocBasic.cpp lib/CodeGen/SplitKit.h

[Jakob Stoklund Olesen]

On Oct 22, 2010, at 4:09 PM, Andrew Trick wrote:

> +void LiveIntervalUnion::unify(LiveInterval &lvr) {
> +  // Add this live virtual register to the union
> +  LiveVirtRegs::iterator pos = std::upper_bound(lvrs_.begin(), lvrs_.end(),
> +                                                &lvr, less_ptr<LiveInterval>());
> +  assert(pos == lvrs_.end() || *pos != &lvr && "duplicate LVR insertion");
> +  lvrs_.insert(pos, &lvr);

Is it just me, or is this the only place lvrs_ is used?

Can it go away?