[LLVM] Fix Clang-tidy llvm-namespace-comment warnings

[Eugene.Zelenko]

Summary

I checked this patch on my own build on RHEL 6. Regressions were OK.

Please check it in if it's OK, because I don't have SVN write access.

[hans]

I'm not sure this kind of clean-up is worth the effort. Some of these are kind of nice as consistency improvements, but many seem redundant; for example a short namespace such as 'object' in include/llvm/ExecutionEngine/RuntimeDyld.h which only contains two declarations probably doesn't need a comment.

Same goes for http://reviews.llvm.org/D11549.

[Eugene.Zelenko]

I think it's better to place comments consistently independently from number of declarations in namespace.

At least Clang-tidy llvm-namespace-comment will be happy and will not complain on same part of code in future.

[alexfh]

Apparently, the check is not completely aligned with the preferences of the
community. The relevant rule in the LLVM Coding Standards
http://llvm.org/docs/CodingStandards.html#namespace-indentation doesn't
mandate the use of the namespace ending comments and it uses a slightly
different wording for them ("end namespace xxx" or "end anonymous
namespace"). There's also no recommended minimum namespace size, after
which the comments should be used. Thus the check first needs to be updated
according with the coding standards and community preferences (and maybe
the relevant rule of the coding standards should be made a bit more
specific, so that it's possible to implement in the check).

I already submitted a cleanup patch for this check once (r240270), but had
to revert it later due to the reasons above.

llvm-svn.src/include/llvm/ADT/APInt.h

 /// Performs a bitwise complement operation on APInt.
 inline APInt Not(const APInt &APIVal) { return ~APIVal; }
 
-} // End of APIntOps namespace
+} // namespace APIntOps
 
 // See friend declaration above. This additional declaration is required in
 // order to compile LLVM with IBM xlC compiler.
 hash_code hash_value(const APInt &Arg);
-} // End of llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/ArrayRef.h

   template <typename T> struct isPodLike<ArrayRef<T> > {
     static const bool value = true;
   };
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/DenseMap.h

   ValueT &getSecond() { return std::pair<KeyT, ValueT>::second; }
   const ValueT &getSecond() const { return std::pair<KeyT, ValueT>::second; }
 };
-}
+} // namespace detail
 
 template <
     typename KeyT, typename ValueT, typename KeyInfoT = DenseMapInfo<KeyT>,

llvm-svn.src/include/llvm/ADT/DenseSet.h

   DenseSetEmpty &getSecond() { return *this; }
   const DenseSetEmpty &getSecond() const { return *this; }
 };
-}
+} // namespace detail
 
 /// DenseSet - This implements a dense probed hash-table based set.
 template<typename ValueT, typename ValueInfoT = DenseMapInfo<ValueT> >

llvm-svn.src/include/llvm/ADT/None.h

 /// and similar types without having to spell out the specialization's name.
 enum class NoneType { None };
 const NoneType None = None;
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/Optional.h

 template<typename T, typename U>
 void operator>(const Optional<T> &X, const Optional<U> &Y);
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/PointerUnion.h

                                   RHS.template get<U>()));
     }
   };
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/STLExtras.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/SetVector.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 // vim: sw=2 ai
 #endif

llvm-svn.src/include/llvm/ADT/SmallPtrSet.h

   }
 };
 
-}
+} // namespace llvm
 
 namespace std {
   /// Implement std::swap in terms of SmallPtrSet swap.

   inline void swap(llvm::SmallPtrSet<T, N> &LHS, llvm::SmallPtrSet<T, N> &RHS) {
     LHS.swap(RHS);
   }
-}
+} // namespace std
 
 #endif

llvm-svn.src/include/llvm/ADT/SmallString.h

   }
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/SmallVector.h

   return X.capacity_in_bytes();
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 namespace std {
   /// Implement std::swap in terms of SmallVector swap.

   swap(llvm::SmallVector<T, N> &LHS, llvm::SmallVector<T, N> &RHS) {
     LHS.swap(RHS);
   }
-}
+} // namespace std
 
 #endif

llvm-svn.src/include/llvm/ADT/StringExtras.h

   return join_impl(Begin, End, Separator, tag());
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/StringMap.h

   }
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/StringRef.h

   // StringRefs can be treated like a POD type.
   template <typename T> struct isPodLike;
   template <> struct isPodLike<StringRef> { static const bool value = true; };
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/StringSet.h

       return base::insert(std::make_pair(Key, '\0'));
     }
   };
