Revert 'Fix a typo 'iff' => 'if''. iff is an abreviation of if and only if. See:...
authorSylvestre Ledru <sylvestre@debian.org>
Thu, 27 Sep 2012 10:16:10 +0000 (10:16 +0000)
committerSylvestre Ledru <sylvestre@debian.org>
Thu, 27 Sep 2012 10:16:10 +0000 (10:16 +0000)
llvm-svn: 164769

23 files changed:
clang/INPUTS/c99-intconst-1.c
clang/include/clang/AST/ASTContext.h
clang/include/clang/AST/DeclObjC.h
clang/include/clang/Analysis/AnalysisContext.h
clang/include/clang/Basic/Diagnostic.h
clang/include/clang/Sema/Sema.h
clang/lib/CodeGen/CGBuiltin.cpp
clang/lib/CodeGen/CGDecl.cpp
clang/lib/CodeGen/CGObjCMac.cpp
clang/lib/CodeGen/CGRTTI.cpp
clang/lib/CodeGen/CGVTables.h
clang/lib/CodeGen/CodeGenFunction.h
clang/lib/CodeGen/CodeGenModule.h
clang/lib/CodeGen/CodeGenTBAA.cpp
clang/lib/CodeGen/TargetInfo.cpp
clang/lib/Driver/Option.cpp
clang/lib/Frontend/TextDiagnostic.cpp
clang/lib/Sema/AnalysisBasedWarnings.cpp
clang/lib/Sema/SemaAccess.cpp
clang/lib/Sema/SemaChecking.cpp
clang/lib/Sema/SemaDecl.cpp
clang/lib/StaticAnalyzer/Checkers/MacOSKeychainAPIChecker.cpp
clang/test/SemaCXX/discrim-union.cpp

