[llvm-commits] CVS: llvm/include/llvm/ADT/SetOperations.h

Chris Lattner lattner at cs.uiuc.edu
Wed Oct 13 08:09:35 PDT 2004



Changes in directory llvm/include/llvm/ADT:

SetOperations.h updated: 1.6 -> 1.7
---
Log message:

Get rid of template templates that were preventing VC from compiling the
set_intersect template.  Thanks to Morten Ofstad and Jeff Cohen for the
patch!



---
Diffs of the changes:  (+18 -18)

Index: llvm/include/llvm/ADT/SetOperations.h
diff -u llvm/include/llvm/ADT/SetOperations.h:1.6 llvm/include/llvm/ADT/SetOperations.h:1.7
--- llvm/include/llvm/ADT/SetOperations.h:1.6	Wed Sep  1 17:55:34 2004
+++ llvm/include/llvm/ADT/SetOperations.h	Wed Oct 13 10:09:21 2004
@@ -17,8 +17,8 @@
 
 namespace llvm {
 
-// set_union(A, B) - Compute A := A u B, return whether A changed.
-//
+/// set_union(A, B) - Compute A := A u B, return whether A changed.
+///
 template <class S1Ty, class S2Ty>
 bool set_union(S1Ty &S1, const S2Ty &S2) {   
   bool Changed = false;
@@ -31,22 +31,22 @@
   return Changed;
 }
 
-// set_intersect(A, B) - Compute A := A ^ B
-// Identical to set_intersection, except that it works on set<>'s and
-// is nicer to use.  Functionally, this iterates through S1, removing
-// elements that are not contained in S2.
-//
-template <template<class S1ElTy> class S1Ty, class ETy, class S2Ty>
-void set_intersect(S1Ty<ETy> &S1, const S2Ty &S2) {
-  for (typename S1Ty<ETy>::iterator I = S1.begin(); I != S1.end();) {
-    const ETy &E = *I;
-    ++I;
-    if (!S2.count(E)) S1.erase(E);   // Erase element if not in S2
-  }
+/// set_intersect(A, B) - Compute A := A ^ B
+/// Identical to set_intersection, except that it works on set<>'s and
+/// is nicer to use.  Functionally, this iterates through S1, removing
+/// elements that are not contained in S2.
+///
+template <class S1Ty, class S2Ty>
+void set_intersect(S1Ty &S1, const S2Ty &S2) {
+   for (typename S1Ty::iterator I = S1.begin(); I != S1.end();) {
+     const typename S1Ty::key_type &E = *I;
+     ++I;
+     if (!S2.count(E)) S1.erase(E);   // Erase element if not in S2
+   }
 }
 
-// set_difference(A, B) - Return A - B
-//
+/// set_difference(A, B) - Return A - B
+///
 template <class S1Ty, class S2Ty>
 S1Ty set_difference(const S1Ty &S1, const S2Ty &S2) {
   S1Ty Result;
@@ -57,8 +57,8 @@
   return Result;
 }
 
-// set_subtract(A, B) - Compute A := A - B
-//
+/// set_subtract(A, B) - Compute A := A - B
+///
 template <class S1Ty, class S2Ty>
 void set_subtract(S1Ty &S1, const S2Ty &S2) { 
   for (typename S2Ty::const_iterator SI = S2.begin(), SE = S2.end();






More information about the llvm-commits mailing list