Improve comments a bit
authorChris Lattner <sabre@nondot.org>
Fri, 23 Jul 2004 08:24:23 +0000 (08:24 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 23 Jul 2004 08:24:23 +0000 (08:24 +0000)
commitec2bc645053e9051ada01fac6a555df17a85c91d
tree03fd93fa7edc2b0a29a91c261d1e6e4f82a504ec
parent5c2e282865821ba65795a2d14bd95e5a82a6be41
Improve comments a bit

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);

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@15130 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/CodeGen/LiveIntervalAnalysis.h
lib/CodeGen/LiveIntervalAnalysis.cpp
lib/CodeGen/LiveIntervalAnalysis.h