[llvm] 5d0c3a8 - [SVE] Remove VectorType::isScalable()

Christopher Tetreault via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 15:40:16 PDT 2020


Author: Christopher Tetreault
Date: 2020-04-23T15:39:59-07:00
New Revision: 5d0c3a8026ea708e72e1084aa659f5a4de68b88f

URL: https://github.com/llvm/llvm-project/commit/5d0c3a8026ea708e72e1084aa659f5a4de68b88f
DIFF: https://github.com/llvm/llvm-project/commit/5d0c3a8026ea708e72e1084aa659f5a4de68b88f.diff

LOG: [SVE] Remove VectorType::isScalable()

Summary:
* This is a property of the instance of VectorType. If
isa<ScalableVectorType>(T) is true, then T->isScalable() would have
returned true and vice-versa. Most code that checks this function uses
the result to bail out if a vector is a scalable vector. This code will
be cleaner if it just calls isa<ScalableVectorType>(T)

Reviewers: efriedma, craig.topper, huntergr, sdesmalen

Reviewed By: sdesmalen

Subscribers: tschuett, rkruppe, psnobl, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D77690

Added: 
    

Modified: 
    llvm/include/llvm/IR/DerivedTypes.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/IR/DerivedTypes.h b/llvm/include/llvm/IR/DerivedTypes.h
index dc0d2012d8fe..5dcecaef20e2 100644
--- a/llvm/include/llvm/IR/DerivedTypes.h
+++ b/llvm/include/llvm/IR/DerivedTypes.h
@@ -525,10 +525,6 @@ class VectorType : public Type {
   /// number of elements in the vector.
   ElementCount getElementCount() const { return EC; }
 
-  /// Returns whether or not this is a scalable vector (meaning the total
-  /// element count is a multiple of the minimum).
-  bool isScalable() const { return EC.Scalable; }
-
   /// Methods for support type inquiry through isa, cast, and dyn_cast.
   static bool classof(const Type *T) {
     return T->getTypeID() == FixedVectorTyID ||


        


More information about the llvm-commits mailing list