diff --git a/include/clang/AST/ExprObjC.h b/include/clang/AST/ExprObjC.h index 09449118678a7da6640b5f6a743586e4053b20f7..d141fda395d7bb09d5ddb5e63826c9c2c734a438 100644 --- a/include/clang/AST/ExprObjC.h +++ b/include/clang/AST/ExprObjC.h @@ -444,9 +444,16 @@ private: /// class, and can be distinguished via \c getReceiverKind(). Example: /// class ObjCMessageExpr : public Expr { + enum { NumArgsBitWidth = 16 }; + /// \brief The number of arguments in the message send, not /// including the receiver. - unsigned NumArgs : 16; + unsigned NumArgs : NumArgsBitWidth; + + void setNumArgs(unsigned Num) { + assert((Num >> NumArgsBitWidth) == 0 && "Num of args is out of range!"); + NumArgs = Num; + } /// \brief The kind of message send this is, which is one of the /// ReceiverKind values. @@ -482,8 +489,10 @@ class ObjCMessageExpr : public Expr { SourceLocation LBracLoc, RBracLoc; ObjCMessageExpr(EmptyShell Empty, unsigned NumArgs) - : Expr(ObjCMessageExprClass, Empty), NumArgs(NumArgs), Kind(0), - HasMethod(0), IsDelegateInitCall(0), SelectorOrMethod(0) { } + : Expr(ObjCMessageExprClass, Empty), Kind(0), + HasMethod(0), IsDelegateInitCall(0), SelectorOrMethod(0) { + setNumArgs(NumArgs); + } ObjCMessageExpr(QualType T, ExprValueKind VK, SourceLocation LBracLoc, diff --git a/lib/AST/Expr.cpp b/lib/AST/Expr.cpp index 232709cb262cd907a84ddb363baf7ea303609739..b0f23f9717eb49dfe2ecb3156df029bccf76d0a8 100644 --- a/lib/AST/Expr.cpp +++ b/lib/AST/Expr.cpp @@ -2724,12 +2724,13 @@ ObjCMessageExpr::ObjCMessageExpr(QualType T, /*TypeDependent=*/false, /*ValueDependent=*/false, /*InstantiationDependent=*/false, /*ContainsUnexpandedParameterPack=*/false), - NumArgs(NumArgs), Kind(IsInstanceSuper? SuperInstance : SuperClass), + Kind(IsInstanceSuper? SuperInstance : SuperClass), HasMethod(Method != 0), IsDelegateInitCall(false), SuperLoc(SuperLoc), SelectorOrMethod(reinterpret_cast<uintptr_t>(Method? Method : Sel.getAsOpaquePtr())), SelectorLoc(SelLoc), LBracLoc(LBracLoc), RBracLoc(RBracLoc) { + setNumArgs(NumArgs); setReceiverPointer(SuperType.getAsOpaquePtr()); if (NumArgs) memcpy(getArgs(), Args, NumArgs * sizeof(Expr *)); @@ -2747,12 +2748,13 @@ ObjCMessageExpr::ObjCMessageExpr(QualType T, : Expr(ObjCMessageExprClass, T, VK, OK_Ordinary, T->isDependentType(), T->isDependentType(), T->isInstantiationDependentType(), T->containsUnexpandedParameterPack()), - NumArgs(NumArgs), Kind(Class), + Kind(Class), HasMethod(Method != 0), IsDelegateInitCall(false), SelectorOrMethod(reinterpret_cast<uintptr_t>(Method? Method : Sel.getAsOpaquePtr())), SelectorLoc(SelLoc), LBracLoc(LBracLoc), RBracLoc(RBracLoc) { + setNumArgs(NumArgs); setReceiverPointer(Receiver); Expr **MyArgs = getArgs(); for (unsigned I = 0; I != NumArgs; ++I) { @@ -2782,12 +2784,13 @@ ObjCMessageExpr::ObjCMessageExpr(QualType T, Receiver->isTypeDependent(), Receiver->isInstantiationDependent(), Receiver->containsUnexpandedParameterPack()), - NumArgs(NumArgs), Kind(Instance), + Kind(Instance), HasMethod(Method != 0), IsDelegateInitCall(false), SelectorOrMethod(reinterpret_cast<uintptr_t>(Method? Method : Sel.getAsOpaquePtr())), SelectorLoc(SelLoc), LBracLoc(LBracLoc), RBracLoc(RBracLoc) { + setNumArgs(NumArgs); setReceiverPointer(Receiver); Expr **MyArgs = getArgs(); for (unsigned I = 0; I != NumArgs; ++I) {