diff options
Diffstat (limited to 'clang/lib/Sema/SemaDeclAttr.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDeclAttr.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/clang/lib/Sema/SemaDeclAttr.cpp b/clang/lib/Sema/SemaDeclAttr.cpp index 40e4971d2b8..a68ddf6c003 100644 --- a/clang/lib/Sema/SemaDeclAttr.cpp +++ b/clang/lib/Sema/SemaDeclAttr.cpp @@ -1465,17 +1465,16 @@ static void HandleRegparmAttr(Decl *d, const AttributeList &Attr, Sema &S) { return; } - if (NumParams.getLimitedValue(4) > 3) { - S.Diag(Attr.getLoc(), diag::err_attribute_regparm_invalid_number) + if (S.Context.Target.getRegParmMax() == 0) { + S.Diag(Attr.getLoc(), diag::err_attribute_regparm_wrong_platform) << NumParamsExpr->getSourceRange(); return; } - const char *TargetPrefix = S.Context.Target.getTargetPrefix(); - unsigned PointerWidth = S.Context.Target.getPointerWidth(0); - if (strcmp(TargetPrefix, "x86") || PointerWidth != 32) { - S.Diag(Attr.getLoc(), diag::err_attribute_regparm_wrong_platform) - << NumParamsExpr->getSourceRange(); + // FIXME: we need to honour command line settings also... + if (NumParams.getLimitedValue(4) > S.Context.Target.getRegParmMax()) { + S.Diag(Attr.getLoc(), diag::err_attribute_regparm_invalid_number) + << S.Context.Target.getRegParmMax() << NumParamsExpr->getSourceRange(); return; } |