-}
+} // namespace llvm
 
 #endif // LLVM_ADT_STRINGSET_H

llvm-svn.src/include/llvm/ADT/Triple.h

   /// @}
 };
 
-} // End llvm namespace
+} // namespace llvm
 
-
 #endif

llvm-svn.src/include/llvm/ADT/Twine.h

   }
 
   /// @}
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/ilist.h

   void resize(size_type newsize) { resize(newsize, NodeTy()); }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 namespace std {
   // Ensure that swap uses the fast list swap...

   void swap(llvm::iplist<Ty> &Left, llvm::iplist<Ty> &Right) {
     Left.swap(Right);
   }
-}  // End 'std' extensions...
+} // namespace std
 
 #endif // LLVM_ADT_ILIST_H

llvm-svn.src/include/llvm/ADT/ilist_node.h

   /// @}
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/iterator.h

   T &operator*() const { return **this->I; }
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ADT/iterator_range.h

 iterator_range<decltype(begin(std::declval<T>()))> drop_begin(T &&t, int n) {
   return make_range(std::next(begin(t), n), end(t));
 }
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Bitcode/BitCodes.h

                                       //                             [id, name]
   };
 
-} // End bitc namespace
+} // namespace bitc
 
 /// BitCodeAbbrevOp - This describes one or more operands in an abbreviation.
 /// This is actually a union of two different things:

     OperandList.push_back(OpInfo);
   }
 };
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Bitcode/BitstreamReader.h

   bool ReadBlockInfoBlock();
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/DebugInfo/DIContext.h

   virtual std::unique_ptr<LoadedObjectInfo> clone() const = 0;
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ExecutionEngine/ExecutionEngine.h

 namespace object {
   class Archive;
   class ObjectFile;
-}
+} // namespace object
 
 /// \brief Helper class for helping synchronize access to the global address map
 /// table.  Access to this class should be serialized under a mutex.

     Interpreter = 0x2
   };
   const static Kind Either = (Kind)(JIT | Interpreter);
-}
+} // namespace EngineKind
 
 /// Builder class for ExecutionEngines. Use this by stack-allocating a builder,
 /// chaining the various set* methods, and terminating it with a .create()

 // Create wrappers for C Binding types (see CBindingWrapping.h).
 DEFINE_SIMPLE_CONVERSION_FUNCTIONS(ExecutionEngine, LLVMExecutionEngineRef)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/ExecutionEngine/MCJIT.h

       LLVMLinkInMCJIT();
     }
   } ForceMCJITLinking;
-}
+} // namespace
 
 #endif

llvm-svn.src/include/llvm/ExecutionEngine/RTDyldMemoryManager.h

 
   namespace object {
     class ObjectFile;
-  }
+  } // namespace object
 
 class MCJITMemoryManager : public RuntimeDyld::MemoryManager {
 public:

llvm-svn.src/include/llvm/ExecutionEngine/RuntimeDyld.h

 namespace object {
   class ObjectFile;
   template <typename T> class OwningBinary;
-}
+} // namespace object
 
 class RuntimeDyldImpl;
 class RuntimeDyldCheckerImpl;

llvm-svn.src/include/llvm/ExecutionEngine/SectionMemoryManager.h

   MemoryGroup RODataMem;
 };
 
-}
+} // namespace llvm
 
 #endif // LLVM_EXECUTION_ENGINE_SECTION_MEMORY_MANAGER_H
 

llvm-svn.src/include/llvm/IR/Argument.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Attributes.h

 /// \brief Which attributes cannot be applied to a type.
 AttrBuilder typeIncompatible(const Type *Ty);
 
-} // end AttributeFuncs namespace
+} // namespace AttributeFuncs
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/BasicBlock.h

 // Create wrappers for C Binding types (see CBindingWrapping.h).
 DEFINE_SIMPLE_CONVERSION_FUNCTIONS(BasicBlock, LLVMBasicBlockRef)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/CallingConv.h

     /// in SSE registers.
     X86_VectorCall = 80
   };
