[llvm-commits] CVS: llvm/include/llvm/Support/ConstantRange.h

[Reid Spencer]

Changes in directory llvm/include/llvm/Support:

ConstantRange.h updated: 1.19 -> 1.20
---
Log message:

Remove the "isSigned" parameters from ConstantRange. It turns out they 
are not needed as the results are the same with or without it. 

Patch by Nicholas Lewycky.


---
Diffs of the changes:  (+5 -7)

 ConstantRange.h |   12 +++++-------
 1 files changed, 5 insertions(+), 7 deletions(-)


Index: llvm/include/llvm/Support/ConstantRange.h
diff -u llvm/include/llvm/Support/ConstantRange.h:1.19 llvm/include/llvm/Support/ConstantRange.h:1.20
--- llvm/include/llvm/Support/ConstantRange.h:1.19	Wed Feb 28 16:02:48 2007
+++ llvm/include/llvm/Support/ConstantRange.h	Thu Mar  1 01:54:15 2007
@@ -40,7 +40,7 @@
 class ConstantRange {
   APInt Lower, Upper;
   static ConstantRange intersect1Wrapped(const ConstantRange &LHS,
-                                         const ConstantRange &RHS, bool sign);
+                                         const ConstantRange &RHS);
  public:
   /// Initialize a full (the default) or empty set for the specified bit width.
   ///
@@ -79,13 +79,11 @@
   /// isWrappedSet - Return true if this set wraps around the top of the range,
   /// for example: [100, 8)
   ///
-  bool isWrappedSet(bool isSigned) const;
+  bool isWrappedSet() const;
 
   /// contains - Return true if the specified value is in the set.
-  /// The isSigned parameter indicates whether the comparisons should be
-  /// performed as if the values are signed or not.
   ///
-  bool contains(const APInt &Val, bool isSigned) const;
+  bool contains(const APInt &Val) const;
 
   /// getSingleElement - If this set contains a single element, return it,
   /// otherwise return null.
@@ -123,7 +121,7 @@
   /// one of the sets but not the other.  For example: [100, 8) intersect [3,
   /// 120) yields [3, 120)
   ///
-  ConstantRange intersectWith(const ConstantRange &CR, bool isSigned) const;
+  ConstantRange intersectWith(const ConstantRange &CR) const;
 
   /// unionWith - Return the range that results from the union of this range
   /// with another range.  The resultant range is guaranteed to include the
@@ -131,7 +129,7 @@
   /// [12,15) is [3, 15), which includes 9, 10, and 11, which were not included
   /// in either set before.
   ///
-  ConstantRange unionWith(const ConstantRange &CR, bool isSigned) const;
+  ConstantRange unionWith(const ConstantRange &CR) const;
 
   /// zeroExtend - Return a new range in the specified integer type, which must
   /// be strictly larger than the current type.  The returned range will