<p dir="ltr">LGTM</p>
<div class="gmail_quote">2015/11/03 16:48 "Eugene Zelenko" <<a href="mailto:eugene.zelenko@gmail.com">eugene.zelenko@gmail.com</a>>:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eugene.Zelenko created this revision.<br>
Eugene.Zelenko added a reviewer: ruiu.<br>
Eugene.Zelenko added a subscriber: llvm-commits.<br>
Eugene.Zelenko set the repository for this revision to rL LLVM.<br>
<br>
I checked this patch on my own build on RHEL 6. Regressions were OK.<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14310" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14310</a><br>
<br>
Files:<br>
  ELF/OutputSections.h<br>
  lib/ReaderWriter/MachO/SectCreateFile.h<br>
<br>
Index: lib/ReaderWriter/MachO/SectCreateFile.h<br>
===================================================================<br>
--- lib/ReaderWriter/MachO/SectCreateFile.h<br>
+++ lib/ReaderWriter/MachO/SectCreateFile.h<br>
@@ -23,7 +23,6 @@<br>
 //<br>
 class SectCreateFile : public File {<br>
 public:<br>
-<br>
   class SectCreateAtom : public SimpleDefinedAtom {<br>
   public:<br>
     SectCreateAtom(const File &file, StringRef segName, StringRef sectName,<br>
@@ -68,19 +67,19 @@<br>
       new (allocator()) SectCreateAtom(*this, seg, sect, std::move(content)));<br>
   }<br>
<br>
-  const AtomVector<DefinedAtom> &defined() const {<br>
+  const AtomVector<DefinedAtom> &defined() const override {<br>
     return _definedAtoms;<br>
   }<br>
<br>
-  const AtomVector<UndefinedAtom> &undefined() const {<br>
+  const AtomVector<UndefinedAtom> &undefined() const override {<br>
     return _noUndefinedAtoms;<br>
   }<br>
<br>
-  const AtomVector<SharedLibraryAtom> &sharedLibrary() const {<br>
+  const AtomVector<SharedLibraryAtom> &sharedLibrary() const override {<br>
     return _noSharedLibraryAtoms;<br>
   }<br>
<br>
-  const AtomVector<AbsoluteAtom> &absolute() const {<br>
+  const AtomVector<AbsoluteAtom> &absolute() const override {<br>
     return _noAbsoluteAtoms;<br>
   }<br>
<br>
Index: ELF/OutputSections.h<br>
===================================================================<br>
--- ELF/OutputSections.h<br>
+++ ELF/OutputSections.h<br>
@@ -247,7 +247,7 @@<br>
 class InterpSection final : public OutputSectionBase<ELFT> {<br>
 public:<br>
   InterpSection();<br>
-  void writeTo(uint8_t *Buf);<br>
+  void writeTo(uint8_t *Buf) override;<br>
 };<br>
<br>
 template <class ELFT><br>
@@ -385,6 +385,8 @@<br>
 template <class ELFT> StringTableSection<ELFT> *Out<ELFT>::StrTab;<br>
 template <class ELFT> SymbolTableSection<ELFT> *Out<ELFT>::DynSymTab;<br>
 template <class ELFT> SymbolTableSection<ELFT> *Out<ELFT>::SymTab;<br>
-}<br>
-}<br>
-#endif<br>
+<br>
+} // namespace elf2<br>
+} // namespace lld<br>
+<br>
+#endif // LLD_ELF_OUTPUT_SECTIONS_H<br>
<br>
<br>
</blockquote></div>