index 3b6bb6c75fc9d333adef2e0b61b6b123e506f651..629b0bcac2057e44e340f80cbb1820f6147382d5 100644 (file)
        CHECK_OCTHEX_CONST(0x##C);              \
        CHECK_OCTHEX_CONST(0X##C);
 
-/* True if "long long" is at least B bits.  This presumes that (B-2)/3 is at
+/* True iff "long long" is at least B bits.  This presumes that (B-2)/3 is at
    most 63.  */
 #define LLONG_AT_LEAST(B)                      \
        (LLONG_MAX >> ((B)-2)/3 >> ((B)-2)/3    \
index 27a910f9acd393ba2007ca652810d82ccfc07433..eeaba7a71da97e9e9ad8606346f856f77e815986 100644 (file)
@@ -840,7 +840,7 @@ public:
   /// Builds the struct used for __block variables.
   QualType BuildByRefType(StringRef DeclName, QualType Ty) const;
 
-  /// Returns true if we need copy/dispose helpers for the given type.
+  /// Returns true iff we need copy/dispose helpers for the given type.
   bool BlockRequiresCopying(QualType Ty) const;
 
   /// \brief Return the uniqued reference to the type for an lvalue reference
index c815babcd4137042394c1de65db8bd4184961ca1..9e0130ad1651ee40e6ccc0281f48e3295a9274b3 100644 (file)
@@ -1821,7 +1821,7 @@ public:
 
   // Helper methods for accessing attributes.
 
-  /// isReadOnly - Return true if the property has a setter.
+  /// isReadOnly - Return true iff the property has a setter.
   bool isReadOnly() const {
     return (PropertyAttributes & OBJC_PR_readonly);
   }
index 639550f850ee8ee6cc760cc5f1168f78ff2a8f21..02b85304dbd85b9e05ba723f8da31210e8a3272d 100644 (file)
@@ -119,7 +119,7 @@ public:
     return cfgBuildOptions;
   }
 
-  /// getAddEHEdges - Return true if we are adding exceptional edges from
+  /// getAddEHEdges - Return true iff we are adding exceptional edges from
   /// callExprs.  If this is false, then try/catch statements and blocks
   /// reachable from them can appear to be dead in the CFG, analysis passes must
   /// cope with that.
index abc5733ee16c038146cd8574d38abacf502a5964..b596937061e1ae39030701160df6f5d8bfc7a4aa 100644 (file)
@@ -831,7 +831,7 @@ class DiagnosticBuilder {
 
   /// \brief Status variable indicating if this diagnostic is still active.
   ///
-  // NOTE: This field is redundant with DiagObj (IsActive if (DiagObj == 0)),
+  // NOTE: This field is redundant with DiagObj (IsActive iff (DiagObj == 0)),
   // but LLVM is not currently smart enough to eliminate the null check that
   // Emit() would end up with if we used that as our status variable.
   mutable bool IsActive;
index c1f0923c34f9d8430b21cffc5f46fae0e2a551cb..97d425ceaea26becb05055d63fb416b089d6ab1b 100644 (file)
@@ -6824,7 +6824,7 @@ public:
   /// CheckMessageArgumentTypes - Check types in an Obj-C message send.
   /// \param Method - May be null.
   /// \param [out] ReturnType - The return type of the send.
-  /// \return true if there were any incompatible types.
+  /// \return true iff there were any incompatible types.
   bool CheckMessageArgumentTypes(QualType ReceiverType,
                                  Expr **Args, unsigned NumArgs, Selector Sel,
                                  ArrayRef<SourceLocation> SelectorLocs,
@@ -6851,7 +6851,7 @@ public:
   ///
   /// \param Loc - A location associated with the condition, e.g. the
   /// 'if' keyword.
-  /// \return true if there were any errors
+  /// \return true iff there were any errors
   ExprResult CheckBooleanCondition(Expr *E, SourceLocation Loc);
 
   ExprResult ActOnBooleanCondition(Scope *S, SourceLocation Loc,
index 4a3b3e9794eca549704690b0f678f8e51663de13..7a0abd71a683548c038c77bcb9700d8ed85104b5 100644 (file)
@@ -597,7 +597,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
   }
 
   case Builtin::BI__builtin___memcpy_chk: {
-    // fold __builtin_memcpy_chk(x, y, cst1, cst2) to memcpy if cst1<=cst2.
+    // fold __builtin_memcpy_chk(x, y, cst1, cst2) to memcpy iff cst1<=cst2.
     llvm::APSInt Size, DstSize;
     if (!E->getArg(2)->EvaluateAsInt(Size, CGM.getContext()) ||
         !E->getArg(3)->EvaluateAsInt(DstSize, CGM.getContext()))
@@ -624,7 +624,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
   }
 
   case Builtin::BI__builtin___memmove_chk: {
-    // fold __builtin_memmove_chk(x, y, cst1, cst2) to memmove if cst1<=cst2.
+    // fold __builtin_memmove_chk(x, y, cst1, cst2) to memmove iff cst1<=cst2.
     llvm::APSInt Size, DstSize;
     if (!E->getArg(2)->EvaluateAsInt(Size, CGM.getContext()) ||
         !E->getArg(3)->EvaluateAsInt(DstSize, CGM.getContext()))
@@ -663,7 +663,7 @@ RValue CodeGenFunction::EmitBuiltinExpr(const FunctionDecl *FD,
     return RValue::get(Dest.first);
   }
   case Builtin::BI__builtin___memset_chk: {
-    // fold __builtin_memset_chk(x, y, cst1, cst2) to memset if cst1<=cst2.
+    // fold __builtin_memset_chk(x, y, cst1, cst2) to memset iff cst1<=cst2.
     llvm::APSInt Size, DstSize;
     if (!E->getArg(2)->EvaluateAsInt(Size, CGM.getContext()) ||
         !E->getArg(3)->EvaluateAsInt(DstSize, CGM.getContext()))
index 88a252a697d4633b39c963c7f47b7aa17610ca04..b9489e3f0409b02cb0ac29e655ce88e026420f54 100644 (file)
@@ -1179,7 +1179,7 @@ void CodeGenFunction::emitAutoVarTypeCleanup(
   // If we haven't chosen a more specific destroyer, use the default.
   if (!destroyer) destroyer = getDestroyer(dtorKind);
 
-  // Use an EH cleanup in array destructors if the destructor itself
+  // Use an EH cleanup in array destructors iff the destructor itself
   // is being pushed as an EH cleanup.
   bool useEHCleanup = (cleanupKind & EHCleanup);
   EHStack.pushCleanup<DestroyObject>(cleanupKind, addr, type, destroyer,
index e9aa3b74acc2bbc5fe2c8f6cf6c5ef66aafe6e3a..08e706c82e7fccc719807107e649ebdfac56f0ff 100644 (file)
@@ -1028,8 +1028,8 @@ private:
   /// EmitMethodDescList - Emit a method description list for a list of
   /// method declarations.
   ///  - TypeName: The name for the type containing the methods.
-  ///  - IsProtocol: True if these methods are for a protocol.
-  ///  - ClassMethds: True if these are class methods.
+  ///  - IsProtocol: True iff these methods are for a protocol.
+  ///  - ClassMethds: True iff these are class methods.
   ///  - Required: When true, only "required" methods are
   ///    listed. Similarly, when false only "optional" methods are
   ///    listed. For classes this should always be true.
index ee1a7701eec8cf602e7d255047199b37faf964a0..e46423b95ae98e5f2845ac2c803ca123b76f111d 100644 (file)
@@ -347,7 +347,7 @@ getTypeInfoLinkage(CodeGenModule &CGM, QualType Ty) {
 
 // CanUseSingleInheritance - Return whether the given record decl has a "single, 
 // public, non-virtual base at offset zero (i.e. the derived class is dynamic 
-// if the base is)", according to Itanium C++ ABI, 2.95p6b.
+// iff the base is)", according to Itanium C++ ABI, 2.95p6b.
 static bool CanUseSingleInheritance(const CXXRecordDecl *RD) {
   // Check the number of bases.
   if (RD->getNumBases() != 1)
@@ -364,7 +364,7 @@ static bool CanUseSingleInheritance(const CXXRecordDecl *RD) {
   if (Base->getAccessSpecifier() != AS_public)
     return false;
   
-  // Check that the class is dynamic if the base is.
+  // Check that the class is dynamic iff the base is.
   const CXXRecordDecl *BaseDecl = 
     cast<CXXRecordDecl>(Base->getType()->getAs<RecordType>()->getDecl());
   if (!BaseDecl->isEmpty() && 
index f5b59c819d743431c8ab4e67ed3d8085777f0aca..828330e5e3c4a3bcc414f20bbdc5f13edf6977c7 100644 (file)
@@ -58,7 +58,7 @@ class CodeGenVTables {
 
   /// MaybeEmitThunkAvailableExternally - Try to emit the given thunk with
   /// available_externally linkage to allow for inlining of thunks.
-  /// This will be done if optimizations are enabled and the member function
+  /// This will be done iff optimizations are enabled and the member function
   /// doesn't contain any incomplete types.
   void MaybeEmitThunkAvailableExternally(GlobalDecl GD, const ThunkInfo &Thunk);
 
index a31a64e6cc123f4f604a72facb785267afb93c29..fc930ec158b005c5865065f3b64bc9a055e02462 100644 (file)
@@ -583,7 +583,7 @@ public:
   JumpDest ReturnBlock;
 
   /// ReturnValue - The temporary alloca to hold the return value. This is null
-  /// if the function has no return value.
+  /// iff the function has no return value.
   llvm::Value *ReturnValue;
 
   /// AllocaInsertPoint - This is an instruction in the entry block before which
@@ -1655,7 +1655,7 @@ public:
 
   /// EmitAggregateCopy - Emit an aggrate copy.
   ///
-  /// \param isVolatile - True if either the source or the destination is
+  /// \param isVolatile - True iff either the source or the destination is
   /// volatile.
   void EmitAggregateCopy(llvm::Value *DestPtr, llvm::Value *SrcPtr,
                          QualType EltTy, bool isVolatile=false,
index 0e15e7b3a6f72b2e0abf2e4f97e7c6231516b3cb..6e81693f7fd56b4a3bd3f5583181eeda259cdaf3 100644 (file)
@@ -372,7 +372,7 @@ public:
     return *ObjCRuntime;
   }
 
-  /// hasObjCRuntime() - Return true if an Objective-C runtime has
+  /// hasObjCRuntime() - Return true iff an Objective-C runtime has
   /// been configured.
   bool hasObjCRuntime() { return !!ObjCRuntime; }
 
@@ -788,15 +788,15 @@ public:
   /// which only apply to a function definintion.
   void SetLLVMFunctionAttributesForDefinition(const Decl *D, llvm::Function *F);
 
-  /// ReturnTypeUsesSRet - Return true if the given type uses 'sret' when used
+  /// ReturnTypeUsesSRet - Return true iff the given type uses 'sret' when used
   /// as a return type.
   bool ReturnTypeUsesSRet(const CGFunctionInfo &FI);
 
-  /// ReturnTypeUsesFPRet - Return true if the given type uses 'fpret' when
+  /// ReturnTypeUsesFPRet - Return true iff the given type uses 'fpret' when
   /// used as a return type.
   bool ReturnTypeUsesFPRet(QualType ResultType);
 
-  /// ReturnTypeUsesFP2Ret - Return true if the given type uses 'fp2ret' when
+  /// ReturnTypeUsesFP2Ret - Return true iff the given type uses 'fp2ret' when
   /// used as a return type.
   bool ReturnTypeUsesFP2Ret(QualType ResultType);
 
index 3d5725db68c34ace73dd84a6064b26b156283c47..bab60afbb7f2f0a34ae9d5060fad4024f691649d 100644 (file)
@@ -135,7 +135,7 @@ CodeGenTBAA::getTBAAInfo(QualType QTy) {
   // Enum types are distinct types. In C++ they have "underlying types",
   // however they aren't related for TBAA.
   if (const EnumType *ETy = dyn_cast<EnumType>(Ty)) {
-    // In C mode, two anonymous enums are compatible if their members
+    // In C mode, two anonymous enums are compatible iff their members
     // are the same -- see C99 6.2.7p1. For now, be conservative. We could
     // theoretically implement this by combining information about all the
     // members into a single identifying MDNode.
index 15ccfd50fc89292f5b50be85a495db5907d7e932..419ce022e5575dbeb23cf4d03f76b45a7af14165 100644 (file)
@@ -109,7 +109,7 @@ bool TargetCodeGenInfo::isNoProtoCallVariadic(const CallArgList &args,
 
 static bool isEmptyRecord(ASTContext &Context, QualType T, bool AllowArrays);
 
-/// isEmptyField - Return true if a the field is "empty", that is it
+/// isEmptyField - Return true iff a the field is "empty", that is it
 /// is an unnamed bit-field or an (array of) empty record(s).
 static bool isEmptyField(ASTContext &Context, const FieldDecl *FD,
                          bool AllowArrays) {
@@ -141,7 +141,7 @@ static bool isEmptyField(ASTContext &Context, const FieldDecl *FD,
   return isEmptyRecord(Context, FT, AllowArrays);
 }
 
-/// isEmptyRecord - Return true if a structure contains only empty
+/// isEmptyRecord - Return true iff a structure contains only empty
 /// fields. Note that a structure with a flexible array member is not
 /// considered empty.
 static bool isEmptyRecord(ASTContext &Context, QualType T, bool AllowArrays) {
@@ -1070,7 +1070,7 @@ class X86_64ABIInfo : public ABIInfo {
   /// be passed in Memory then at least the classification of \arg Lo
   /// will be Memory.
   ///
-  /// The \arg Lo class will be NoClass if the argument is ignored.
+  /// The \arg Lo class will be NoClass iff the argument is ignored.
   ///
   /// If the \arg Lo class is ComplexX87, then the \arg Hi class will
   /// also be ComplexX87.
index 6a3418fa1786fbbbad1fa0c5488bd032c74a550b..3be141e61da056f72772a7dbaa1322cafdc33ab7 100644 (file)
@@ -120,7 +120,7 @@ Arg *Option::accept(const ArgList &Args, unsigned &Index) const {
     return A;
   }
   case SeparateClass:
-    // Matches if this is an exact match.
+    // Matches iff this is an exact match.
     // FIXME: Avoid strlen.
     if (getName().size() != strlen(Args.getArgString(Index)))
       return 0;
@@ -132,7 +132,7 @@ Arg *Option::accept(const ArgList &Args, unsigned &Index) const {
     return new Arg(getUnaliasedOption(),
                    Index - 2, Args.getArgString(Index - 1));
   case MultiArgClass: {
-    // Matches if this is an exact match.
+    // Matches iff this is an exact match.
     // FIXME: Avoid strlen.
     if (getName().size() != strlen(Args.getArgString(Index)))
       return 0;
index d872e426913f781b1874c60d80cfe9bc1933999a..a8a5613eaace114a1248c4000485aa129f9f7976 100644 (file)
@@ -88,7 +88,7 @@ static int bytesSincePreviousTabOrLineBegin(StringRef SourceLine, size_t i) {
 /// \param SourceLine The line of source
 /// \param i Pointer to byte index,
 /// \param TabStop used to expand tabs
-/// \return pair(printable text, 'true' if original text was printable)
+/// \return pair(printable text, 'true' iff original text was printable)
 ///
 static std::pair<SmallString<16>, bool>
 printableTextForNextCharacter(StringRef SourceLine, size_t *i,
index 2b7d4becd90031d061e9ec45deaa28127e0bec2b..a3aee9afe0831c0a9644808d8074e9b8f541c33c 100644 (file)
@@ -87,10 +87,10 @@ enum ControlFlowKind {
 /// CheckFallThrough - Check that we don't fall off the end of a
 /// Statement that should return a value.
 ///
-/// \returns AlwaysFallThrough if we always fall off the end of the statement,
-/// MaybeFallThrough if we might or might not fall off the end,
-/// NeverFallThroughOrReturn if we never fall off the end of the statement or
-/// return.  We assume NeverFallThrough if we never fall off the end of the
+/// \returns AlwaysFallThrough iff we always fall off the end of the statement,
+/// MaybeFallThrough iff we might or might not fall off the end,
+/// NeverFallThroughOrReturn iff we never fall off the end of the statement or
+/// return.  We assume NeverFallThrough iff we never fall off the end of the
 /// statement but we may return.  We assume that functions not marked noreturn
 /// will return.
 static ControlFlowKind CheckFallThrough(AnalysisDeclContext &AC) {
index ddb5e4e367409816a164951785455d0b10d4772a..58b1a51ae573bf7278f50986b1ebc82392d8bd6c 100644 (file)
@@ -918,7 +918,7 @@ static AccessResult HasAccess(Sema &S,
 ///     let AccessToBase = Merge(Access(B_i, B_{i+1}), ACAB(i+1)) in
 ///     if Accessible(B_i, AccessToBase) then public else AccessToBase
 ///
-/// B is an accessible base of N at R if ACAB(1) = public.
+/// B is an accessible base of N at R iff ACAB(1) = public.
 ///
 /// \param FinalAccess the access of the "final step", or AS_public if
 ///   there is no final step.
index 0fa87483ba0a20f6415fe9acc9437de0a2201581..7951a71e4ed0a99bf8e73531300ab628106e7c2d 100644 (file)
@@ -5366,7 +5366,7 @@ namespace {
 /// Consider whether capturing the given variable can possibly lead to
 /// a retain cycle.
 static bool considerVariable(VarDecl *var, Expr *ref, RetainCycleOwner &owner) {
-  // In ARC, it's captured strongly if the variable has __strong
+  // In ARC, it's captured strongly iff the variable has __strong
   // lifetime.  In MRR, it's captured strongly if the variable is
   // __block and has an appropriate type.
   if (var->getType().getObjCLifetime() != Qualifiers::OCL_Strong)
index 17db68ed59326b52a7c91c9bdfcf78b1ae5c0585..01aaf8be323bb13ef639763454743ca75a4c53ea 100644 (file)
@@ -4799,7 +4799,7 @@ class DifferentNameValidatorCCC : public CorrectionCandidateCallback {
 /// or performing typo correction if there are no previous declarations with
 /// the same name.
 ///
-/// Returns a NamedDecl if typo correction was performed and substituting in
+/// Returns a NamedDecl iff typo correction was performed and substituting in
 /// the new declaration name does not cause new errors.
 static NamedDecl* DiagnoseInvalidRedeclaration(
     Sema &SemaRef, LookupResult &Previous, FunctionDecl *NewFD,
@@ -8237,7 +8237,7 @@ static unsigned getRedeclDiagFromTagKind(TagTypeKind Tag) {
 /// \brief Determine if tag kind is a class-key compatible with
 /// class for redeclaration (class, struct, or __interface).
 ///
-/// \returns true if the tag kind is compatible.
+/// \returns true iff the tag kind is compatible.
 static bool isClassCompatTagKind(TagTypeKind Tag)
 {
   return Tag == TTK_Struct || Tag == TTK_Class || Tag == TTK_Interface;
index b4a2210e3d85c19e0b9b83f2930f5683ddf75953..21db9e67c14e4a4e38add3a3f686895d7cee6a99 100644 (file)
@@ -240,8 +240,8 @@ static SymbolRef getAsPointeeSymbol(const Expr *Expr,
 // 1) noErr / [0]
 // 2) someErr / [1, inf]
 // 3) unknown
-// If noError, returns true if (1).
-// If !noError, returns true if (2).
+// If noError, returns true iff (1).
+// If !noError, returns true iff (2).
 bool MacOSKeychainAPIChecker::definitelyReturnedError(SymbolRef RetSym,
                                                       ProgramStateRef State,
                                                       SValBuilder &Builder,
index 43629f316b298d2d2050ff1ea99227fe378e4910..15c9a225ed9a97bab761ae1e8d72b3f5193e9d76 100644 (file)
@@ -76,7 +76,7 @@ public:
     impl(detail::select<impl_t::index(detail::type<U>())>(), move(t.value)) {}
 
   // Destruction disabled to allow use in a constant expression.
-  // FIXME: declare a destructor if any element has a nontrivial destructor
+  // FIXME: declare a destructor iff any element has a nontrivial destructor
   //~either() { impl.destroy(elem); }
 
   constexpr unsigned index() noexcept { return elem; }