-} // End CallingConv namespace
+} // namespace CallingConv
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Comdat.h

   return OS;
 }
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Constant.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Constants.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/DataLayout.h

   }
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/DerivedTypes.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Function.h

 
 DEFINE_TRANSPARENT_OPERAND_ACCESSORS(Function, Value)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/GlobalAlias.h

 
 DEFINE_TRANSPARENT_OPERAND_ACCESSORS(GlobalAlias, Constant)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/GlobalObject.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/GlobalValue.h

 
 namespace Intrinsic {
   enum ID : unsigned;
-}
+} // namespace Intrinsic
 
 class GlobalValue : public Constant {
   GlobalValue(const GlobalValue &) = delete;

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/GlobalVariable.h

 
 DEFINE_TRANSPARENT_OPERAND_ACCESSORS(GlobalVariable, Value)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/InstrTypes.h

 
 DEFINE_TRANSPARENT_OPERAND_ACCESSORS(CmpInst, Value)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Instruction.h

   enum { NumLowBitsAvailable = 2 };
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Instructions.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Intrinsics.h

   /// of IITDescriptors.
   void getIntrinsicInfoTableEntries(ID id, SmallVectorImpl<IITDescriptor> &T);
 
-} // End Intrinsic namespace
+} // namespace Intrinsic
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/LLVMContext.h

   return reinterpret_cast<LLVMContextRef*>(const_cast<LLVMContext**>(Tys));
 }
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/LegacyPassManager.h

   Module *M;
 };
 
-} // End legacy namespace
+} // namespace legacy
 
 // Create wrappers for C Binding types (see CBindingWrapping.h).
 DEFINE_STDCXX_CONVERSION_FUNCTIONS(legacy::PassManagerBase, LLVMPassManagerRef)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Metadata.h

   }
 };
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Module.h

   return reinterpret_cast<Module*>(MP);
 }
   
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/OperandTraits.h

   return this->OpFrom<Idx_nocapture>(this); \
 }
 
+} // namespace llvm
 
-} // End llvm namespace
-
 #endif

llvm-svn.src/include/llvm/IR/SymbolTableListTraits.h

   static ValueSymbolTable *toPtr(ValueSymbolTable &R) { return &R; }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Type.h

   return reinterpret_cast<LLVMTypeRef*>(const_cast<Type**>(Tys));
 }
   
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Use.h

 // Create wrappers for C Binding types (see CBindingWrapping.h).
 DEFINE_SIMPLE_CONVERSION_FUNCTIONS(Use, LLVMUseRef)
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/User.h

   }
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/Value.h

   return reinterpret_cast<LLVMValueRef*>(const_cast<Value**>(Vals));
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/ValueHandle.h

   virtual void allUsesReplacedWith(Value *) {}
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/IR/ValueSymbolTable.h

 /// @}
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/InitializePasses.h

 void initializeFloat2IntPass(PassRegistry&);
 void initializeLoopDistributePass(PassRegistry&);
 void initializeSjLjEHPreparePass(PassRegistry&);
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Object/Binary.h

 }
 
 ErrorOr<OwningBinary<Binary>> createBinary(StringRef Path);
-}
-}
+} // namespace object
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Object/ELFObjectFile.h

   return EF.getHeader()->e_type == ELF::ET_REL;
 }
 
-}
-}
+} // namespace object
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Object/Error.h

 namespace std {
 template <>
 struct is_error_code_enum<llvm::object::object_error> : std::true_type {};
-}
+} // namespace std
 
 #endif

llvm-svn.src/include/llvm/Object/MachO.h

   return OwningObject;
 }
 
-}
-}
+} // namespace object
+} // namespace llvm
 
 #endif
 

llvm-svn.src/include/llvm/Object/RelocVisitor.h

   }
 };
 
-}
-}
+} // namespace object
+} // namespace llvm
+
 #endif

llvm-svn.src/include/llvm/Object/SymbolicFile.h

   return OwningObject;
 }
 
-}
-}
+} // namespace object
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Option/OptSpecifier.h

     bool operator==(OptSpecifier Opt) const { return ID == Opt.getID(); }
     bool operator!=(OptSpecifier Opt) const { return !(*this == Opt); }
   };
-}
-}
+} // namespace opt
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Pass.h

 /// @brief This is the storage for the -time-passes option.
 extern bool TimePassesIsEnabled;
 
-} // End llvm namespace
+} // namespace llvm
 
 // Include support files that contain important APIs commonly used by Passes,
 // but that we want to separate out to make it easier to read the header files.

