r186591 - Switching to use checkAttributeNumArgs for trivial cases. No functional changes intended.
Aaron Ballman
aaron at aaronballman.com
Thu Jul 18 11:01:48 PDT 2013
Author: aaronballman
Date: Thu Jul 18 13:01:48 2013
New Revision: 186591
URL: http://llvm.org/viewvc/llvm-project?rev=186591&view=rev
Log:
Switching to use checkAttributeNumArgs for trivial cases. No functional changes intended.
Modified:
cfe/trunk/lib/Sema/SemaDeclAttr.cpp
Modified: cfe/trunk/lib/Sema/SemaDeclAttr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclAttr.cpp?rev=186591&r1=186590&r2=186591&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDeclAttr.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDeclAttr.cpp Thu Jul 18 13:01:48 2013
@@ -1547,10 +1547,8 @@ static void handleWeakRefAttr(Sema &S, D
static void handleAliasAttr(Sema &S, Decl *D, const AttributeList &Attr) {
// check the attribute arguments.
- if (Attr.getNumArgs() != 1) {
- S.Diag(Attr.getLoc(), diag::err_attribute_wrong_number_arguments) << 1;
+ if (!checkAttributeNumArgs(S, Attr, 1))
return;
- }
Expr *Arg = Attr.getArg(0);
Arg = Arg->IgnoreParenCasts();
@@ -1670,10 +1668,8 @@ static void handleAlwaysInlineAttr(Sema
static void handleTLSModelAttr(Sema &S, Decl *D,
const AttributeList &Attr) {
// Check the attribute arguments.
- if (Attr.getNumArgs() != 1) {
- S.Diag(Attr.getLoc(), diag::err_attribute_wrong_number_arguments) << 1;
+ if (!checkAttributeNumArgs(S, Attr, 1))
return;
- }
Expr *Arg = Attr.getArg(0);
Arg = Arg->IgnoreParenCasts();
@@ -3130,8 +3126,7 @@ static void handleInitPriorityAttr(Sema
return;
}
- if (Attr.getNumArgs() != 1) {
- S.Diag(Attr.getLoc(), diag::err_attribute_wrong_number_arguments) << 1;
+ if (!checkAttributeNumArgs(S, Attr, 1)) {
Attr.setInvalid();
return;
}
@@ -4146,8 +4141,7 @@ bool Sema::CheckRegparmAttr(const Attrib
if (Attr.isInvalid())
return true;
- if (Attr.getNumArgs() != 1) {
- Diag(Attr.getLoc(), diag::err_attribute_wrong_number_arguments) << 1;
+ if (!checkAttributeNumArgs(*this, Attr, 1)) {
Attr.setInvalid();
return true;
}
More information about the cfe-commits
mailing list