From: John McCall Date: Thu, 14 Oct 2010 02:06:32 +0000 (+0000) Subject: Whoops. This really shouldn't compile in clang, either. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=008df5dce3938456ae7ea2e7ab3b2d12391ebf3e;p=clang 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 5901737389..b85d626bd7 100644 --- 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());