[cfe-commits] r43614 - /cfe/trunk/include/clang/AST/Builtins.h

Ted Kremenek kremenek at apple.com
Thu Nov 1 09:35:25 PDT 2007


Author: kremenek
Date: Thu Nov  1 11:35:25 2007
New Revision: 43614

URL: http://llvm.org/viewvc/llvm-project?rev=43614&view=rev
Log:
Reverted unnecessary inlining of operator!=, since negating operator== is
just as efficient and much more succinct.

Modified:
    cfe/trunk/include/clang/AST/Builtins.h

Modified: cfe/trunk/include/clang/AST/Builtins.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Builtins.h?rev=43614&r1=43613&r2=43614&view=diff

==============================================================================
--- cfe/trunk/include/clang/AST/Builtins.h (original)
+++ cfe/trunk/include/clang/AST/Builtins.h Thu Nov  1 11:35:25 2007
@@ -39,12 +39,7 @@
            !strcmp(Type, RHS.Type) &&
            !strcmp(Attributes, RHS.Attributes);
   }
-  
-  bool operator!=(const Info &RHS) const { 
-    return strcmp(Name, RHS.Name) ||
-           strcmp(Type, RHS.Type) ||
-           strcmp(Attributes, RHS.Attributes);    
-  }
+  bool operator!=(const Info &RHS) const { return !(*this == RHS); }
 };
 
 /// Builtin::Context - This holds information about target-independent and





More information about the cfe-commits mailing list