]> granicus.if.org Git - clang/commitdiff
Simplifying a test for duplicate vec_type_hint attributes. The assert and attribute...
authorAaron Ballman <aaron@aaronballman.com>
Mon, 2 Dec 2013 22:16:55 +0000 (22:16 +0000)
committerAaron Ballman <aaron@aaronballman.com>
Mon, 2 Dec 2013 22:16:55 +0000 (22:16 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@196162 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Sema/SemaDeclAttr.cpp

index 21f3526da8c3cebf43d7c9bb99117e108f4acaba..60bc7b79ada6fe1a4d231ed94f39f055edad91fe 100644 (file)
@@ -2319,8 +2319,6 @@ static void handleWorkGroupSize(Sema &S, Decl *D,
 }
 
 static void handleVecTypeHint(Sema &S, Decl *D, const AttributeList &Attr) {
-  assert(Attr.getKind() == AttributeList::AT_VecTypeHint);
-
   if (!Attr.hasParsedType()) {
     S.Diag(Attr.getLoc(), diag::err_attribute_wrong_number_arguments)
       << Attr.getName() << 1;
@@ -2339,9 +2337,7 @@ static void handleVecTypeHint(Sema &S, Decl *D, const AttributeList &Attr) {
     return;
   }
 
-  if (Attr.getKind() == AttributeList::AT_VecTypeHint &&
-      D->hasAttr<VecTypeHintAttr>()) {
-    VecTypeHintAttr *A = D->getAttr<VecTypeHintAttr>();
+  if (VecTypeHintAttr *A = D->getAttr<VecTypeHintAttr>()) {
     if (!S.Context.hasSameType(A->getTypeHint(), ParmType)) {
       S.Diag(Attr.getLoc(), diag::warn_duplicate_attribute) << Attr.getName();
       return;