[llvm] r186969 - Revert "Remove use of asymmetric std::lower_bound comparator."

[Ahmed Bougacha]

Author: ab
Date: Tue Jul 23 12:44:11 2013
New Revision: 186969

URL: http://llvm.org/viewvc/llvm-project?rev=186969&view=rev
Log:
Revert "Remove use of asymmetric std::lower_bound comparator."

This reverts commit r185676.
Originally done because of VS 2008.

Modified:
    llvm/trunk/include/llvm/MC/MCModule.h
    llvm/trunk/lib/MC/MCModule.cpp

Modified: llvm/trunk/include/llvm/MC/MCModule.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCModule.h?rev=186969&r1=186968&r2=186969&view=diff
==============================================================================
--- llvm/trunk/include/llvm/MC/MCModule.h (original)
+++ llvm/trunk/include/llvm/MC/MCModule.h Tue Jul 23 12:44:11 2013
@@ -51,11 +51,6 @@ class MCModule {
 
   /// \brief Insert an atom in the module, using its Begin and End addresses.
   void map(MCAtom *NewAtom);
-
-  /// \brief Return an iterator to the first atom that is completely or in part
-  /// located after \p Addr, i.e., that \p Addr is bigger than its end address.
-  AtomListTy::const_iterator atom_lower_bound(uint64_t Addr) const;
-  AtomListTy::      iterator atom_lower_bound(uint64_t Addr);
   /// @}
 
   /// \name Function tracking

Modified: llvm/trunk/lib/MC/MCModule.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCModule.cpp?rev=186969&r1=186968&r2=186969&view=diff
==============================================================================
--- llvm/trunk/lib/MC/MCModule.cpp (original)
+++ llvm/trunk/lib/MC/MCModule.cpp Tue Jul 23 12:44:11 2013
@@ -14,19 +14,8 @@
 
 using namespace llvm;
 
-static bool AtomComp(const MCAtom *LHS, const MCAtom *RHS) {
-  return LHS->getEndAddr() < RHS->getEndAddr();
-}
-
-MCModule::const_atom_iterator MCModule::atom_lower_bound(uint64_t Addr) const {
-  // This is a dummy atom, because VS 2008 doesn't like asymmetric comparators.
-  MCDataAtom AddrAtom(0, Addr, Addr);
-  return std::lower_bound(atom_begin(), atom_end(), &AddrAtom, AtomComp);
-}
-
-MCModule::atom_iterator MCModule::atom_lower_bound(uint64_t Addr) {
-  MCDataAtom AddrAtom(0, Addr, Addr);
-  return std::lower_bound(atom_begin(), atom_end(), &AddrAtom, AtomComp);
+static bool AtomComp(const MCAtom *L, uint64_t Addr) {
+  return L->getEndAddr() < Addr;
 }
 
 void MCModule::map(MCAtom *NewAtom) {
@@ -35,7 +24,8 @@ void MCModule::map(MCAtom *NewAtom) {
   assert(Begin <= NewAtom->End && "Creating MCAtom with endpoints reversed?");
 
   // Check for atoms already covering this range.
-  AtomListTy::iterator I = atom_lower_bound(Begin);
+  AtomListTy::iterator I = std::lower_bound(atom_begin(), atom_end(),
+                                            Begin, AtomComp);
   assert((I == atom_end() || (*I)->getBeginAddr() > NewAtom->End)
          && "Offset range already occupied!");
 
@@ -58,13 +48,15 @@ MCDataAtom *MCModule::createDataAtom(uin
 // remap - Update the interval mapping for an atom.
 void MCModule::remap(MCAtom *Atom, uint64_t NewBegin, uint64_t NewEnd) {
   // Find and erase the old mapping.
-  AtomListTy::iterator I = atom_lower_bound(Atom->Begin);
+  AtomListTy::iterator I = std::lower_bound(atom_begin(), atom_end(),
+                                            Atom->Begin, AtomComp);
   assert(I != atom_end() && "Atom offset not found in module!");
   assert(*I == Atom && "Previous atom mapping was invalid!");
   Atoms.erase(I);
 
   // Insert the new mapping.
-  AtomListTy::iterator NewI = atom_lower_bound(NewBegin);
+  AtomListTy::iterator NewI = std::lower_bound(atom_begin(), atom_end(),
+                                               NewBegin, AtomComp);
   Atoms.insert(NewI, Atom);
 
   // Update the atom internal bounds.
@@ -73,14 +65,16 @@ void MCModule::remap(MCAtom *Atom, uint6
 }
 
 const MCAtom *MCModule::findAtomContaining(uint64_t Addr) const {
-  AtomListTy::const_iterator I = atom_lower_bound(Addr);
+  AtomListTy::const_iterator I = std::lower_bound(atom_begin(), atom_end(),
+                                                  Addr, AtomComp);
   if (I != atom_end() && (*I)->getBeginAddr() <= Addr)
     return *I;
   return 0;
 }
 
 MCAtom *MCModule::findAtomContaining(uint64_t Addr) {
-  AtomListTy::iterator I = atom_lower_bound(Addr);
+  AtomListTy::iterator I = std::lower_bound(atom_begin(), atom_end(),
+                                            Addr, AtomComp);
   if (I != atom_end() && (*I)->getBeginAddr() <= Addr)
     return *I;
   return 0;