Whoops. This really shouldn't compile in clang, either.

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

diff --git a/lib/Sema/SemaType.cpp b/lib/Sema/SemaType.cpp
index 5901737389e249996720e06f367a3710030532ad..b85d626bd73fb302d8c12e029251dd0dd5d686a5 100644 (file)
--- a/lib/Sema/SemaType.cpp
+++ b/lib/Sema/SemaType.cpp
@@ -1902,14 +1902,14 @@ bool ProcessFnAttr(Sema &S, QualType &Type, const AttributeList &Attr) {
       S.Diag(Attr.getLoc(), diag::err_attribute_regparm_wrong_platform)
         << NumParamsExpr->getSourceRange();
       Attr.setInvalid();
-      return;
+      return false;
     }
 
     if (NumParams.getLimitedValue(255) > S.Context.Target.getRegParmMax()) {
       S.Diag(Attr.getLoc(), diag::err_attribute_regparm_invalid_number)
         << S.Context.Target.getRegParmMax() << NumParamsExpr->getSourceRange();
       Attr.setInvalid();
-      return;
+      return false;
     }
 
     Type = S.Context.getRegParmType(Type, NumParams.getZExtValue());