Skip to content
  • Chris Lattner's avatar
    Improve comments a bit · 53280cd2
    Chris Lattner authored
    Use an explicit LiveRange class to represent ranges instead of an std::pair.
    This is a minor cleanup, but is really intended to make a future patch simpler
    and less invasive.
    
    Alkis, could you please take a look at LiveInterval::liveAt?  I suspect that
    you can add an operator<(unsigned) to LiveRange, allowing us to speed up the
    upper_bound call by quite a bit (this would also apply to other callers of
    upper/lower_bound).  I would do it myself, but I still don't understand that
    crazy liveAt function, despite the comment. :)
    
    Basically I would like to see this:
        LiveRange dummy(index, index+1);
        Ranges::const_iterator r = std::upper_bound(ranges.begin(),
                                                    ranges.end(),
                                                    dummy);
    
    Turn into:
        Ranges::const_iterator r = std::upper_bound(ranges.begin(),
                                                    ranges.end(),
                                                    index);
    
    llvm-svn: 15130
    53280cd2
Loading