llvm-svn.src/include/llvm/PassAnalysisSupport.h

   return *(AnalysisType*)ResultPass->getAdjustedAnalysisPointer(PI);
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/PassInfo.h

   PassInfo(const PassInfo &) = delete;
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/PassRegistry.h

 // Create wrappers for C Binding types (see CBindingWrapping.h).
 DEFINE_STDCXX_CONVERSION_FUNCTIONS(PassRegistry, LLVMPassRegistryRef)
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/PassSupport.h

   virtual void passEnumerate(const PassInfo *) {}
 };
 
+} // namespace llvm
 
-} // End llvm namespace
-
 #endif

llvm-svn.src/include/llvm/Support/Atomic.h

     cas_flag AtomicAdd(volatile cas_flag* ptr, cas_flag val);
     cas_flag AtomicMul(volatile cas_flag* ptr, cas_flag val);
     cas_flag AtomicDiv(volatile cas_flag* ptr, cas_flag val);
-  }
-}
+  } // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Casting.h

   return (Val && isa<X>(Val)) ? cast<X>(Val) : nullptr;
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/CodeGen.h

   // Relocation model types.
   namespace Reloc {
     enum Model { Default, Static, PIC_, DynamicNoPIC };
-  }
+  } // namespace Reloc
 
   // Code model types.
   namespace CodeModel {
     enum Model { Default, JITDefault, Small, Kernel, Medium, Large };
-  }
+  } // namespace CodeModel
 
   namespace PICLevel {
     enum Level { Default=0, Small=1, Large=2 };
-  }
+  } // namespace PICLevel
 
   // TLS models.
   namespace TLSModel {

       InitialExec,
       LocalExec
     };
-  }
+  } // namespace TLSModel
 
   // Code generation optimization level.
   namespace CodeGenOpt {

       Default,     // -O2, -Os
       Aggressive   // -O3
     };
-  }
+  } // namespace CodeGenOpt
 
   // Create wrappers for C Binding types (see CBindingWrapping.h).
   inline CodeModel::Model unwrap(LLVMCodeModel Model) {

     }
     llvm_unreachable("Bad CodeModel!");
   }
-}  // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/DataStream.h

 
 std::unique_ptr<DataStreamer> getDataFileStreamer(const std::string &Filename,
                                                   std::string *Err);
-}
+} // namespace llvm
 
 #endif  // LLVM_SUPPORT_DATASTREAM_H_

llvm-svn.src/include/llvm/Support/Debug.h

 //
 #define DEBUG(X) DEBUG_WITH_TYPE(DEBUG_TYPE, X)
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/DynamicLibrary.h

     static void AddSymbol(StringRef symbolName, void *symbolValue);
   };
 
-} // End sys namespace
-} // End llvm namespace
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/ErrorHandling.h

   LLVM_ATTRIBUTE_NORETURN void
   llvm_unreachable_internal(const char *msg=nullptr, const char *file=nullptr,
                             unsigned line=0);
-}
+} // namespace llvm
 
 /// Marks that the current location is not supposed to be reachable.
 /// In !NDEBUG builds, prints the message and location info to stderr.

llvm-svn.src/include/llvm/Support/FileSystem.h

     intptr_t IterationHandle;
     directory_entry CurrentEntry;
   };
-}
+} // namespace detail
 
 /// directory_iterator - Iterates through the entries in path. There is no
 /// operator++ because we need an error_code. If it's really needed we can make

     uint16_t Level;
     bool HasNoPushRequest;
   };
-}
+} // namespace detail
 
 /// recursive_directory_iterator - Same as directory_iterator except for it
 /// recurses down into child directories.

llvm-svn.src/include/llvm/Support/FileUtilities.h

     /// will not be removed when the object is destroyed.
     void releaseFile() { DeleteIt = false; }
   };
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/FormattedStream.h

 /// debug output.  Use it like: fdbgs() << "foo" << "bar";
 formatted_raw_ostream &fdbgs();
 
-} // end llvm namespace
+} // namespace llvm
 
-
 #endif

llvm-svn.src/include/llvm/Support/Host.h

   ///
   /// \return - True on success.
   bool getHostCPUFeatures(StringMap<bool> &Features);
-}
-}
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/MD5.h

   const uint8_t *body(ArrayRef<uint8_t> Data);
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/ManagedStatic.h

   ~llvm_shutdown_obj() { llvm_shutdown(); }
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/MathExtras.h

 }
 
 extern const float huge_valf;
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Memory.h

     /// as writable.
     static bool setRangeWritable(const void *Addr, size_t Size);
   };
