remove the loc argument to Type::isConstantSizeType, which isn't useful.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@45155 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/AST/Type.cpp b/AST/Type.cpp
index ccd4630843d7496f3bf2f4ec60d7d88d56a88e62..04c236c31b374354da04f63426b820cf982a398b 100644 (file)
--- a/AST/Type.cpp
+++ b/AST/Type.cpp
@@ -446,17 +446,13 @@ bool Type::isAggregateType() const {
          CanonicalType->getTypeClass() == VariableArray;
 }
 
-// The only variable size types are auto arrays within a function. Structures 
-// cannot contain a VLA member. They can have a flexible array member, however
-// the structure is still constant size (C99 6.7.2.1p16).
-bool Type::isConstantSizeType(ASTContext &Ctx, SourceLocation *Loc) const {
+/// isConstantSizeType - Return true if this is not a variable sized type,
+/// according to the rules of C99 6.7.5p3.  It is not legal to call this on
+/// incomplete types.
+bool Type::isConstantSizeType(ASTContext &Ctx) const {
   assert(!isIncompleteType() && "This doesn't make sense for incomplete types");
-  if (const VariableArrayType *VAT =dyn_cast<VariableArrayType>(CanonicalType)){
-    // The VAT must have a size, as it is known to be complete.
-    if (Loc) *Loc = VAT->getSizeExpr()->getLocStart();
-    return false;
-  }
-  return true;
+  // The VAT must have a size, as it is known to be complete.
+  return !isa<VariableArrayType>(CanonicalType);
 }
 
 /// isIncompleteType - Return true if this is an incomplete type (C99 6.2.5p1)

diff --git a/include/clang/AST/Type.h b/include/clang/AST/Type.h
index 641e62f1ba5591f43f1f87ca1f7e34d192e26314..24388ea88894f2b335ff0457e28cdf1c41522587 100644 (file)
--- a/include/clang/AST/Type.h
+++ b/include/clang/AST/Type.h
@@ -338,10 +338,9 @@ public:
   bool isUnsignedIntegerType() const;
 
   /// isConstantSizeType - Return true if this is not a variable sized type,
-  /// according to the rules of C99 6.7.5p3.  If Loc is non-null, it is set to
-  /// the location of the subexpression that makes it a vla type.  It is not
-  /// legal to call this on incomplete types.
-  bool isConstantSizeType(ASTContext &Ctx, SourceLocation *Loc = 0) const;
+  /// according to the rules of C99 6.7.5p3.  It is not legal to call this on
+  /// incomplete types.
+  bool isConstantSizeType(ASTContext &Ctx) const;
 private:  
   QualType getCanonicalTypeInternal() const { return CanonicalType; }
   friend class QualType;