]> granicus.if.org Git - clang/commitdiff
Use else if when checking multiple attributes.
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 5 Dec 2014 18:03:55 +0000 (18:03 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 5 Dec 2014 18:03:55 +0000 (18:03 +0000)
Only one of these can really match.

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

lib/Sema/SemaDeclAttr.cpp

index e5eb61c1eb4d3ee940dd70677dbb94d72ecc0d9c..af7cf5713de2df3e0bcc0f7c399e5f0ea484daa0 100644 (file)
@@ -4764,12 +4764,10 @@ void Sema::ProcessDeclAttributeList(Scope *S, Decl *D,
     if (Attr *A = D->getAttr<ReqdWorkGroupSizeAttr>()) {
       Diag(D->getLocation(), diag::err_opencl_kernel_attr) << A;
       D->setInvalidDecl();
-    }
-    if (Attr *A = D->getAttr<WorkGroupSizeHintAttr>()) {
+    } else if (Attr *A = D->getAttr<WorkGroupSizeHintAttr>()) {
       Diag(D->getLocation(), diag::err_opencl_kernel_attr) << A;
       D->setInvalidDecl();
-    }
-    if (Attr *A = D->getAttr<VecTypeHintAttr>()) {
+    } else if (Attr *A = D->getAttr<VecTypeHintAttr>()) {
       Diag(D->getLocation(), diag::err_opencl_kernel_attr) << A;
       D->setInvalidDecl();
     }