-}
-}
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/MemoryObject.h

   virtual bool isValidAddress(uint64_t address) const = 0;
 };
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Mutex.h

     };
 
     typedef SmartScopedLock<false> ScopedLock;
-  }
-}
+  } // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Program.h

       ///< string is non-empty upon return an error occurred while invoking the
       ///< program.
       );
-  }
-}
+  } // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/RWMutex.h

       }
     };
     typedef SmartScopedWriter<false> ScopedWriter;
-  }
-}
+  } // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Regex.h

     struct llvm_regex *preg;
     int error;
   };
-}
+} // namespace llvm
 
 #endif // LLVM_SUPPORT_REGEX_H

llvm-svn.src/include/llvm/Support/Registry.h

   template <typename T, typename U>
   typename Registry<T,U>::listener *Registry<T,U>::ListenerTail;
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Signals.h

   /// different thread on some platforms.
   /// @brief Register a function to be called when ctrl-c is pressed.
   void SetInterruptFunction(void (*IF)());
-} // End sys namespace
-} // End llvm namespace
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/SourceMgr.h

              bool ShowKindLabel = true) const;
 };
 
-}  // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/StreamingMemoryObject.h

 MemoryObject *getNonStreamedMemoryObject(
     const unsigned char *Start, const unsigned char *End);
 
-}
+} // namespace llvm
+
 #endif  // STREAMINGMEMORYOBJECT_H_

llvm-svn.src/include/llvm/Support/TargetSelect.h

 #endif
   }  
 
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Threading.h

   /// the thread stack.
   void llvm_execute_on_thread(void (*UserFn)(void*), void *UserData,
                               unsigned RequestedStackSize = 0);
-}
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/TimeValue.h

   return difference;
 }
 
-}
-}
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/Timer.h

   void PrintQueuedTimers(raw_ostream &OS);
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/UniqueLock.h

 
     bool owns_lock() { return locked; }
   };
-}
+} // namespace llvm
 
 #endif // LLVM_SUPPORT_UNIQUE_LOCK_H

llvm-svn.src/include/llvm/Support/Valgrind.h

   #define TsanIgnoreWritesBegin()
   #define TsanIgnoreWritesEnd()
 #endif
-}
-}
+} // namespace sys
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/raw_os_ostream.h

   ~raw_os_ostream() override;
 };
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/raw_ostream.h

 namespace sys {
 namespace fs {
 enum OpenFlags : unsigned;
-}
-}
+} // namespace fs
+} // namespace sys
 
 /// This class implements an extremely fast bulk output stream that can *only*
 /// output to a stream.  It does not support seeking, reopening, rewinding, line

   ~buffer_ostream() { OS << str(); }
 };
 
-} // end llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Support/type_traits.h

   typedef const typename std::remove_pointer<T>::type *type;
 };
 
-}
+} // namespace llvm
 
 #ifdef LLVM_DEFINED_HAS_FEATURE
 #undef __has_feature

llvm-svn.src/include/llvm/Target/TargetMachine.h

 // The old pass manager infrastructure is hidden in a legacy namespace now.
 namespace legacy {
 class PassManagerBase;
-}
+} // namespace legacy
 using legacy::PassManagerBase;
 
 //===----------------------------------------------------------------------===//

                          bool DisableVerify = true) override;
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Target/TargetOptions.h

       Soft,    // Soft float.
       Hard     // Hard float.
     };
-  }
+  } // namespace FloatABI
 
   namespace FPOpFusion {
     enum FPOpFusionMode {

       Standard, // Only allow fusion of 'blessed' ops (currently just fmuladd).
       Strict    // Never fuse FP-ops.
     };
-  }
+  } // namespace FPOpFusion
 
   namespace JumpTable {
     enum JumpTableType {

                        // primitive, and function pointer.
       Full             // Use one table per unique function type
     };
-  }
+  } // namespace JumpTable
 
   namespace ThreadModel {
     enum Model {
       POSIX,  // POSIX Threads
       Single  // Single Threaded Environment
     };
-  }
+  } // namespace ThreadModel
 
   class TargetOptions {
   public:

   return !(LHS == RHS);
 }
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Target/TargetRecip.h

   void parseIndividualParams(const std::vector<std::string> &Args);
 };
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif

llvm-svn.src/include/llvm/Transforms/IPO.h

 /// to bitsets.
 ModulePass *createLowerBitSetsPass();
 
-} // End llvm namespace
+} // namespace llvm
 
 #endif