Revert r111082. No warnings for this common pattern.

llvm-svn: 111102

llvm/lib/Analysis/InstCount.cpp

@@ -45,7 +45,7 @@ namespace {
 
 #include "llvm/Instruction.def"
 
-    void visitInstruction(Instruction &I) ATTRIBUTE_UNUSED {
+    void visitInstruction(Instruction &I) {
       errs() << "Instruction Count does not know about " << I;
       llvm_unreachable(0);
     }

llvm/lib/Bitcode/Reader/BitcodeReader.cpp

@@ -135,8 +135,8 @@ namespace {
   /// @brief A class for maintaining the slot number definition
   /// as a placeholder for the actual definition for forward constants defs.
   class ConstantPlaceHolder : public ConstantExpr {
-    ConstantPlaceHolder() ATTRIBUTE_UNUSED;      // DO NOT IMPLEMENT
-    void operator=(const ConstantPlaceHolder &) ATTRIBUTE_UNUSED;//NOT IMPLEMENT
+    ConstantPlaceHolder();                       // DO NOT IMPLEMENT
+    void operator=(const ConstantPlaceHolder &); // DO NOT IMPLEMENT
   public:
     // allocate space for exactly one operand
     void *operator new(size_t s) {

llvm/lib/Linker/LinkModules.cpp

@@ -78,8 +78,8 @@ class LinkerTypeMap : public AbstractTypeUser {
   typedef DenseMap<const Type*, PATypeHolder> TheMapTy;
   TheMapTy TheMap;
 
-  LinkerTypeMap(const LinkerTypeMap&) ATTRIBUTE_UNUSED; // DO NOT IMPLEMENT
-  void operator=(const LinkerTypeMap&) ATTRIBUTE_UNUSED; // DO NOT IMPLEMENT
+  LinkerTypeMap(const LinkerTypeMap&); // DO NOT IMPLEMENT
+  void operator=(const LinkerTypeMap&); // DO NOT IMPLEMENT
 public:
   LinkerTypeMap() {}
   ~LinkerTypeMap() {

llvm/lib/Target/X86/X86MCCodeEmitter.cpp

@@ -19,13 +19,12 @@
 #include "llvm/MC/MCExpr.h"
 #include "llvm/MC/MCInst.h"
 #include "llvm/Support/raw_ostream.h"
-#include "llvm/Support/Compiler.h"
 using namespace llvm;
 
 namespace {
 class X86MCCodeEmitter : public MCCodeEmitter {
-  X86MCCodeEmitter(const X86MCCodeEmitter &) ATTRIBUTE_UNUSED; // DONT IMPLEMENT
-  void operator=(const X86MCCodeEmitter &) ATTRIBUTE_UNUSED; // DO NOT IMPLEMENT
+  X86MCCodeEmitter(const X86MCCodeEmitter &); // DO NOT IMPLEMENT
+  void operator=(const X86MCCodeEmitter &); // DO NOT IMPLEMENT
   const TargetMachine &TM;
   const TargetInstrInfo &TII;
   MCContext &Ctx;

llvm/lib/VMCore/Verifier.cpp

@@ -151,8 +151,8 @@ namespace {
     SmallSetVector<const Type *, 16> Types;
 
     // Disallow copying.
-    TypeSet(const TypeSet &) ATTRIBUTE_UNUSED;
-    TypeSet &operator=(const TypeSet &) ATTRIBUTE_UNUSED;
+    TypeSet(const TypeSet &);
+    TypeSet &operator=(const TypeSet &);
   };
 
   struct Verifier : public FunctionPass, public InstVisitor<Verifier> {

llvm/utils/unittest/googletest/include/gtest/internal/gtest-port.h

@@ -556,12 +556,12 @@
 // A macro to disallow operator=
 // This should be used in the private: declarations for a class.
 #define GTEST_DISALLOW_ASSIGN_(type)\
-  void operator=(type const &) GTEST_ATTRIBUTE_UNUSED_
+  void operator=(type const &)
 
 // A macro to disallow copy constructor and operator=
 // This should be used in the private: declarations for a class.
 #define GTEST_DISALLOW_COPY_AND_ASSIGN_(type)\
-  type(type const &) GTEST_ATTRIBUTE_UNUSED_;\
+  type(type const &);\
   GTEST_DISALLOW_ASSIGN_(type)
 
 // Tell the compiler to warn about unused return values for functions declared